![[personal profile]](https://www.dreamwidth.org/img/silk/identity/user.png)
When I was doing some code cleanup a few months ago, I saw some lines that looked like this:
I decided that looked redundant and changed it to:
Take my advice: don't combine "my" statements and postfix conditionals. Ever.
I've updated the programming guidelines to warn people away from this as well.
( Brief explanation of why this is a bad thing )
my $foo;
$foo = $bar unless $baz;
I decided that looked redundant and changed it to:
my $foo = $bar unless $baz;
Take my advice: don't combine "my" statements and postfix conditionals. Ever.
I've updated the programming guidelines to warn people away from this as well.
( Brief explanation of why this is a bad thing )