Радиокнопка в DataGridView с помощью C#

Question

У меня есть сетка данных и у меня есть две кнопки радио в DataGrid то есть одобрять или отклонять, а также одна кнопка в сетке, т.е. представить

то, что я хочу сделать, это в случае утверждения выбрана кнопка и отправить кнопку в datagrid для одной строки. Я хочу, чтобы данные этой строки сохранялись в базе данных с Isactive как 1, и строка должна быть удалена из datagrid, но данные должны храниться в базе данных.

Аналогично, когда отвергаем кнопка выбрана и кнопка отправки в DataGrid щелкают, данные этой строки должны быть сохранены в базе данных с IsActive равным 0, и строка должна быть удалена из DataGrid

детали должны быть хранящихся в базе данных.

Активна не обновляется в базе данных

Может кто-то пожалуйста, скажите мне, что случилось в моем коде? Ниже моего C# код, который я пытался ..

protected void submit(object sender, EventArgs e) 
{ 
    // *Get the Gridview Row* // 
    DataGridItem drow = (DataGridItem)(sender as Control).Parent.Parent; 

    RadioButton rbpApprove = (RadioButton)drow.FindControl("rbtnapprove"); 
    RadioButton rbpReject = (RadioButton)drow.FindControl("rbtnreject"); 

    if (rbpApprove.Checked == true) 
    { 
     conn.Open(); 
     SqlCommand cmd = new SqlCommand("Update table set IsActive= 0 where ARGID=@ARGID", conn); 

     cmd.ExecuteNonQuery(); 
     conn.Close();   
    } 
    else if (rbpReject.Checked == true) 
    { 
     conn.Open(); 
     SqlCommand cmd = new SqlCommand("Update table set IsActive= 1 where ARGID=@ARGID", conn); 
     cmd.ExecuteNonQuery(); 
     conn.Close(); 
    } 


    string empid = dgi.Cells[0].Text; 
    string employeename = dgi.Cells[2].Text; 
    string designation = dgi.Cells[3].Text; 

    conn.Open(); 
    SqlCommand comm = new SqlCommand("insert into [table] values (" + empid + ",'" + employeename + "','" + designation + "')", conn); 
    comm.ExecuteNonQuery(); 
    conn.Close();   
} 

Answer 1

Первая проблема:

if (rbpApprove.Checked == true) 
{ 
    conn.Open(); 
    SqlCommand cmd = new SqlCommand("Update table set IsActive= 0 where ARGID=@ARGID", conn); 

    cmd.ExecuteNonQuery(); 
    conn.Close();   
} 
else if (rbpReject.Checked == true) 
{ 
    conn.Open(); 
    SqlCommand cmd = new SqlCommand("Update table set IsActive= 1 where ARGID=@ARGID", conn); 
    cmd.ExecuteNonQuery(); 
    conn.Close(); 
} 

Этот блок ничего не делает (или, по крайней мере, не то, что нужно)

Этот бит: where ARGID=@ARGID проверяет базу данных для записей с помощью «@ARGID». Я ожидаю, что вы должны были добавить параметры, но забыл?

cmd.Parameters.AddWithValue("@ARGID", /* The value to be checking for */) 

Вторая проблема:

SqlCommand comm = new SqlCommand("insert into [T_TADA_aaprovereject_groupdirector] values (" + empid + ",'" + employeename + "','" + designation + "')", conn); 

Это должно быть параметризацию, как и другие 2 заявления, чтобы защититься от SQL Injection

Так что я хотел бы предложить вам изменить его на:

SqlCommand comm = new SqlCommand("insert into [T_TADA_aaprovereject_groupdirector] values (@empid, @employeename, @designation)", conn); 
comm.Parameters.AddWithValue(@empid, empid) 
comm.Parameters.AddWithValue(@employeename, employeename) 
comm.Parameters.AddWithValue(@designation, designation)