Sibling monks have alluded to this but not quite spelt it out: I'd strongly encourage you to Use More Modules.

Even in this small script you could save yourself some lines of code, improve readability, increase flexibility and guard against snafu, by using Mail::Sendmail.

And of course (I mean *of course*) you could be using CGI.pm *much* more. The whole of parse_form() would become redundant, just to start with.

There's such a wealth of good work in CPAN; and of course there is some time to be spent learning how to use each new module; but you get that time back almost at once as the module speeds up your work.

And learning to use a module puts you in the way of looking over some of the best code around, which does no harm to one's programming style.

§ George Sherston