Comments on: If, else and nothing else http://www.turnleafdesign.com/if-else-and-nothing-else Ramblings of a junior developer Thu, 20 Jan 2011 09:21:13 +0000 hourly 1 http://wordpress.org/?v=3.0.1 By: Eric T. http://www.turnleafdesign.com/if-else-and-nothing-else/comment-page-1#comment-64 Eric T. Tue, 06 Oct 2009 13:53:04 +0000 http://www.turnleafdesign.com/?p=81#comment-64 Be minimalist when it will lead to better code design. For you new guys out there try not to make the code "too cute" with look what I learned today type of stuff. Having said that, I think that it is great that you are hungry for learning new things. Your code and career will certainly be benefit from it. Be minimalist when it will lead to better code design. For you new guys out there try not to make the code “too cute” with look what I learned today type of stuff.

Having said that, I think that it is great that you are hungry for learning new things. Your code and career will certainly be benefit from it.

]]> By: Andreas http://www.turnleafdesign.com/if-else-and-nothing-else/comment-page-1#comment-15 Andreas Fri, 02 Oct 2009 09:09:57 +0000 http://www.turnleafdesign.com/?p=81#comment-15 It's not alwas the best thing to have a short line of code. Software is always changing and other people will have to change your code. if (a > b) { is more readable, because it's a lot closer to our natural language. If a greater then b do that. For getBoolValue = a > b; you have to think about if it's a condition or not. It's an implicit condition. And for the sake of readability, I prefer using the explicit expression. And I have another point about shortening. You can shorten the if expression like this. int boolValue = true; if (a < b) { boolValue = false; } It’s not alwas the best thing to have a short line of code. Software is always changing and other people will have to change your code.

if (a > b) {

is more readable, because it’s a lot closer to our natural language.

If a greater then b do that.

For

getBoolValue = a > b;

you have to think about if it’s a condition or not.
It’s an implicit condition.

And for the sake of readability, I prefer using the explicit expression.

And I have another point about shortening.
You can shorten the if expression like this.

int boolValue = true;
if (a < b) {
boolValue = false;
}

]]>