Delete checked item in datagridview ?

Question

1.33/5 (3 votes)

See more:

I want to delete item that has been checked in datagridview. but document also print double.

What I have tried:

C#

for(int xyz=0;xyz<datagridview1.rows.count;xyz++)>
            { 
                DataGridViewCheckBoxCell chk = (DataGridViewCheckBoxCell)dataGridView1.Rows[xyz].Cells["Calckeck"];
                if (Convert.ToBoolean(chk.Value) == true)
                { 
                    DataTable dt = blord.getallfromorder_id(Convert.ToInt32(dataGridView1.Rows[xyz].Cells["cal_order_id"].Value));
                    if (dt.Rows.Count > 0)
                    {
                        foreach (DataGridViewRow dr in dataGridView1.Rows)
                        {
                            decimal total_to_be_remove = Convert.ToDecimal(dr.Cells["cal_total"].Value);
                            txtgrandtotal.Text = (Convert.ToDecimal(txtgrandtotal.Text) - Convert.ToDecimal(dataGridView1.Rows[xyz].Cells["cal_total"].Value)).ToString();
                            txtsubtotal.Text = (Convert.ToDecimal(txtgrandtotal.Text) - Convert.ToDecimal(dataGridView1.Rows[xyz].Cells["cal_total"].Value)).ToString();
                            lblsub_total.Text = txtsubtotal.Text;
                            printcalcel();
                            int i = blord.deleteitemfromorder(Convert.ToInt32(dataGridView1.Rows[xyz].Cells["cal_order_id"].Value));
                            dataGridView1.Rows.RemoveAt(xyz);
                        }                       
                    }

Posted 8-May-16 0:40am

Member 12242717

Updated 8-May-16 3:03am

Michael_Davies

v2

Add a Solution

Comments

Karthik_Mahalingam 8-May-16 6:56am

what is the issue ?

Member 12242717 8-May-16 7:38am

it shows 2 times document print.

Sergey Alexandrovich Kryukov 8-May-16 9:58am

What would you expect? You call printcalcel() inside loop and don't even show what is that.
Use the debugger, after all.
—SA

1 solution

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)

RickZeeland · Accepted Answer · 2016-05-08T03:03:00

Using .RemoveAt() inside a foreach loop is asking for trouble, there are other ways.
DataTable can filter the data, you could use DataTable.Select() to filter your data table, here is an example:
DataTable.Select Method (String) (System.Data)[^]

C#

var expression = "Date > #1/1/00#";
DataRow[] foundRows;

// Use the Select method to find all rows matching the filter.
foundRows = table.Select(expression);

// Print column 0 of each returned row.
for(int i = 0; i < foundRows.Length; i ++)
{
    Console.WriteLine(foundRows[i][0]);
}