OK, so a couple of generic comments:
- first this section is meant for complete code,
not code where some sub-routines (parseget
and dieniceare missing.
- I know I need to learn the CGI module, etc., but
all things in due time.
_Now_ is the time. Actually before you wrote this
program would have been the time. It's not like
CGI.pm is hard to grasp:
add a use CGI pw(:cgi-lib); at the top
of your code, then my %FORM= Vars;
instead of your call to parseget and your
code will run unchanged, except that the parsing is
most likely way more robust than what you have right
now. It takes about 10 minutes of scanning the CGI.pm
doc to find it, how long did it take you to write
parseget?
- Your indentation style is highly confusing, in your
main loop the block under <code>if ($urlcategory ne $cattest){
is not indented, this is the kind of code that makes
Python evangelists happy ;--(
- I see you use -wT which is good, but you
forgot their friend use strict;. I would
recommend you use strict for anything that's not a one-liner.
It will catch typos in variable names for examples.
