Gridview with checkbox,not working... please help

Question

3.33/5 (3 votes)

See more:

I am working on checkbox in GRIDVIEW. I want to delete all the selected records.. I have this code below for your reference... Please help me to make it work ... Thanks....

protected void btnDelete_Click(object sender, EventArgs e)
{
  string currentrow;
  for (int index = 0; index <= GridView1.Rows.Count - 1; index++)
  {
  CheckBox cb =  (CheckBox)GridView1.Rows[index].FindControl("chkDelete");
  if (cb.Checked)
  {
  currentrow = GridView1.DataKeys[index].Value.ToString();
  string constr = @"Data Source=OM;Initial Catalog=TestDB;Integrated    Security=True";
  string query = @"Delete FROM Categories where CategoryID='" + currentrow + "'";
  SqlConnection con = new SqlConnection(constr);
  SqlCommand cmd = new SqlCommand(query);
  SqlDataAdapter da = new SqlDataAdapter(query, constr);
  DataSet ds = new DataSet();
  da.Fill(ds, "Categories");

  GridView1.DataSource = ds.Tables["Categories"];
  GridView1.DataBind();

  }
 
  }
  }

===================================================================
For this code i am also getting an index out of range error... I am a beginner please help me find solution to this problem....Thanks

[Edited]Code is blocked in "pre" tags[/Edited]

Posted 30-Apr-11 19:50pm

su_ma2010

Updated 30-Apr-11 19:55pm

RaviRanjanKr

v2

Add a Solution

5 solutions

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

Sandeep Mewara · Answer 1 · 2011-04-30T20:12:00

What you are doing is starting a for loop for all the rows that were present at the beginning.

for (int index = 0; index <= GridView1.Rows.Count - 1; index++)

And then, if you find it selected, you remove it immediately(that can be done) and then rebind the grid (this is wrong). Thus, the rows are no more selected and the count is also reduced and thus you are bound to get an error here:

CheckBox cb =  (CheckBox)GridView1.Rows[index].FindControl("chkDelete");

All you need is to remove the bind code out of For loop. Try:

protected void btnDelete_Click(object sender, EventArgs e)
{
  string currentrow;
  for (int index = 0; index <= GridView1.Rows.Count - 1; index++)
  {
       CheckBox cb =  (CheckBox)GridView1.Rows[index].FindControl("chkDelete");
       if (cb.Checked)
       {
          // Just delete the selected row and let the loop repeat. 
       }
  }
}

BTW, use Visual Studio DEBUUGER, it's a great tool. Had you used that you would have find it yourself. It was a logical mistake.

Wonde Tadesse · Answer 2 · 2011-05-01T04:33:00

I will start the answer with a question.

1. Have you bind the CountryID in the GridView?
2. To what type of control it is binded ? Is it a TextBox, a Label, a DropDownList etc. If so then

C++

protected void clickMe(object sender, EventArgs e)
    {
        DataTable dt = new DataTable();
        for (int index = 0; index <= GridView1.Rows.Count - 1; index++)
        {
            CheckBox cb = (CheckBox)GridView1.Rows[index].FindControl("chkDelete");
            if (cb.Checked)
            {
                Label lblCountryID = (Label)GridView1.Rows[index].FindControl("lblCountryID");// Assuming that CountryID column is binded to Label control with a name lblCountryID.
                if (!string.IsNullOrWhiteSpace(lblCountryID.Text))
                {
                    string constr = "Data Source=OM;Initial Catalog=TestDB;Integrated Security=True";
                    string nonQuery = "Delete FROM Categories where CategoryID='" + lblCountryID.Text + "'";
                    string query = "Select * Categories";
                    SqlConnection con = new SqlConnection(constr);
                    SqlCommand cmd = new SqlCommand(nonQuery, con);
                    cmd.ExecuteNonQuery();
                    SqlDataAdapter da = new SqlDataAdapter(query, con);
                    da.Fill(dt);
                }
            }
        }

        GridView1.DataSource = dt;
        GridView1.DataBind();
    }

I hope this will fix your problem.

Shahriar Iqbal Chowdhury/Galib · Answer 3 · 2011-04-30T20:28:00

Hi,

To iterate GridView row items you can use this code

foreach (GridViewRow grv in Gridview1.Rows)
{
   CheckBox chkDelete = grv.FindControl("chkDelete") as CheckBox;
   if (chkDelete.Checked)
   {
     string constr = @"Data Source=OM;Initial Catalog=TestDB;Integrated    Security=True";
     string query = @"Delete FROM Categories where CategoryID='" + Gridview1.DataKeys[grv.DataItemIndex].Value.ToString() + "'";
     SqlConnection con = new SqlConnection(constr);
     SqlCommand cmd = new SqlCommand(query);
     cmd.Connection = con;
     cmd.ExecuteNonQuery();
   }
}

Hope this will help.

Ashishmau · Answer 4 · 2011-05-01T19:33:00

Solution 4

see this link .it helps u to understand basics

http://www.codedigest.com/Articles/ASPNET/132_GridView_with_CheckBox_%E2%80%93_Select_All_and_Highlight_Selected_Row.aspx[^]

Posted 1-May-11 19:33pm

Ashishmau

nit_singh · Answer 5 · 2011-05-01T20:49:00

Try this

protected void btnDelete_Click(object sender, EventArgs e)
        {
            string currentrow;
            string constr = @"Data Source=OM;Initial Catalog=TestDB;Integrated    Security=True";
            string query = string.Empty;
            SqlConnection con = new SqlConnection(constr);
            SqlCommand cmd = null;
            SqlDataAdapter da = null;
            for (int index = 0; index <= GridView1.Rows.Count - 1; index++)
            {
                CheckBox cb = (CheckBox)GridView1.Rows[index].FindControl("chkDelete");
                if (cb.Checked)
                {
                    currentrow = GridView1.DataKeys[index].Value.ToString();
                    query = @"Delete FROM Categories where CategoryID='" + currentrow + "'";
                    
                    cmd = new SqlCommand(query);
                    da = new SqlDataAdapter(query, constr);
                    cmd.ExecuteNonQuery();
                }
            }
            query = @"Select * FROM Categories";
            cmd = new SqlCommand(query);
            da = new SqlDataAdapter(query, constr);
            DataSet ds = new DataSet();
            da.Fill(ds, "Categories");
            GridView1.DataSource = ds.Tables["Categories"];
            GridView1.DataBind();
        }

Gridview with checkbox,not working... please help

5 solutions

Solution 1

Solution 3

Solution 2

Solution 4

Solution 5

Add your solution here

Preview 0