ColtsFoot, UnderMine, and rdfield have all made good observations I missed. Have you looked into their suggestions?

That done, here are some debugging techniques.

• (Added) Pick an indenting style and use it religiously. perlstyle is good dogma if you want help with that.
• Turn on warnings with -w on the splatline or use warnings;
• Run the script from the command line, giving the cgi form parameters as name=value pairs on the command line.
• Once that works, redirect the html printed to a file and read that file with a browser. Check it for html compliance.
• When ready to try it on the server, temporarily add use CGI::Carp qw(fatalsToBrowser warningsToBrowser); right after use CGI;. With that you can see errors live without having to go to the server error log.

The key is to get the system to tell you what it thinks is wrong.

I meant to comment earlier that your code shows great promise. There are lots of little errors, but your use of the correct modules in proper ways is a good sign. You are trying to do the right things, and you do nothing useless or wrong headed.

After Compline,
Zaxo