| [reply] [Watch: Dir/Any] |
Its tough write an elaborate answer to such a minimal question. We don't even know what that code is supposed to do. Try framing the code with how you are using it, what the interesting parts are, what problems you're having with it, etc.
Time spent asking a good question will be rewarded with meaningful answers. Read dominus's comments on how to ask a good question. After all, if you aren't willing to invest more than five words, why should we?
-Blake
| [reply] [Watch: Dir/Any] |
OK. Did you really mean to use -s¹ in the
shebang line, or did you mean -w?
There are a lot of regular expressions in your program, but
no comments and every single variable name is composed of a
single character. It looks like it fell out of a 1970s
Fortran/Basic timewarp. The indenting is reasonable. Marks out
of 10: 3 (provisional, based on the assumption that you
meant to run with warnings enabled).
¹From the Perl 5 Pocket Reference:
-s Interprets -xxx on the command line as a switch
and sets the corresponding variable $xxx in the
script to 1. If the switch is of the form -xxx=yyy
the $xxx variable is set to yyy.
| [reply] [Watch: Dir/Any] [d/l] [select] |
OK, in order to even attempt a sensible answer to the
question, we'll need more information.
- Given the presence of the -s we'll need to
know the command line that you use to run the script so we
can know valid values for $matchstr, $g, $r and $y.
- The script seems to closely tied to the syntax of the
input file, so it would be helpful to see some sample
data.
Is that more helpful?
p.s. I still think it's a horrible script :)
--
<http://www.dave.org.uk>
"The first rule of Perl club is you don't talk about
Perl club."
| [reply] [Watch: Dir/Any] |