Signs You're a Crappy Programmer (and don't know it)

Re:Signs You're a Crappy Programmer (and don't know it) by DanielBMarkham on Sunday November 25, 2007 @06:33PM (#83415)

I have to admit it's a pet peeve for me -- even more than choice of languages, oddly enough. When scanning code, I just like the ability to see my control blocks clearly. Simply because you can do something like this

 if (a==b) { doStuff(); } else doOtherStuff();

Doesn't mean you should.

I would consider this a little bit of a hangup on my part, though, as it seems other programmers go the other way. An interesting topic for a book is all the weird things programmers get hung up about. Variable naming comes to mind, as does error handling strategies. And when we get hung up, usually we're not too shy about explaining why everybody else is wrong!

[ Parent | on HN ]

Re:Signs You're a Crappy Programmer (and don't know it) by Tichy on Monday November 26, 2007 @02:50AM (#83517)

I would not put everything on the same line, but I admit to putting the opening bracket on the same line. In fact, I consider the other variant extremely ugly. Just consider

  if(...)
  {
    ...
  }
  else
  {
  ... 
  }

  if(...) {
    ...
  } else {
    ...
  }

I can't for the life of me understand why someone would want to move the bracket to the new line ;-) It's just empty (wasted) space without any meaning. I can see the control blocks with the tighter version just fine, plus, it's the Sun Coding Convention for Java. But it's really a fruitless discussion...

[ Parent | on HN ]

Re:Signs You're a Crappy Programmer (and don't know it) by jamongkad on Wednesday November 28, 2007 @12:34AM (#84139)

I on the other hand find this quite elegant

(...) ? $doThis : $doThat;

[ Parent | on HN ]