in reply to A matter of style: how to perform a simple action based on a simple condition?

(almost) Never One - it requires too much brain work to figure out what's going on when scanning through code.

Never three - all that is seen is the condition; the action gets lost.

Most often Two. The action is obvious and generally the most important thing. If the condition is really important I'd:

if ($var) { sub (); }

to make the condition obvious and the action easy to find.

As a general rule stuff at the end of a long or complicated line will tend to be ignored. If it's important it be seen don't stick it on the end of a line.

True laziness is hard work
  • Comment on Re: A matter of style: how to perform a simple action based on a simple condition?
  • Download Code