リストビューからリストボックスへの転送とSQL Serverデータベースの更新

Question

私はlistviewからlistboxへのデータをSQLクエリで転送するフォームを持っています。エラーがなく、SQLコードがクエリを読み込んでいて、listviewからlistboxにデータを転送していますが、私のSqlCommandは実行中ではない/私のdatagridviewを更新しているようです。これが正しい場所であるかどうかわかりませんSqlCommand - これを解決するのを手伝ってください。

for (int intCount = 0; intCount < listViewOrders.Items.Count; intCount++) 
{ 
    listBoxInvoice.Items.Add(listViewOrders.Items[intCount].Text); 
    listBoxInvoice.Items.Add("        x" + listViewOrders.Items[intCount].SubItems[2].Text + 
          "     @    " + listViewOrders.Items[intCount].SubItems[1].Text); 

    connect.Open(); 

    SqlCommand cmd = new SqlCommand("UPDATE tblProducts SET productQuantity = productQuantity - " + 
          listViewOrders.Items[intCount].SubItems[2].Text + "WHERE productName = '" + listViewOrders.Items[intCount].Text + "'", connect); 

    connect.Close(); 
} 

dataGridProd.Update(); 
dataGridProd.Refresh(); 

ありがとうございます。どんなタイプの応答も高く評価されています。

Answer 1

私はそれが具体的にWHERE句であなたのクエリの間隔と思います。

だから、このようにそれを実行します。

SqlCommand cmd = new SqlCommand("UPDATE tblProducts SET productQuantity = productQuantity - " + listViewOrders.Items[intCount].SubItems[2].Text + " WHERE productName = '" + listViewOrders.Items[intCount].Text + "'", connect); 
cmd.ExecuteNonQuery(); 

か、文字列の補間を使用し、その後+ C＃6を使用している場合。

var cmd = new SqlCommand($@"UPDATE tblProducts SET productQuantity = productQuantity - {listViewOrders.Items[intCount].SubItems[2].Text} WHERE productName = '{listViewOrders.Items[intCount].Text}'", connect); 
cmd.ExecuteNonQuery(); 

は、あなたがデータを 更新しているので、ExecuteNonQueryを使用して、それを実行することを忘れないでください。

そして、listViewOrders.Items[intCount].Textに値が設定されていることを確認してください。チェックするためにブレークポイントを追加することができます。あなたがSQLパラメータを使用している場合

=============== UPDATE ================

はるかに良いですSQLインジェクションを避ける。

SqlCommand cmd = new SqlCommand("UPDATE tblProducts SET productQuantity = productQuantity - @quantity WHERE productName = @productName", connect); 
cmd.Parameters.AddWithValue("@quantity", listViewOrders.Items[intCount].SubItems[2].Text); 
cmd.Parameters.AddWithValue("@productName", listViewOrders.Items[intCount].Text); 
cmd.ExecuteNonQuery();