Ick. That's really hard to read through. Everything pretty much looks the same. I also find that using CGI's methods to generate HTML to create messy code and prefer to use a templating module such as HTML::Template. A template something like this should do the trick:

<TMPL_LOOP PHONEBOOK>
    <tr>
    <td>
        <TMPL_IF PIC>
            <TMPL_IF TYPE_OTHER>
                <img src="/images/thumb_<TMPL_VAR NAME=PIC>"
            </TMPL_IF>
                <a href="<TMPL_VAR NAME=PIC"><TMPL_VAR NAME=NAME></a>
       <TMPL_ELSE>
            <TMPL_VAR NAME=NAME>
        </TMPL_IF>
    </td>
    <td><TMPL_VAR NAME=PHONE></td>
    <td><TMPL_VAR NAME=FAX></td>
    <td><TMPL_VAR NAME=LOC></td>
    <td>
        <TMPL_IF TYPE_C>
            <TMPL_VAR NAME=CON>        
        <TMPL_ELSE>
            <a href="mailto:<TMPL_VAR NAME=EMAIL>"><TMPL_VAR NAME=EMAI
+L></a>
        </TMPL_IF>
    </td>    
    </tr>
</TMPL_LOOP>
[download]

Then all you have to do is create an LoH (list of hashes) that has your data plus TYPE_C and TYPE_OTHER set and use it like:

my $t = HTML::Template->new;
$t->param(PHONEBOOK => \@data);
print $t->output;
[download]

This has the advantage of seperating out your code and design and allowing somebody else to tweak the output. For more info on HTML::Template see its documentation or using CGI, DBI and HTML::Template (a mini tutorial with example code).

Hope this helps.

gav^