I wrote the following code to insert records into a database.  The user selects the rows from the RadGrid and the insert command is executed when that user clicks the button.  I've been spending a lot of time trying to make this work (nothing currently happens when I click the button).  However, I just realized my approach is probably very inefficient due to the fact that it will result in the application repeatedly opening and closing database connections.  Even though I'm dealing with only a few hundred records I'd like to follow best practices as much as possible.How would you rewrite this code to be more efficient? My gut tells me it would be better to gather all of the new records and then deal with them as a collection.    protected void RadButton2_Click(object sender, EventArgs e){    foreach (GridDataItem item in RadGrid1.SelectedItems)    {        //GridDataItem item = (GridDataItem)RadGrid1.SelectedItems;        int UserID = Convert.ToInt16(item[UserID].Text);        string Type = D;        DateTime Date = DateTime.Now;        string Description = Monthly Storage Fee - Tag:  + item[PackageTag].Text + Label3.Text;        Int32 AmountDue = Convert.ToInt32(item[AmtDue].Text);        string connectionString = ConfigurationManager.ConnectionStrings[Foo].ConnectionString;        SqlConnection connection = new SqlConnection(connectionString);        try        {            SqlCommand cmd = new SqlCommand(INSERT INTO Billing (UserID, Type, Date, Description, Amount) VALUES (@UserID, @Type, @Date, @Description, @AmountDue), connection);            cmd.Parameters.AddWithValue(@UserID, UserID);            cmd.Parameters.AddWithValue(@Type, Type);            cmd.Parameters.AddWithValue(@Date, Date);            cmd.Parameters.AddWithValue(@Description, Description);            cmd.Parameters.AddWithValue(@AmountDue, AmountDue);            connection.Open();            cmd.ExecuteNonQuery();        }        catch        {            Label4.Text = uh oh;        }        finally        {            connection.Close();        }
Inserting records into a database
c#;asp.net;sql
This is not best practice but simple logic.Why you have add inside the loop this (non change) parameters ?    string Type = D;    DateTime Date = DateTime.Now; string connectionString =        ConfigurationManager.ConnectionStrings[Foo].ConnectionString;Get them out of the loop.Second place the connection inside using, and totally outside the loop, why you need to open it and close it all the time.protected void RadButton2_Click(object sender, EventArgs e){    //GridDataItem item = (GridDataItem)RadGrid1.SelectedItems;     string Type = D;    DateTime Date = DateTime.Now;    string connectionString = ConfigurationManager.ConnectionStrings[Foo].ConnectionString;    SqlConnection connection = new SqlConnection(connectionString);    using(SqlConnection connection = new SqlConnection(connectionString))    {        connection.Open();        foreach (GridDataItem item in RadGrid1.SelectedItems)        {            try            {                string Description = Monthly Storage Fee - Tag:  + item[PackageTag].Text + Label3.Text;                Int32 AmountDue = Convert.ToInt32(item[AmtDue].Text);                int UserID = Convert.ToInt16(item[UserID].Text);                using (SqlCommand cmd = new SqlCommand(INSERT INTO Billing (UserID, Type, Date, Description, Amount) VALUES (@UserID, @Type, @Date, @Description, @AmountDue), connection))                {                    cmd.Parameters.AddWithValue(@UserID, UserID);                    cmd.Parameters.AddWithValue(@Type, Type);                    cmd.Parameters.AddWithValue(@Date, Date);                    cmd.Parameters.AddWithValue(@Description, Description);                    cmd.Parameters.AddWithValue(@AmountDue, AmountDue);                    cmd.ExecuteNonQuery();                }            }            catch            {                Label4.Text = uh oh;            }        }        }}Need some extra test for check if the connection is open or fail