ASP.NET Delete Multiple GridView rows via Checkbox template

Question

I have a GirdView table in my application. I have a Delete link button and a checkbox in my first column. Below are snapshots:

enter image description here

I can manually delete each row by clicking Delete link button on each row.

However, I would want to enable the checkbox to delete multiple rows.

Below is my code for my Delete link button:

LinkButton lnk = (LinkButton)sender; 
        string stid = lnk.CommandArgument.ToString();
        SqlConnection conn = new SqlConnection("Data Source=DATASOURCE");
        string sql = string.Format("DELETE FROM [UserDB] where Employee like '%{0}%'",stid);
        SqlCommand cmd = new SqlCommand(sql,conn);
        conn.Open();
        cmd.ExecuteNonQuery();

Below is my code for the current Delete Checked button:

bool atLeastOneRowDeleted = false;
        foreach (GridViewRow row in GridView1.Rows)
        {
            // Access the CheckBox
            CheckBox cb = (CheckBox)row.FindControl("UserSelection");
            if (cb != null && cb.Checked)
            {
                atLeastOneRowDeleted = true;
                // How do I get the value from my Employee column?
                string userID = ???;


           SqlConnection conn = new SqlConnection("Data Source=DATASOURCE");
           string sql = string.Format("DELETE FROM [UserDB] where Employee like '%{0}%'",userID);
           SqlCommand cmd = new SqlCommand(sql,conn);
           conn.Open();
           cmd.ExecuteNonQuery();
            }
        }

I would like to know how should I grep userID, which is Employee key in the GridView, to INSERT it into my SQL delete statement?

bumble_bee_tuna · Accepted Answer · 2011-11-18 03:56:16Z

1

I think you want

bool atLeastOneRowDeleted = false;
    foreach (GridViewRow row in GridView1.Rows)
    {
        // Access the CheckBox
        CheckBox cb = (CheckBox)row.FindControl("UserSelection");
        if (cb != null && cb.Checked)
        {
            atLeastOneRowDeleted = true;
            //assuming you have the ID column after the controls
            string CusId = (row.Cells[3].Text);


       SqlConnection conn = new SqlConnection("Data Source=DATASOURCE");
       string sql = string.Format("DELETE FROM [UserDB] where Employee like '%{0}%'",userID);
       SqlCommand cmd = new SqlCommand(sql,conn);
       conn.Open();
       cmd.ExecuteNonQuery();
        }
    }

answered Nov 18, 2011 at 3:56

bumble_bee_tuna

3,5618 gold badges48 silver badges87 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

bumble_bee_tuna Over a year ago

CBRRacer racer has a point that you'd be better of getting the keys and then doing one delete. Otherwise you'll be blowing out a lot of uneeded sql. And it will never scale.

CBRRacer · Accepted Answer · 2011-11-18 03:58:39Z

1

well without seeing your code for the checkboxes, I would say set the value of the checkboxes to the UserID of the employee. Then on submit set the checked values to a list or array and then parse that list or array with your delete single method.

answered Nov 18, 2011 at 3:58

CBRRacer

4,6791 gold badge26 silver badges27 bronze badges

Comments

Zo Has · Accepted Answer · 2011-11-18 11:04:11Z

0

Using 'row.Cells[3].Text' to fetch column values is a bad way. If he adds another column in between then the index would change. Use dataKeys instead to reference your column values.

answered Nov 18, 2011 at 11:04

Zo Has

13.1k24 gold badges92 silver badges150 bronze badges

Comments

Sangram Nandkhile · Accepted Answer · 2015-08-20 06:00:30Z

0

This question has already been answered but here's an article which shows which entire flow using datbase, Gridview, Business Logic and DAL.

Final Preview looks something like this

You can find the entire source code here

answered Aug 20, 2015 at 6:00

Sangram Nandkhile

18.2k19 gold badges85 silver badges116 bronze badges

Collectives™ on Stack Overflow

ASP.NET Delete Multiple GridView rows via Checkbox template

4 Answers 4

1 Comment

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

1 Comment

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related