Du hast nach dem else eine schließende Klammer zu viel und vor dem else fehlt eine (die zum if gehört).

Noch ein paar Hinweise:

*) Du solltest die ?-Notation von DBI verwenden, um die Sicherheit Deines Skripts zu erhöhen.

*) Warum verwendest Du mal die objektorientierte Form von CGI und mal die importierten Funktionen? Du solltest Dich auf eins festlegen.

*) Du solltest Dir mal Template-Systeme wie HTML::Template::Compiled anschauen. Damit kannst Du das Layout (HTML) vom Code (Perl) trennen.