Обновление нескольких строк в DataGridView C#

Question

private void button1_Click(object sender, EventArgs e) 
{ 
    cn.Open(); 
    SqlCommand cmd = new SqlCommand("UPDATE MEDICINE_REGISTRATION SET stock = @RStock WHERE id= " + Order_Datagridview.SelectedRows[0].Cells["id"].Value.ToString() + "", cn); 

    foreach (DataGridViewRow row in Order_Datagridview.Rows) 
    { 

     // cmd.Parameters.AddWithValue("@RStock", row.Cells["RStock"].Value); 

     cmd.Parameters.Add("@RStock", SqlDbType.VarChar).Value = Order_Datagridview.SelectedRows[0].Cells["RStock"].Value.ToString(); 
     cmd.ExecuteNonQuery(); 
     MessageBox.Show("Success"); 
     cmd.Parameters.Clear(); 

    } 
    cn.Close(); 
} 

Answer 1

Инициализация условия WHERE находится вне цикла и всегда содержит идентификатор первой строки, а не идентификатор последующих строк. параметров должны быть добавлены для каждого значения, которое вы принимаете от вашей сетки, а также идентификатор должен быть параметризованным

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

private void button1_Click(object sender, EventArgs e) 
{ 
    cn.Open(); 
    SqlCommand cmd = new SqlCommand(@"UPDATE MEDICINE_REGISTRATION 
            SET stock = @RStock 
            WHERE id= @id", cn); 

    cmd.Parameters.Add("@RStock", SqlDbType.VarChar).Value = ""; 
    cmd.Parameters.Add("@id", SqlDbType.Integer).Value = 0; 

    foreach (DataGridViewRow row in Order_Datagridview.Rows) 
    { 
     // Use the row indexer here, not the fixed row at index zero 
     cmd.Parameters["@RStock"].Value = row.Cells["RStock"].Value.ToString(); 
     cmd.Parameters["@id"].Value = Convert.ToInt32(row.Cells["id"].Value); 
     cmd.ExecuteNonQuery(); 
    } 
    cn.Close();} 
} 

Есть что сказать и об использовании одноразовых объектов, таких как соединение и команда. Эти объекты должны быть созданы при необходимости и размещены сразу после использования. Проверить о using statement для этой модели

Answer 2

возможно

foreach (DataGridViewRow row in Order_Datagridview.SelectedRows) 
{ 
    ..... DataGridViewRow.Cells["RStock"].Value.ToString(); 
} 

?

Answer 3

Внутри вашего Еогеаспа, вы не используете DataGridViewRow, вы все еще используете DataGridView.Selected [0]. Если вы хотите сделать это, я хотел бы сделать пару изменений:

1. сохранить возвращаемое значение ExecuteNonQuery() и MessageBox.Show, что значение
2. в cmd.Parameters, убедитесь, что вы добавить текущую строку ,
3. Я бы, вероятно, создал отдельную команду для каждой строки.

Что-то вроде этого, может быть:

private void button1_Click(object sender, EventArgs e) 
{ 
    cn.Open(); 

    foreach (DataGridViewRow row in Order_Datagridview.Rows) 
    { 
     SqlCommand cmd = new SqlCommand("UPDATE MEDICINE_REGISTRATION SET stock = " + row.cless["RStock"].Value + " WHERE id= " + row.Cells["id"].Value.ToString() + " ", cn); 

     MessageBox.Show(cmd.ExecuteNonQuery()); 

    } 

    cn.Close(); 
} 

Если это не сработает, попробуйте схватить DataTable или DataView источник, лежащий в основе этой таблицы и сделать то же самое, что и выше.