There are so many things wrong with that, that I'm not surprised that it doesn't do what you want.
1) Don't use VS Default names - You may remember today that TextBox11 holds the Favourite place (or is that TextBox17?) but you won't next week. Use sensible names instead.
2) Do not concatenate strings to build a SQL command. It leaves you wide open to accidental or deliberate SQL Injection attack which can destroy your entire database. Use Parametrized queries instead.
3) Don't trap an error and then ignore it - or strange things happen and you don't know why. Errors mean there is a problem: fix it, or report it. Never ignore it.
4) Never store passwords in clear text - it is a major security risk. There is some information on how to do it here:
Password Storage: How to do it.[
^]
5) Never assume that the user can type: check the email address could be valid, the mobile number is a number rather than a rude message. And so on.
Do the above, and then try again.