(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.
In reply to Re: A matter of style: how to perform a simple action based on a simple condition?
by GrandFather
in thread A matter of style: how to perform a simple action based on a simple condition?
by muba
| For: | Use: | ||
| & | & | ||
| < | < | ||
| > | > | ||
| [ | [ | ||
| ] | ] |