A couple of things:
Firstly, don't do it like that: 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.
Secondly, When you use a DataAdapter and call ExecuteNonQuery on teh same command:
SqlDataAdapter objda = new SqlDataAdapter(objcmd);
objda.Fill(objds);
objcmd.ExecuteNonQuery();
It is going to insert the same row twice...
Thirdy, never "swallow" exceptions: log them, tell the user, whatever - do something with them. When you use an empty
catch
block you hide any evidence of what your code's problems might be, even from yourself. And you often need that to fix it...