bool function for data gird view

Question

1.00/5 (1 vote)

See more:

Hello guys,

I've problem with bool function always giving me errors, but syntactic and logic are correct, here's the function and the call.

C#

bool exist=false;
exist = checking(stock_code);

public bool checking(int x)
        {
            string i;
            i = Convert.ToString(x);
            for (int j = 0; j < dataGridView1.Rows.Count; j++)
            {
                if (dataGridView1.Rows[j].Cells[0].Value == i.ToString ())
                {
                    exist = true;
                    return true;
                }
                else
                {
                    exist = false;
                    return false;
                }
                //else
                    //return false;
             }
        }

Posted 9-Sep-12 14:09pm

Member 8584763

Add a Solution

Comments

[no name] 9-Sep-12 20:16pm

And we are supposed to guess what "errors" you are getting? Why are you converting x to a string (i) and then convert i to a string again? It's already a string.

Member 8584763 9-Sep-12 20:28pm

not all code paths return a value

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)

Wes Aday · Answer 1 · 2012-09-09T14:34:00

Solution 1

It means exactly what is says. Not all of your code paths return a value.

C#

public bool checking(int x)
        {
            for (int j = 0; j < dataGridView1.Rows.Count; j++)
            {
                if (dataGridView1.Rows[j].Cells[0].Value.ToString == x.ToString())
                {
                    exist = true;
                    return true;
                }
                else
                {
                    exist = false;
                    return false;
                }
             }
             return false;
        }

Posted 9-Sep-12 14:34pm

Wes Aday

Comments

Member 8584763 9-Sep-12 20:58pm

this function always gives me false, even if the value in the data grid view

[no name] 9-Sep-12 21:05pm

Your code presupposes that x is only in the first column of your grid.

Member 8584763 9-Sep-12 21:07pm

that's right, the call is inside try and catch. it's given me error like this
Object reference not set to an instance of an object. in catch

[no name] 9-Sep-12 21:10pm

And which object is null?

Member 8584763 9-Sep-12 21:17pm

system.nullreferenceexpception:object reference not set to an instance of an object
it's say in these line: in the call and in the condition of the function
exist = checking(stock_code);
if (dataGridView1.Rows[j].Cells[0].Value.ToString () == x.ToString ())

[no name] 9-Sep-12 21:20pm

Okay and when you ran your code in the debugger, which object is null?

Member 8584763 9-Sep-12 21:22pm

I think object sender.

Member 8584763 9-Sep-12 21:18pm

If I put the hole code would it help?

[no name] 9-Sep-12 21:22pm

Probably not. I cannot debug your code. Not without your entire project. Null references are almost the easiest to find if you just run your code in the debugger.

Member 8584763 9-Sep-12 21:25pm

I tried the debug mode, it gets caught in catch.

[no name] 9-Sep-12 21:33pm

Best guess, stock_code is null wherever that comes from.

Member 8584763 9-Sep-12 21:36pm

I'm sorry for taking too long with you, but stock_code isn't null I checked.

Sergey Alexandrovich Kryukov 11-Sep-12 15:43pm

Ha-ha. A 5. :-)

By the way, take a look at the last OP's answer:
http://www.codeproject.com/Questions/457301/central-database-and-cilents
This is why I looked at the present page: wanted to know what else could as a person like that. :-)

--SA

[no name] 11-Sep-12 15:47pm

Thanks. I had forgotten about this one. I have way more fun in comments.

Sergey Alexandrovich Kryukov 11-Sep-12 17:37pm

:-)