coding rules, NO nested if...else?

Question

Our company uses checkstyle to ensure the java source code having no nested if..else. i am trying to get used to it. but i have this doubt, is this a efficient way to improve code quality?

we have many rules,

no nested if else
Method should be no longer than 50 lines.
anonymous inner class no longer than 15 lines.
there must be a white space before {, which means we have to use Ctrl+Shift+F to format code.

What's an embedded if...else? Things like if(...) { if(...) {...} }? — In silico
– In silico, Commented Feb 25, 2012 at 2:26
Indentation going over to the right hand side of the screen typically indicates a problem. This particular coding style seems a bit, er, eccentric. But if you've got this rule, there are probably far more serious things you might want to clear up first. — Tom Hawtin - tackline
– Tom Hawtin - tackline, Commented Feb 25, 2012 at 2:46
Yes, it is nested if else, we can't use if(...) { if(...) {...} } — Sam K
– Sam K, Commented Feb 25, 2012 at 2:52

Dylan Bijnagte · Accepted Answer · 2012-02-25 02:38:38Z

2

There are arguments for favoring polymorphism over use of switch or if else blocks. Here is a link to a google tech talk discussing the issue

answered Feb 25, 2012 at 2:38

Dylan Bijnagte

1,3569 silver badges17 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Matt T · Accepted Answer · 2012-02-25 02:28:08Z

1

Switch / select case statements are generally more efficient, but that seems like a lot of effort over a fairly small issue.

answered Feb 25, 2012 at 2:28

Matt T

5112 silver badges8 bronze badges

1 Comment

curioustechizen Over a year ago

Not to mention that switch only works on values that evaluate to compile time constants.

Collectives™ on Stack Overflow

coding rules, NO nested if...else?

2 Answers 2

Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related