in reply to setting session variables
You could simplify your code's appearance by defining a utility function like
Then you'd replace the param-setting if-statement withsub maybe_set_param { my ( $self, $param, $default, $value ) = @_; $self->session->param( $param => defined $value ? $value : $default +); }
Or, if you always get your param values from this %post hash, then you can simplify things a bit further like this:$self->maybe_set_param( page => $post{ page }, 1 );
In the last version, I used an accessor post defined to return a ref to the %post hash.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 );
And from here you can simplify things even further, by making maybe_set_param capable of handling multiple params in one call:
Now you can set various parameters at once like this: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 ); } }
$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 |