UBB.Developers Forums Community Development Programming PHP and Server-side Scripting Anything to improve?

Since you're using CGI.pm objects, I think it would be cleaner to have 'use CGI()' instead of 'use CGI qw(:standard)', to avoid unnecessary importing of names into the global namespace.

If you do that, 'print header' should be 'print $q->header()'.

It's also a good idea to turn on warnings, taint and strict modes:
#!/usr/bin/perl -wT
use strict;
use re 'taint'; # suppress implicit untainting by regexes

With strict mode on, all variables have to be declared:
my $q = new CGI;

After reading about possible ambiguities in accessing object methods, I've gotten in the habit of using this style when calling an object constructor:
my $q = CGI::->new();

If I'm wrong about any of these suggestions, please jump in and correct me

This message has been edited by Dave_L on January 18, 2001 at 10:31 AM