displaying mysql results in columns

caralane

I am trying to make this start a new colum after the first 18 rows. I do not get an error, but it just shows 1 column all the way down the page. Ideally, I would like it to fill across 4 cells and then begine the next row. Any help would be appreciated.
<?
if(!IsSet($start))
$start = 0;
$result = mysql_query("SELECT site_name, site_address, site_image, publish FROM sites WHERE publish = 'Y' ORDER BY site_name ASC LIMIT $start,144") or die(mysql_error());
?>
<table width="100%" border="0" cellspacing="4" cellpadding="4" align="center">
<?
$row_count = 0;
while($row = mysql_fetch_object($result))
{

if($row_count % 18 == 0)
{
$row_count = 18;
print "<tr>";
}
?>
<td width="25%">
<div align="center">
<p><a href= "<? echo $row->site_address ?>" target="blank"><img src="samples/<? echo $row->site_image ?>" width="100" height="75" border="0">
</a><br>
<br>
<a href= "<? echo $row->site_address ?>" target="blank">
<? echo $row->site_name; ?>
</a> </p>
</div>
</td>
</tr>
<?
if($row_count % 18 == 0)
{
$row_count = 0;
print "<tr>";
}

$row_count++;
}
?>
</table>

pinkbike

At a glance a couple of problems. in your last
if($row_count % 18 == 0)
{
$row_count = 0;
print "<tr>";
}

the <tr> should be </tr>

Also you want to generate the beginning <tr> at the beginning of the row and the </tr> at the end. Since the same IF condition generates both the <tr> and the </tr> you will get them only generated in the same itteration. You probably want something like
if($row_count==17) //for a 18 col setup
{
$row_count = 0;
print "</tr>";
}

Anyway, here is a bare bones setup of what you want to do. Just check out the structure and plug your own data in.

$cColumns=8;
$cDatamax=53;

$usRow_count=0;
$usData_count=0;

while($usData_count<$cDatamax)
{
$usData_count++;

if (($usRow_count % ($cColumns)) == 0)
{
print("\n<tr>");
}
?>
<td>
<?echo $usData_count?>
</td>
<?

if (($usRow_count==($cColumns-1)))
{
print("</tr>");
}

$usRow_count++;

}

</table>

Hope if makes sense as I'm a little new to all this stuff.