There must be a better way to write this

[raydot]

Hi all,

I wrote some code that I'm using to color text in a table across the diagonal from the top right to the bottom left of the table.

The code I wrote to do this is (psuedocode example):

$highlighter = 0;
$hrow = 9;
while ($row = [data set pulled from db]) {
[INDENT]echo "<TR><TD>".$row['data']."</TD>";
for ($i = 0; $i <= 9; $i++){
[INDENT]if ((($highlighter++) % 10) == $hrow { //highlight the text
echo "<TD bgcolor='#FFFFCC'>".$row['otherdata']."</TD>";
} else { //no hilight
echo "<TD>".$row['otherdata']."</TD>"
} //end if [/INDENT]
} // end for
echo "</TR>";
$hrow--; [/INDENT]
} // end while

Seems like one variable too many ($hrow or $highlighter), and the modulo calculation seems wonky, but I looked at it for quite a while and couldn't think of any other way. Anyone got a better idea?

Thanks.

[raydot]

I forgot to mention, it's a 10 x 10 item table.

[Weedpacket]

for($i=0; $i<10; $i++)
{
    $row = ...;
    echo "<tr>";
    for($j=0; $j<10; $j++)
   {
      $diagonal = ($i==$j) ? 'style="background-color:#ffc"' : '';
      echo "<td $diagonal></td>";
   }
  echo "</tr>";
}

[raydot]

I'll give that a try, and nice sig!