Delete multiple asp.net repeater Items with checkbox selection only

Question

here it does not show any error only page loading what is the error?

C# code

protected void imgs_Click(object sender, ImageClickEventArgs e)  
        {        
            foreach (RepeaterItem ri in repeater.Items)
            {

            CheckBox item_check = (CheckBox)ri.FindControl("item_check");
            Label txt_id = (Label)ri.FindControl("txt_id");

                if (item_check.Checked)
                {
                    con = new SqlConnection(strcon);
                    SqlCommand cmd = new SqlCommand("ram", con);
                    cmd.CommandType = CommandType.StoredProcedure;
                    con.Open();
                    cmd.Parameters.AddWithValue("@Action", "DELETE");
                    cmd.Parameters.AddWithValue("@eid", txt_id);
                    repeat();
               }                
        }     
}

asp.code

put it in a try catch block and print out the error in the catch block to see if really no errors are made — Vajura
– Vajura, Commented Dec 2, 2014 at 7:42

Dgan · Accepted Answer · 2014-12-02 07:49:33Z

1

You Forgot To Write cmd.ExecuteNonQuery();

Stop Using AddWithValue

Always use using for SqlConnection and SqlCommand which implements IDisposable

protected void imgs_Click(object sender, ImageClickEventArgs e)
{

    foreach (RepeaterItem ri in repeater.Items)
    {

    CheckBox item_check = (CheckBox)ri.FindControl("item_check");
    Label txt_id = (Label)ri.FindControl("txt_id");

        if (item_check.Checked)
        {
      Using(SqlConnection con = new SqlConnection(strcon))
            {
      Using(SqlCommand cmd = new SqlCommand("ram", con))
             {
            cmd.CommandType = CommandType.StoredProcedure;
            con.Open();
            cmd.Parameters.Add("@Action",SqlDbType.Varchar,50).Value="DELETE";
            cmd.Parameters.Add("@eid",SqlDbType.Int).Value=Convert.ToInt16(txt_id.Text);
            cmd.ExecuteNonQuery();
           repeat();
            }
           }
       }                
}

}

edited Dec 2, 2014 at 7:49

answered Dec 2, 2014 at 7:44

Dgan

10.3k1 gold badge33 silver badges51 bronze badges

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

1 Comment

Prakash J Over a year ago

i m getting this error Collection was modified; enumeration operation may not execute.

Prakash J · Accepted Answer · 2014-12-02 10:05:28Z

    foreach (RepeaterItem ri in repeater.Items)
    {

        CheckBox item_check = (CheckBox)ri.FindControl("item_check");
        Label txt_id = (Label)ri.FindControl("txt_id");

        if (item_check.Checked)
        {

            con = new SqlConnection(strcon);
            cmd = new SqlCommand("ram", con);
            cmd.CommandType = CommandType.StoredProcedure;
            cmd.Parameters.AddWithValue("@Action", "DELETE");
            cmd.Parameters.AddWithValue("@eid", txt_id.Text);
            try
            {
                con.Open();
                cmd.ExecuteNonQuery();

            }
            catch (Exception ex)
            {
                ex.Message.ToString();
            }
            finally
            {

                con.Close();
            }
        }
    }
    repeat();

Collectives™ on Stack Overflow

Delete multiple asp.net repeater Items with checkbox selection only

2 Answers 2

1 Comment

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Related