What the poster means is that you should be using the CGI.pm module instead of trying to roll your own query parsing. There are numerous, well-documented reasons for this (see use CGI or die;, for example).

As for your question, you really haven't provided enough information for anyone to give you an intelligent answer. The reason for your program's failure can't be deduced from the snippet of code you've listed.