odd numbers

[jowatkins]

Hi,

I want to return results from a query with alternate row formatting, but it won't work for some reason.

$i=1;
while(list($key,$value)=each($units))
{
  if(($i % 2) )
  {
      $col = "#9999ff";
  }
  else
  {
       $col = "#9999ff";
  }

  printf("\n  <tr bgcolor=\"$col\">");
  //print data here
  printf("\n  </tr>\n"); //end line, new row
  $i++;
}

Any ideas? it just keeps outputting the color for an even number (darker blue).

[The_Chancer]

Use this little code snippet

while(list($key,$value)=each($units))
{
if ( $count == 1) 
{ 
	$bg	= '#cococo';
	$count = 0;
}
else
{
	$bg	= '#dddddd';
	$count= 1;
}
print("<tr>
<td bgcolor='$bg' align=center></td>
}

[jowatkins]

I did something like this originally, but when I retrieve a large amount of records it goes wrong. As best I can tell it's always on the same lines, but rows 21&22 will be the same, and then it will go back to alternate, and then another few rows the same color. Thats why I thought it might be better to go by odd/even numbers.

[The_Chancer]

Very odd, I use it for approx 5,000 records and all turn out perfectly...

Incidentially, do you know of a way to reproduce the headers every so often (say every 50 rows)
A function like

if (($i/50)== //a whole number but I don't know how to call it)
{
//reprint the table headers
}

any ideas ?

[jowatkins]

poss.

if ($i % 50)
{
  echo $header;
  echo $row;
} 
else 
{
  echo $row;
}

I think the % acts as a divider, so if it can devide $i by 50, then returns true, other wise:

if (($i == 50) || ($i == 100))

etc.

[bobthedog]

% is a modulus operator - i.e. it returns the remainder of any division...

so --

0 % 50 = 0 (or 0 with a 0 remainder)
1 % 50 = 1 (or 0 with a 1 remainder)
2 % 50 = 2 (or 0 with a 2 remainder)
....
49 % 50 = 49 (or 0 with a 49 remainder)
50 % 50 = 0 ( or 1 with a 0 remainder)
...
61 % 50 = 11 (or 1 with a 11 remainder)

etc etc 🙂

So code shud be:-

if (($i % 50) == 0)
{
echo $header;
}

echo $row;

should print out a header every 50 rows (you may have to change number to 49 - depends on whether you have a 0 based count)

Hope this helps

[The_Chancer]

Bob's your uncle - seems to work for me 🙂

ALternating rows of differing grey niceness with column headers every 50 rows...

Cheers.

code as follows

$result=do_query("SELECT * FROM agents ORDER BY company ASC");
$i=1;
while($agents=mssql_fetch_array($result))
{
	if ( $count == 1) 
 	{ 
		$bg	= '#cccccc';
		$count = 0;
	} 	
	else
	{
		$bg	= '#dddddd';
		$count= 1;
	}
	print("<input type=hidden name=ref[] value=$agents[ref]>
	<tr onmouseover=\"setPointer(this, 'over', '$bg', '#CCFFCC', '#FFCC99')\" onmouseout=\"setPointer(this, 'out', '$bg', '#CCFFCC', '#FFCC99')\" onclick=\"setPointer(this, 'click', '#CCCCCC', '#CCFFCC', '#FFCC99')+detailcheck($agents[ref]);\">
	<td bgcolor=$bg>$agents[company]</td>
	<td bgcolor=$bg>$agents[contact]</td>
	<td bgcolor=$bg>$agents[address1]</td>
	<td bgcolor=$bg>$agents[city]</td>
	<td bgcolor=$bg>$agents[county]</td>
	<td bgcolor=$bg>$agents[postcode]</td>
	<td bgcolor=$bg>$agents[phone]</td>
	</tr>");
	if (($i%50)==0)
	{
		print("
		<tr>
		<td class=th>Company Name</td>
		<td class=th>Contact Name</td>
		<td class=th>Address1</td>
		<td class=th>City</td>
		<td class=th>County</td>
		<td class=th>Postcode</td>
		<td class=th>Phone No.</td>
		</tr>");
	}
	$i++;
}

Incase you wanted to know...

[Weedpacket]

Y'know, if you stopped to see how many records you had before beginning the loop (and you should be, in case there aren't any at all), you could streamline things a bit.

for($i=0; $i<numrows; ++$i)
{   $row=mysql_fetch....blablabla
...
    $bg=($i%2)?'#cccccc':'#dddddd';
...
    if(($i%50)==0)
    { ...do a nice header row again...
    }
}

[The_Chancer]

True - more streamlined and looks nicer in the code...
Little problem though, starting at $i=0 makes a header row after line 1, using

for($i=1; $i<numrows; ++$i)
{   $row=mysql_fetch....blablabla
...
    $bg=($i%2)?'#cccccc':'#dddddd';
...
    if(($i%50)==0)
    { ...do a nice header row again...
    }
}

Stops it though... (and the colours still look nice)

[Weedpacket]

Originally posted by The Chancer
True - more streamlined and looks nicer in the code...
Little problem though, starting at $i=0 makes a header row after line 1, using

//elided

Stops it though... (and the colours still look nice) [/B]

Well, you put the if(i%50) line before the table row, and then the header row will go before line 1. I presume you'll be wanting a header row at the very head of the table 🙂

Otherwise, you'll need to loop up to $i<=$numrows - otherwise the last record in the result set will be missed.