Different ways of writing the “if” statement [closed]

Answer 1

using with the braces is suggested, I have seen some issue with the if else statement without braces ,(I don't remember exactly) i.e. Statement under if was not executed, when I added the same with braces then only worked.( Using Visual studio & C#4.0).

Answer 2

I agree with the ternary operator. Very under utilized in code that I come across, and I think it is much easier and nicer to read than all the extra brackets and indents it takes to write out an if/else statement.

Answer 3

For cases like this, there's also the conditional operator:

output = (val % 2 == 1) ? "Number is odd" : "Number is even";

If you're definitely going to use an "if" I'd use version 2 or version 4, depending on the rest of your bracing style. (At work I use 4; for personal projects I use 2.) The main thing is that there are braces even around single statements.

BTW, for testing parity it's slightly quicker to use:

if ((val & 1) == 1)

Answer 4

I prefer 4 myself, but I think 2 is definitely good too.

Answer 5

I use version 2.

One reason to use curly braces becomes more clear if you don't have an else.

if(SomeCondition)
{
  DoSomething();
}

If you then need to add another line of code, you are less likely to have an issue:

if(SomeCondition)
{ 
  DoSomething();
  DoSomethingElse();
}

Without the braces you might have done this:

if(SomeCondition)
   DoSomething();
   DoSomethingElse();

Answer 6

Version 2. I always include the brackets because if you ever need to put more than one line under the conditional you won't have to worry about putting the brackets in at a later date. That, and it makes sure that ALL of your if statements have the same structure which helps when you're scanning code for a certain if statement.