How to make string which contains datagridviews value

Question

0.00/5 (No votes)

See more:

Here Is my Code to check available data in table if there exist any one value of Gridviews Column it will return the available value.
But there is somthing wrong so why it check only one value which is first in gridview.
The Code is as below
ok Here is the Full Code...

C#

private void SureToDeleteBill() // Cheking For There Is No Imei in Bill Which Is Sold
        {
            
                try
                {
                    for (int i = 0; i <= dataGridView2.Rows.Count-1; i++)
                    {
                        // check either imei is sold or available in Imei Sold table
                        cs.Open();
                        cmd = new OleDbCommand("select * from tblsIMEI where IMEI = @a0", cs);
                        cmd.Parameters.AddWithValue("@a0", dataGridView2.Rows[i].Cells[1].Value.ToString());
                        dr = cmd.ExecuteReader();
                        if (dr.Read())
                        {
                            string msg;
                            msg = dr["IMEI"].ToString();
                            MessageBox.Show("Sorry !!\nYou Can Not DeleteBill !\n Sold IMEI :  ", "Sold IMEI", MessageBoxButtons.OK, MessageBoxIcon.Asterisk);
                            return;
                        }
                        else
                        {
                            DialogResult r = MessageBox.Show("Sure To Proceed?", "Delete Data", MessageBoxButtons.YesNo, MessageBoxIcon.Question);
                            if (r == DialogResult.Yes)
                            {
                                DeleteFromtblPurchase();
                                DeleteFromtblIMEI();
                                DeleteFromPurchaseModel();
                                DeleteFromDealerBill();
                                MessageBox.Show("Record Delete Success!!", "Delete");
                                this.Close();
                            }
                            else
                            {
                                return;
                            }
                        }
                        cs.Close();
                    }
                    }
                
                catch (Exception ex)
                {
                    MessageBox.Show(ex.Message, "Sure To Delete");
                }
                finally
                {
                    cs.Close();
                }
        }

Please help to solve it.

Posted 12-Jan-15 2:36am

Member Albert

Updated 12-Jan-15 4:39am

Herman<T>.Instance

v4

Add a Solution

Comments

ZurdoDev 12-Jan-15 8:44am

What's the problem? You have it in a loop so it should process each row in dataGridView2.

Member Albert 12-Jan-15 8:55am

No it isnt. It just take first rows value only.

ZurdoDev 12-Jan-15 9:05am

All you have to do is step through the code and see what is happening. Should be pretty easy to figure out.

CHill60 12-Jan-15 8:44am

First thing I notice when I tried to put the pre-tags in to format your code is that the braces {} don't seem to match up.
Secondly you're jumping out of the function with those return statements. Use the debugger to step through the code and you should be able to see your problem

2 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)

OriginalGriff · Answer 1 · 2015-01-12T02:54:00

Well...it isn't going to compile. Or work if it did.

Let's have a look at your control structures:

C#

try
{
    for (int i = 0; i <= dataGridView2.Rows.Count-1; i++)
    {
        if (dr.Read())
        {
        }
        else
        {
        }
        else
        {
        }
    }
}
}

catch (Exception ex)
{
}
finally
{
}
}

So, starting in the middle, you have this:

C#

if (a) A;
else B;
else C;

What is that? you can't have two else clause for the same if! That's wrong, and I'm not sure what you are trying to do there...

Ignore that for a moment, (or assume you fixed it) and the catch doesn't match with the try - you have a spurious "}" in there.

Now let's look at what happens if (by a miracle) it did compile and work.
You open the connection in the loop. Why? You also close it in the loop. And in the finally block...

And whichever way you go in the if the result is the same: return or close the form completely!

Frankly, this looks like you randomly grabbed lines of code from the internet and through them together, hoping it would magically work. And when it didn't, stuffed it up here in the hope we would fix it for you.
Development doesn't work like that: it's one of the few subjects you can study at school where you have to think about what you are trying to do. In that, it's a great introduction to the real world - but if you don't start to think here, you don;t end up flipping burgers by the end of term. You do in the real world...:laugh:

Surendra Adhikari SA · Answer 2 · 2015-01-12T19:08:00

Solution 2

You should use

C#

continue;

instead of

C#

return;

:-)

Posted 12-Jan-15 19:08pm

Surendra Adhikari SA