Ideally, you would do this in some new() method, but _init() should work just fine, too. You want to use __PACKAGE__ because you want the methods defined at the lowest package possible. That way, code re-use is preserved.

Update: broquaint made the good point that this could also be put in some BEGIN block.

my @methods = qw(
    param
    foo
    bar
);
sub _init
{
   my $self = shift;

   {
      no strict 'refs';
      unless (defined \&{__PACKAGE__."::$methods[0]"})
      {
         foreach my $method (@methods)
         {
            *{__PACKAGE__."::$method"} = sub {
               my $self = shift;
               return undef unless UNIVERSAL::isa($self->query}, CGI);
               return $self->{query}->$method{@_);
            };
         }
      }
   }

# Rest of stuff here.
}
[download]

I personally don't like AUTOLOAD. It smacks too much of hidden black magic to me. :-)

------
We are the carpenters and bricklayers of the Information Age.

Don't go borrowing trouble. For programmers, this means Worry only about what you need to implement.