Und der Verzicht auf CGI::Vars() ist auch nicht gerechtfertigt, da Du dann halt bei Elementen, die "\0 packed strings" liefern selbst splitten kannst.
Aber CGI bricht garantiert keine Skripte ab. Ich selbst benutze es so (mit Vars() und dann splitten) in meinen Skripten.
Dann könnte man das getcgihash z.B. schon mal komplett vergessen...

Und ich kann dem nur zustimmen, dass man die HTML-Sachen am besten irgendwie auslagert...

Du hast gesagt, dass das Programm nicht ausgeführt wird. Hast Du mal mit Testausgaben probiert, ob er überhaupt die if-Abfrage so macht, wie Du es willst. Oder wird das Programm einfach nicht ausgeführt??