If Checking order in C#

Question

0.00/5 (No votes)

See more:

Hi friends,

I want to know if I can use "if statement" like this

C#

 if ( listView1.SelectedItems.Count >0 && MessageBox.Show("Remove this Contact?", Application.ProductName, MessageBoxButtons.YesNo) == System.Windows.Forms.DialogResult.Yes)
{
//Some Code here...
}

Instead of

C#

 if ( listView1.SelectedItems.Count >0)
      if(MessageBox.Show("Remove this Contact?", Application.ProductName, MessageBoxButtons.YesNo) == System.Windows.Forms.DialogResult.Yes)
{
//Some Code here...
}

I want it to get the Message Box confirmation only if the selected items count is greater than 0....
Both working fine now..... But I am afraid if any problems will occur in future.... :( :-/ :'(
Please Helppp.... !!!!

Posted 2-Jul-13 22:07pm

Yesudasan Moses

Add a Solution

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

CPallini · Accepted Answer · 2013-07-02T22:23:00

Solution 3

if conditional expression is evaluated (and short-circuited, see "&& Operator (C# Reference)"[^] ) left to right. So first and second forms are equivalent and you don't have to worry about future. :-)

Posted 2-Jul-13 22:23pm

CPallini

Updated 2-Jul-13 22:29pm

v2

Comments

Yesudasan Moses 3-Jul-13 4:26am

Thanks dude :)

CPallini 3-Jul-13 4:33am

You are welcome.

ArunRajendra · Accepted Answer · 2013-07-02T22:18:00

Solution 2

Yes both are perferctly fine. But considering the coding standards 1st option is more readable.

Posted 2-Jul-13 22:18pm

ArunRajendra

Comments

Yesudasan Moses 3-Jul-13 4:25am

Thanks my friend :)

ArunRajendra 3-Jul-13 4:56am

Welcome :)

Johnny J. · Accepted Answer · 2013-07-02T22:17:00

Solution 1

As far as I know, the fist one is completely valid as well. It will check the first statement and only check the next statement if the first one is true...

[EDIT] I found the proof on MSDN[^] [/EDIT]

Posted 2-Jul-13 22:17pm

Johnny J.

Updated 2-Jul-13 22:22pm

v4

Comments

Yesudasan Moses 3-Jul-13 4:26am

Thanks for your time dude :)

Johnny J. 3-Jul-13 4:30am

NP ;-)