CODEWITHSUNDEEP
c#sqlvisual-studio
spacebarisdefect
spacebarisdefect

Reputation: 75

SQL Insert Data not working properly

I want to insert some values into my table based on a users input. I get these values from some textboxes. When i put a breakpoint before my SQL logic, and go through every line with F10, everything works just fine. I open the SQLConnection, create a SQLCommand, execute it, and close the connection again. I refresh the table and have all the values in there. But when I delete or disable the breakpoint, and the programm goes over these lines of code by itself, it doesn't work, meaning no values are being added to the table, no matter how often i refresh.

Here is the code I'm referring to:

try
{
     SqlConnection con = new SqlConnection("Server=...;Database=...;Integrated Security=true;");
     con.Open();

     SqlCommand com = new SqlCommand("INSERT INTO TestTable (Type,Serialnumber) VALUES('" + TypeText + "','" + SerText + "')", con);
     //Debug.WriteLine(com.CommandText);

     com.BeginExecuteNonQuery();

     con.Close();
}
catch(Exception ex)
{
     MessageBox.Show(ex.Message);
}

Upvotes: 0

Views: 227

Answers (4)

Amit Pore
Amit Pore

Reputation: 127

Best to use using with ExecuteNonQuery than BeginExecuteNonQuery

 try
            {
                using (SqlConnection conn = new SqlConnection("your Connection string"))
                {
                    conn.Open();

                    using (SqlCommand cmd = conn.CreateCommand())
                    {
                         cmd.CommandText = @"INSERT INTO TestTable 
                                                (Type,Serialnumber )
                                                VALUES (@type,@serialnumber)";


                        cmd.Parameters.AddWithValue("@type", type);
                        cmd.Parameters.AddWithValue("@serialnumber", serialnumber);

                        int rows = cmd.ExecuteNonQuery();

                        if (rows == 0)
                            throw new Exception("Nothing Inserted into the DB");
                    }   
                }
            }
            catch (Exception ex)
            {
            }

Upvotes: 0

Monkey D. luffy
Monkey D. luffy

Reputation: 28

you need to consider many things before writing any DML code, kindly consider below link which gives you the way to handling resources.

https://stackoverflow.com/a/12142911/6783279

Upvotes: 0

Mrunalini
Mrunalini

Reputation: 256

The BeginExecuteNonQuery method starts the process of asynchronously executing a Transact-SQL statement or stored procedure that does not return rows, so that other tasks can run concurrently while the statement is executing. When the statement has completed, developers must call the EndExecuteNonQuery method to finish the operation. The BeginExecuteNonQuery method returns immediately (CommandTimeout has no effect on BeginExecuteNonQuery), but until the code executes the corresponding EndExecuteNonQuery method call, it must not execute any other calls that start a synchronous or asynchronous execution against the same SqlCommand object. Calling the EndExecuteNonQuery before the command's execution is completed causes the SqlCommand object to block until the execution is finished.

Upvotes: 0

Cleptus
Cleptus

Reputation: 3541

When it runs on your own, it calls BeginExecuteNonQuery() and right next line connection is closed without waiting the async call to end. While using the debugger doing F10 you give it time to efectively end the execution. Consider using ExecuteNonQuery() instead, note the lack of the Begin.

Upvotes: 1

Related Questions