I'm stupid, why doesn't this work?

[grubindex.com]

$result=mysql_query("SELECT * FROM cat");
$category = mysql_fetch_array($result);

$i = 1;
$numrows = mysql_numrows($result);
while($row=mysql_fetch_array($result)) {
$cat=($row["category"]);
extract($row);
printf('<td>%s</td>%s, $category, (($i%2) == 0) ? "</tr><tr>\n" : "");
$i++;
}
if(($numrows%2 == 1)) {
print "<td>&nbsp;</td></tr>";
} 
else {
print "</tr>";
}
?> 

help

[theFinn]

Wow..thats nasty try this:

$i = 1; // Don't forget this
while($row = mysql_fetch_array($result))
{
   extract($row);
   if(!($i % 2))
   {
      $extra = "</tr><tr>";
   }
   echo "<td>$category</td>$extra";
   $i++;
}
... 

Not only is it more readable but it'll work...

[grubindex.com]

i need two columns

[freddydoesphp]

It wasn't nasty in the beginning, it wasn't till after it was butchered that it became nasty. It shoul read like this:

print "<table><tr>";
$result=mysql_query("SELECT * FROM cat");
$numrows = mysql_numrows($result);
$i = 1;
while($row=mysql_fetch_array($result)) {
   extract($row);
   printf('<td>%s</td>%s, $category, ((($i%2) == 0) && ($i != $numrows)) ? "</tr><tr>\n" : "");
   $i++;
   }
if(($numrows%2 == 1)) {
   print "<td>&nbsp;</td></tr>";
   } 
else {
   print "</tr>";
   }
print "</table>";

[grubindex.com]

that doesn't work.