not able to delete multiple records in gridview

Question

0.00/5 (No votes)

See more:

C#

protected void btnDel_Click(object sender, EventArgs e)
   {

       try
       {
           con.ConnectionString = ConString;
           con.Open();
           foreach (GridViewRow row in grdvw.Rows)
           {
               //CheckBox cb = (CheckBox)row.FindControl("chkDel");
               CheckBox cb = (CheckBox)grdvw.Rows[row.RowIndex].FindControl("chkDel");
               if (cb.Checked)
               {
                   //int id =Convert.ToInt32( grdvw.DataKeys[row.RowIndex]["Emp_no"].ToString);
                   int Emp_no = Convert.ToInt32(grdvw.DataKeys[row.RowIndex].Values[0].ToString());

                   cmd = new SqlCommand("Delete from employee where Emp_no=" + Emp_no, con);
                   cmd.ExecuteNonQuery();
                   con.Close();
                   BindData();
                   lblMsg.Text = "Selected Records Deleted";

               }
           }

       }

       catch (Exception ex)
       {
           Response.Write(ex.Message);
       }

   }

I am getting an error "Index was out of range. Must be non-negative and less than the size of the collection. Parameter name: index "
and only single record is deleted

Posted 22-Jul-12 20:20pm

priti2010

Updated 22-Jul-12 20:22pm

Mohamed Mitwalli

v2

Add a Solution

Comments

dimpledevani 23-Jul-12 2:23am

what does the method binddata() do??

4 solutions

Solution 1

Try this:

C#

int Emp_no = Convert.ToInt32(grdvw.DataKeys[row.RowIndex].Value);

Instead of

C#

int Emp_no = Convert.ToInt32(grdvw.DataKeys[row.RowIndex].Values[0].ToString());

Refer:
Delete rows from datgridview and database access[^]

Posted 22-Jul-12 20:27pm

Prasad_Kulkarni

Comments

priti2010 23-Jul-12 2:35am

Now i am getting this error

Unable to cast object of type 'System.Collections.Specialized.OrderedDictionary' to type 'System.IConvertible'.

Prasad_Kulkarni 23-Jul-12 2:39am

Try this:

int Emp_no = int.Parse(grdvw.CurrentCell.RowIndex.ToString());
string Emp_noString = grdvw[0, currentRow].Value.ToString();
Emp_noInt = int.Parse(Emp_noString );

Solution 4

C#

con.ConnectionString = ConString;
           con.Open();

You have open the connnection before foreach loop and you are closing the connection after deleting one record.

Open the connection in same foreach loop

Posted 22-Jul-12 21:07pm

Raghunatha_Reddy_S

Solution 2

Check that you have provided the field in DataKeyNames property of GridView

Posted 22-Jul-12 21:02pm

Raghunatha_Reddy_S

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)

_Amy · Accepted Answer · 2012-07-22T21:06:00

Hi,
Try this:

C#

con.ConnectionString = ConString;
con.Open();
foreach (GridViewRow row in grdvw.Rows)
{
   //CheckBox cb = (CheckBox)row.FindControl("chkDel");
   CheckBox cb = (CheckBox)grdvw.Rows[row.RowIndex].FindControl("chkDel");
   if (cb.Checked)
   {
        int Emp_no = int.Parse(GridView1.DataKeys[row.RowIndex].Value.ToString());
        cmd = new SqlCommand("Delete from employee where Emp_no='" + Emp_no + "'", con);
        cmd.ExecuteNonQuery();
        //con.Close(); this should not come here.
        BindData();
        lblMsg.Text = "Selected Records Deleted";
    }
}
con.Close();

--Amit