Не удается вставить данные

Question

Я не могу найти ошибку, почему мои данные не были вставлены в базу данных. Я использую сервер sql. DB - 3 поля: 1) id (int) 2) test1 (varchar (50)) 3) test2 (varchar (50)). Я пытаюсь вставить данные из текстового поля и даты.

private void SaveToDB() 
    { 
     string strSql = ""; 
     string[] tos = txtTo.Text.Split(';'); 
     for (int i = 0; i < tos.Length; i++) 
     { 
      strSql += "INSERT INTO test (test1, test2) VALUES ('" + txtContent.Text.Trim() + "','" + DateTime.Now.ToString() + "');"; 
     } 
     using (SqlConnection connection = new SqlConnection(Common.ConnetionString)) 
     { 
      connection.Open(); 
      SqlTransaction tran = connection.BeginTransaction(); 
      try 
      { 
       SqlCommand cmd = new SqlCommand(strSql, connection, tran); 
       cmd.ExecuteNonQuery(); 
       tran.Commit(); 
      } 
      catch (Exception e) 
      { 
       tran.Rollback(); 
      } 
      finally 
      { 
       connection.Close(); 
       Response.Redirect("messagelist.aspx?flag=2"); 
      } 
     } 
    } 

Но как я должен изменить код, что параметр работы в цикле

Answer 1

Ваш цикл, вероятно, следует охватить весь блок. Кроме того, параметризуйте свой запрос, чтобы вы не были широко открыты для инъекций. Просто предложение, но я бы также использовал BEGIN TRANSACTION в вашем заявлении sql, противоположном SqlTransaction tran, он обработает откат для вас и немного очистит ваш код.

string[] tos = txtTo.Text.Split(';'); 
for (int i = 0; i < tos.Length; i++) 
{ 
    string strSql = "INSERT INTO test (test1, test2) VALUES (@Content, @DateTime);"; 
    using (SqlConnection connection = new SqlConnection(Common.ConnetionString)) 
    { 
     connection.Open(); 
     SqlTransaction tran = connection.BeginTransaction(); 
     try 
     { 
      using (SqlCommand cmd = new SqlCommand(strSql, connection, tran)) 
      { 
       cmd.Parameters.AddWithValue("@Content", txtContent.Text.Trim()); 
       cmd.Parameters.AddWithValue("@DateTime", DateTime.Now.ToString()); 
       cmd.ExecuteNonQuery(); 
       tran.Commit(); 
      } 
     } 
     catch (Exception e) 
     { 
      tran.Rollback(); 
     } 
     finally 
     { 
      connection.Close(); 
      Response.Redirect("messagelist.aspx?flag=2"); 
     } 
    } 
}