in reply to setting session variables

You could simplify your code's appearance by defining a utility function like

sub maybe_set_param { my ( $self, $param, $default, $value ) = @_; $self->session->param( $param => defined $value ? $value : $default +); }
Then you'd replace the param-setting if-statement with
$self->maybe_set_param( page => $post{ page }, 1 );
Or, if you always get your param values from this %post hash, then you can simplify things a bit further like this:
sub maybe_set_param { my ( $self, $param, $default ) = @_; my $value = $self->post->{ $param }; $self->session->param( $param => defined $value ? $value : $default +); } # ... $self->maybe_set_param( page => 1 );
In the last version, I used an accessor post defined to return a ref to the %post hash.

And from here you can simplify things even further, by making maybe_set_param capable of handling multiple params in one call:

sub maybe_set_param { my $self = shift; my %defaults = @_; my $post = $self->post; while ( my ( $param, $default ) = each %defaults ) { my $value = $post->{ $param }; $self->session->param( $param => defined $value ? $value : $defaul +t ); } }
Now you can set various parameters at once like this:
$self->maybe_set_param( page => 1, foo => 15, bar => -1 );

the lowliest monk

Replies are listed 'Best First'.
Re^2: setting session variables
by boboson (Monk) on May 24, 2005 at 08:15 UTC
    now you're talking...thanks alot