in reply to clean code

use CGI::Simple; my $data = CGI::Simple->new( $memory_data ); # At this point, you just do $data->param( 'foo' ) to retrieve the val +ue # associated with 'foo'. Instead of what you're doing which is $data{ +'foo' };

My criteria for good software:
  1. Does it work?
  2. Can someone else come in, make a change, and be reasonably certain no bugs were introduced?