in reply to Re^6: Cleaning up Code
in thread Cleaning up Code

Honestly, I haven't been motivated to address this post since it'll take a bit of time and the issues I've already raised haven't been addressed.

I'd also like to see the following replaced with CGI or CGI::Simple:

#Parse Data read(STDIN, $buffer, $ENV{'CONTENT_LENGTH'}); @pairs = split(/&/, $buffer); foreach $pair (@pairs) { ($name, $value) = split(/=/, $pair); $value =~ tr/+/ /; $value =~ s/%([a-fA-F0-9][a-fA-F0-9])/pack("C", hex($1))/eg; $value =~ s/^\s+//; $FORM{$name} = $value; }

That means that instances of $FORM{XXX} become calls to param.

Replies are listed 'Best First'.
Re^8: Cleaning up Code
by atmosphere (Novice) on Dec 17, 2008 at 20:31 UTC
    I am in the process of cleaning up the code with your recommendations, it is unfortunately taking some time to do so as I now have several errors in my code to deal with. Once I have completed all of your suggestions I will reply back with my modified code, and/or questions.

    Again thanks for all your help thus far... Don't give up on me just yet :)
Re^8: Cleaning up Code
by atmosphere (Novice) on Jan 09, 2009 at 16:17 UTC
    I have made the adjustments you have suggested, and according to strict, and warnings the only errors I have now are with the block of code you have suggested. Any ideas, or obvious errors in the code that I may have messed up... I will post the errors I am receiving shortly.