Beefy Boxes and Bandwidth Generously Provided by pair Networks
more useful options
 
PerlMonks  

Re^2: if,if and again,if

by Tanktalus (Canon)
on Aug 23, 2012 at 17:57 UTC ( #989363=note: print w/ replies, xml ) Need Help??


in reply to Re: if,if and again,if
in thread if,if and again,if

Data-driven approaches FTW!

Though I'd be tempted to do things a bit more perlishly (at least for my own definition of perlish):

my @matchers = ( [ qr/(Lz0|PLATO)/i => 'Apps' ], ... ); for my $check (@matchers) { if ($release_name =~ $check->[0]) { $category = $check->[1]; print "[INFO] Category: $category\n"; last; } }
Actually, a bit more perlish, IMO, would be to replace that for loop altogether:
use List::Util qw(first); my $category = map { $_ ? $_->[1] : undef; } first { $release_name =~ $_->[0]; } @matchers; if ($category) { print "[INFO] Category: $category\n"; } else { print "[ERROR] Unrecognised release type: $release_name\n"; }
But maybe that's just me. :-)


Comment on Re^2: if,if and again,if
Select or Download Code
Re^3: if,if and again,if
by moritz (Cardinal) on Aug 23, 2012 at 18:04 UTC

    If we go all perlish, I'd get rid of the ugly map with a ternary :-). Either

    my $category = first { ... } ... ; if ($category) { $category = $category->[1]; print "[INFO] Category: $category\n"; }

    Or if you insist on a map on a one-element list, write it as

    map $_->[1], grep $_, first { .... } ...;

    IMHO several simple operations are easier to read than a few complicated ones.

Log In?
Username:
Password:

What's my password?
Create A New User
Node Status?
node history
Node Type: note [id://989363]
help
Chatterbox?
and the web crawler heard nothing...

How do I use this? | Other CB clients
Other Users?
Others meditating upon the Monastery: (6)
As of 2015-07-04 06:31 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?

    The top three priorities of my open tasks are (in descending order of likelihood to be worked on) ...









    Results (58 votes), past polls