In addition to dws's and grep's notes, you are not printing the submit and reset buttons, and the end_form method. You probably meant comma for semicolon in the previous print statement.

Update: Clarified language, added ref to grep's contribution.

After Compline,
Zaxo