A po co tyle tych if'ów

Spróbuj czegoś takiego:
<?php
$arrTab = array( 'czarny' => '', 'zielony' => '', 'szary' => '', 'czerwony' => '', 'żółty' => '', 'niebieski' => '', 'fioletowy' => '', 'biały' => '' );
$arrTab[ $_POST[kolor] ] = 'selected';
echo\"
<tr class=$clas>
<td align='right'><b>Ulubiony kolor:</b>
</td>
<td class='formularz'>
<select size='1' name='kolor'>
<option value='wybierz'>-Wybierz-</option>
<option $arrTab[ 'czarny' ] value='czarny'>Czarny</option>
<option $arrTab[ 'zielony' ] value='zielony'>Zielony</option>
<option $arrTab[ 'szary' ] value='szary'>Szary</option>
<option $arrTab[ 'czerwony' ] value='czerwony'>Czerwony</option>
<option $arrTab[ 'żólty' ] value='zolty'>Żółty</option>
<option $arrTab[ 'niebieski' ] value='niebieski'>Niebieski</option>
<option $arrTab[ 'fioletowy' ] value='fioletowy'>Fioletowy</option>
<option $arrTab[ 'biały' ] value='bialy'>Biały</option>
</select>
<br>
</td>
</tr>
\";
?>
Można by to jeszcze bardziej skrócić puszczając w jakąś pętlę, ale chciałem pokazać metodę. Skrócić możesz sam.
P.S. Pisane z głowy, nie testowałem, ale powinno działać ew. po niewielkich poprawkach.