The
Switch module is probably what could help here, though you are basically emulating a giant if-elsif-else statement with that.
Alternatively, assuming that I understand the problem right, you can create an array of hashes as shown below:
my @actions = (
{
match => sub { $_[0] =~ /^(\w*)\.(\d*)$/; },
type => "inventory",
action => &add_inventory;
},
{
match => sub { $_[0] =~ /^maimed(\w*)$/; },
type => "maimed",
action => &handle_maimed;
},
{
match => sub { $_[0] },
type => "default",
action => sub { print "I cannot handle ", $_[0]; }
}
);
# much later
foreach my $file ( @filelist ) {
foreach my $action ( @actions ) {
if ( my @parts = &{ $action->{ match } }( $file ) ) {
print "Am doing $action->{ type } on $file\n";
&{ $action->{ action } }( @parts );
last;
}
}
}
This avoids large if or switch blocks, and allows you to wrap code a bit better with a better explaination in the code to what is going on with each possible file type.
-----------------------------------------------------
Dr. Michael K. Neylon - mneylon-pm@masemware.com
||
"You've left the lens cap of your mind on again, Pinky" - The Brain
"I can see my house from here!"
It's not what you know, but knowing how to find it if you don't know that's important
-
Are you posting in the right place? Check out Where do I post X? to know for sure.
-
Posts may use any of the Perl Monks Approved HTML tags. Currently these include the following:
<code> <a> <b> <big>
<blockquote> <br /> <dd>
<dl> <dt> <em> <font>
<h1> <h2> <h3> <h4>
<h5> <h6> <hr /> <i>
<li> <nbsp> <ol> <p>
<small> <strike> <strong>
<sub> <sup> <table>
<td> <th> <tr> <tt>
<u> <ul>
-
Snippets of code should be wrapped in
<code> tags not
<pre> tags. In fact, <pre>
tags should generally be avoided. If they must
be used, extreme care should be
taken to ensure that their contents do not
have long lines (<70 chars), in order to prevent
horizontal scrolling (and possible janitor
intervention).
-
Want more info? How to link
or How to display code and escape characters
are good places to start.