3 ms·

I'm not a big fan of single line if statements. They seem to cause bugs more often than necessary.

• corysama 12y ago
  Too each his own. Personally, I keep hearing about these bugs, but I can count maybe three times I've witnessed one first hand in two decades of coding. I have however frequently witnessed cases of five single-line ifs in a row stretched out into 30 lines of code. What could have been 5 lines of dense and vertically symmetric code was instead a sea of noise and whitespace.
  • nimblegorilla 12y ago
    Seems like an exaggerated straw man. I don't see how anyone could reasonably stretch a one-liner into 6. But yeah - "dense" is an adjective that I try to avoid when coding. Code should be clean, simple, and easy to read.
    • corysama 12y ago
      if (test) { body; } if (... So, 5 lines each. Not 6. I miscounted. if (test1) body1; if (test2) body2; if (test3) body3; if (test4) body4; if (test5) body2; Between the two formats, I greatly prefer the second. It is understandable at a glance and makes the possible error in the last line stand out much more. In practice, I usually encounter the first style -even in cases as trivial as single-function-call-per-if.