if (@ARGV == "clean"){ print "cleaning up dirs"; clean(); }
@ARGV is a list of command-line parameters. What you probably want to do is check the first of these parameters:
instead of comparing "the list".if($ARGV[0] eq "clean") { # eq, not ==, for strings print "cleaning up dirs"; clean(); }
A more advanced issue (feel free to skip this bit for now, while you get your code working) is that code like this (a huge chain of if..elsif...elsif...else) is likely to be difficult to maintain, if only because large blocks of code are daunting to mess with. Instead, I'd suggest building a dispatch table. First, write a bunch of handler functions, for instance:
Next, build a hash of these handlers, keyed on the commands:sub clean_handler { print "cleaning up dirs"; clean(); } sub make_handler { print "making project"; make(); } # ...
Now, when you get a command, retrieve the handler function from the %cmds hash and call it:my %cmds = ( 'clean' => \&clean, # reference to the subroutine 'clean' 'make' => \&make, # reference to 'make' # ... );
my $cmd = $cmds{$ARGV[0]}; # get the coderef &$cmd(); # call it
--
F
o
x
t
r
o
t
U
n
i
f
o
r
m
Found a typo in this node? /msg me
% man 3 strfry
In reply to Re: using a sub routing as an argument
by FoxtrotUniform
in thread using a sub routing as an argument
by gitarwmn
| For: | Use: | ||
| & | & | ||
| < | < | ||
| > | > | ||
| [ | [ | ||
| ] | ] |