我的目标是更新每个SKU的QTY
。我正在使用SqlDataAdapter
来完成此任务。程序运行正常。只是没有结果发生。
问题:为什么没有结果?我的数据库保持不变。
以下代码
public static void updateInventoryfromAMZ(DataTable datatable)
{
int index = 0;
string connString = System.Configuration.ConfigurationManager.ConnectionStrings["MyConnectionString"].ToString();
DataSet amzInventoryDataSet = new DataSet("AMZINVDATASET");
amzInventoryDataSet.Tables.Add(datatable);
// FOR EACH ROW - PERFORM AN UPDATE //
using (SqlConnection connection = new SqlConnection(connString))
{
SqlDataAdapter adapter = new SqlDataAdapter();
foreach (DataRow row in amzInventoryDataSet.Tables[index].Rows)
{
string sku = datatable.Rows[index]["seller-sku"].ToString();
string qty = datatable.Rows[index]["quantity"].ToString();
// Create the UpdateCommand.
SqlCommand command = new SqlCommand(
"UPDATE Inventory SET qty = @qty" +
"WHERE sku = @sku", connection);
// Add the parameters for the UpdateCommand.
command.Parameters.Add("@qty", SqlDbType.Int, qty.Length, qty);
command.Parameters.Add("@sku", SqlDbType.VarChar, sku.Length, sku);
adapter.UpdateCommand = command;
adapter.Update(amzInventoryDataSet.Tables[index]);
index++;
}
}
}
答案 0 :(得分:2)
我在想你的问题是这些问题:
"UPDATE Inventory SET qty = @qty" +
"WHERE sku = @sku", connection);
他们将生成一个字符串,如:
"UPDATE Inventory SET qty = @qtyWHERE sku = @sku", connection);
并未定义变量@qtyWHERE
。
请改为尝试:
"UPDATE Inventory SET qty = @qty WHERE sku = @sku", connection);
答案 1 :(得分:2)
当你连接两个sql字符串时,最好在第一个字符串的末尾或第二个字符串的开头添加空格。正如Gordon Linoff指出你的sql语句不正确。并且还需要根据参数的类型更改设置参数和值。
尝试以下代码,我使用SqlCommand
和ExecuteNonQuery
方法更新每一行数据
using (SqlConnection connection = new SqlConnection(connString))
using (SqlCommand cmd = new SqlCommand("UPDATE Inventory SET qty = @qty WHERE sku = @sku", connection))
{
connection.Open();
var paramqty= cmd.Parameters.Add("@qty", SqlDbType.Int);
var parasku = cmd.Parameters.Add("@sku", SqlDbType.VarChar);
foreach (DataRow row in amzInventoryDataSet.Tables[0].Rows)
{
parasku.Value = row["seller-sku"].ToString();
paramqty.Value = int.Parse(row["quantity"].ToString());
cmd.ExecuteNonQuery();
}
}