# corrected original
my ($value1, $value2, $value3) = (1..3);

print $cgi->Tr({ -align=>'LEFT', -valign=>'TOP'},
    [
     td( { -OnClick=>"javascript:dothis()" },
        [ $cgi->a( {-href=>"http://example.com", -target=>"_new"}, $cgi->img('blah.gif')) .
                $cgi->a( {-href => 'http://example.com' }, $value1 ). "\n",
          $cgi->hidden( -name=>'name2', -value=>"$value2" ).
                $cgi->a( {-href => 'http://example.com' }, $value2 ) . "\n",
          $cgi->hidden( -name=>'name3', -value=>"$value3" ).
                $cgi->a( {-href => 'http://example.com' }, $value3 ) . "\n",
        ]),
    ]);
print "\n";

# td call separated
print $cgi->Tr({ -align=>'LEFT', -valign=>'TOP'},
    td($cgi->a( {-href=>"http://example.com", -target=>"_new"}, $cgi->img('blah.gif')) .
                $cgi->a( {-href => 'http://example.com' }, $value1 )) . "\n" .
     td( { -OnClick=>"javascript:dothis()" },
        [
          $cgi->hidden( -name=>'name2', -value=>"$value2" ).
                $cgi->a( {-href => 'http://example.com' }, $value2 ) . "\n",
          $cgi->hidden( -name=>'name3', -value=>"$value3" ).
                $cgi->a( {-href => 'http://example.com' }, $value3 ) . "\n",
        ]),
    );


##</code><code>##

my @names = (qw(name1 name2 name3));

my %cells = (name1 => {href => 'http://example.com/1', value => $value1},
             name2 => {href => 'http://example.com/2', value => $value2},
             name3 => {href => 'http://example.com/3', value => $value3});

sub row {
  my ($names, $cells) = @_;
  my $first = shift @$names;
  my @result;
  push @result, CGI::td(CGI::a({-href=> $cells->{$first}{href}, -target => "_new"}, CGI::img('blah.gif')) .
                         CGI::a({-href=> $cells->{$first}{href}}, $cells->{$first}{value}));
  for my $name (@$names) {
    push @result, CGI::td({ -OnClick=>"javascript:dothis()" },
                          CGI::hidden(-name => '$name', -value => $cells->{$name}{value}) .
                          CGI::a({-href => $cells->{$name}{href}}, $cells->{$name}{value}));
  }
  return join '', @result;
}

print $cgi->Tr({ -align=>'LEFT', -valign=>'TOP'}, row(\@names, \%cells));