# Line 20-24 my $year = $cgi->param('year') ? encode_entities($cgi->param('year'), '<>"') : ''; my $media = $cgi->param('media') ? encode_entities($cgi->param('media'), '<>"') : ''; my $genre = $cgi->param('genre') ? encode_entities($cgi->param('genre'), '<>"') : ''; my $source = $cgi->param('source') ? encode_entities($cgi->param('source'), '<>"') : ''; my $title = $cgi->param('title') ? encode_entities($cgi->param('title'), '<>"') : ''; #### # How I'da dunnit: my %param = map { $cgi->param($_) ? ($_ => encode_entities($cgi->param($_), '')) : () } qw(year media genre source title); #### # Line 33-39: if ( $year || $media || $genre || $source || $title ) { my @selections; push @selections, qq($title) if $title; push @selections, qq($year) if $year; push @selections, qq($media) if $media; push @selections, qq($genre) if $genre; push @selections, qq($source) if $source; ... #### # How I'da dunnit: if (keys %param) { my @selections = values %param; ...