null значение исключение при вводе в базу данных

Question

Я создал таблицу в базе данных, основанной на услугах, для хранения информации о продажах. В таблицу включены столбцы дата, время, имя, покупка, количество, стоимость (в этом конкретном виде). код для вставки в таблицу

SqlConnection conn = new SqlConnection(@"Data Source=(LocalDB)\v11.0;AttachDbFilename=C:\Users\rnawa_000\Documents\Visual Studio 2013\Projects\Random\Random\sales.mdf;Integrated Security=True"); 
SqlCommand cmd = new SqlCommand(); 
cmd.Connection = conn; 
conn.Open(); 
cmd.CommandText = "insert into salesTB (Date,Time,Name,Quantity,Cost) values ('"+date.Text+"','"+time.Text+"','"+txtName.Text+"','"+listBox1.Items.Count+"','"+ txtCost.Text+")"; 

foreach (string item in listBox1.Items) 
{ 
    cmd.CommandText = "insert into salesTB (Purchase) values ('" +item.Substring(0,10) + "')"; 
} 
cmd.ExecuteNonQuery(); 
cmd.Clone(); 
conn.Close(); 

В date.text и time.text этикетки отображает дату и время respectively.they присваивается значения в случае загрузки формы. в txtName.text и txtCost.text метки получают свои значения из предыдущей формы с помощью конструктора, как

public Form2(ListBox.ObjectCollection objectCollection, string name,string total) 
{ 
    InitializeComponent(); 
    this.listBox1.Items.AddRange(objectCollection); 
    txtName.Text = name; 
    txtCost.Text = total; 
} 

Когда я выполнения этого, он дает мне ошибку говоря «Не удается вставить значение NULL в столбец" Стоимость». И если я установил разрешить nulls в столбце затрат в базе данных, тогда он показывает любое другое имя столбца в ошибке исключения.

Answer 1

Чтобы избежать ошибок, вам необходимо использовать параметризованный запрос, поскольку вы объединяете явные строковые значения. Также ваша логика ошибочна, поэтому это приведет к ненормированным данным, которые будут ужасны для любого администратора базы данных. Я бы подумал об использовании этого вместо этого:

SqlConnection conn = new SqlConnection(@"Data Source=(LocalDB)\v11.0;AttachDbFilename=C:\Users\rnawa_000\Documents\Visual Studio 2013\Projects\Random\Random\sales.mdf;Integrated Security=True"); 

SqlCommand cmd = new SqlCommand(); 

cmd.Connection = conn; 
conn.Open(); 

foreach (string item in listBox1.Items) 
{ 
    cmd.CommandText = "insert into salesTB (Date,Time,Name,Quantity,Cost,Purchase) values (@date, @time, @name, @quantity, @cost, @purchase)"; 

    cmd.Parameters.Add(new SqlParameter("date", date.Text)); 
    cmd.Parameters.Add(new SqlParameter("time", time.Text)); 
    cmd.Parameters.Add(new SqlParameter("name", txtName.Text)); 
    cmd.Parameters.Add(new SqlParameter("quantity", listBox1.Items.Count)); 
    cmd.Parameters.Add(new SqlParameter("cost", txtCost.Text)); 
    cmd.Parameters.Add(new SqlParameter("purchase", item.Substring(0,10))); 

    cmd.ExecuteNonQuery(); 
    cmd.Clone(); 
} 

conn.Close(); 

Это не проверено, но вы получаете эту идею.

Answer 2

Вот лучше и безопасный способ сделать то, что ваш после:

 string SQLQuery = "INSERT INTO salesTB (Date,Time,Name,Quantity,Cost)" + 
          "VALUES (@date, @time, @name, @quantity, @cost)"; 

     using (SqlConnection DBConn = new SqlConnection(cs.ToString())) 
     { 
      using (SqlCommand sqlCmd = new SqlCommand(SQLQuery, DBConn)) 
      { 
       sqlCmd.Parameters.Add("@date", SqlDbType.Text); 
       sqlCmd.Parameters["@date"].Value = date.Text; 

       sqlCmd.Parameters.Add("@time", SqlDbType.Text); 
       sqlCmd.Parameters["@time"].Value = time.Text; 

       sqlCmd.Parameters.Add("@name", SqlDbType.Text); 
       sqlCmd.Parameters["@name"].Value = txtName.Text; 

       sqlCmd.Parameters.Add("@quantity", SqlDbType.Int); 
       sqlCmd.Parameters["@quantity"].Value = listBox1.Items.Count; 

       sqlCmd.Parameters.Add("@cost", SqlDbType.Text); 
       sqlCmd.Parameters["@cost"].Value = txtCost.Text 

       DBConn.Open(); 
       sqlCmd.ExecuteNonQuery(); 
       DBConn.Close(); 
      } 
     }