Don't Repeat Yourself.The fetching part of the script should be rolled into a function that handles a given strip, and then called for each of the strips. What you're doing there is a maintenance nightmare.
is effectivelyopen PAGE, ">comics.html" || die $!;
The part in parens will ALWAYS evaluate to ">comics.html". What you want is:open PAGE, (">comics.html" || die $!);
which is effectivelyopen PAGE, ">comics.html" or die $!;
(open PAGE, ">comics.html") or die $!;
open my $fh, ">", "comics.html" or die $!;
xoxo,
Andy
--
<megaphone>
Throw down the gun and tiara and come out of the float!
</megaphone>
In reply to Re: Online Comics code review
by petdance
in thread Online Comics
by Boots111
| For: | Use: | ||
| & | & | ||
| < | < | ||
| > | > | ||
| [ | [ | ||
| ] | ] |