Re^2: String equality and undef

On the other hand, dealing with a bogus value is usually shorter than what you do with a real value, and I think a good style guideline is to keep "else" as close as possible to the condition it negates.

if ( ! defined $id ) {
    die "Why is this ID not defined?"
}
else {
    # Twenty
    # or
    # more
    # lines
}
[download]

I like this guideline because I've too many times found myself looking at a screen of code like this:

        # blah
        # blah
    }
    else {
        return undef;
    }
}
else {
   unlink $tmpfile;
   return undef;
}
[download]

What were the conditions?

Even better in this case would have been an early return so as to avoid the extra level of indent anyway, but that's beside the point.

Comment on Re^2: String equality and undef Select or Download Code

Replies are listed 'Best First'.
Re^3: String equality and undef by webfiend (Vicar) on Dec 09, 2008 at 16:36 UTC
Fair point, but you don't really need to worry about the `else` when your reaction to the `if` condition is death. `unless ( defined $id ) { die "Why is this ID not defined?"; } # Life goes on normally for everyone else.` [download] And I disagree with your closing, an early return in cases like that is exactly the point. You want to take any steps you can to make the code more readable, especially if it means completely ignoring my initial "switch the logic" comment.	[reply] [d/l]

Replies are listed 'Best First'.

Re^3: String equality and undef
by webfiend (Vicar) on Dec 09, 2008 at 16:36 UTC

Fair point, but you don't really need to worry about the else when your reaction to the if condition is death.

unless ( defined $id ) {
    die "Why is this ID not defined?";
}

# Life goes on normally for everyone else.
[download]

And I disagree with your closing, an early return in cases like that is exactly the point. You want to take any steps you can to make the code more readable, especially if it means completely ignoring my initial "switch the logic" comment.

[reply]
[d/l]