alternate colors between rows.

[peter2332]

Hi
The following code creates alternate colors between (table) rows.But for some error the code is not working,can anyone point out the error?

<?php
// Define colors here

$color_one = '#FFFFFF';
$color_two = '#BBCCED';

$array = array('apple','pine','strawberry',
                    'pear','banana', 'cranberry','kiwifruit');

$i = 1;

?>

<!-- Table with alternate colors -->

<table width="100%" height=100 width=150 cellspacing="0">
<?php
foreach($array as $value)
{
/*
Check if ($i / 2) is an integer and determine which color to show.
We will divide the number by 2
*/

$color = (is_int($i/2)) ? $color_one : $color_two;
?>
<tr>
<td bgcolor="<?=$color?>"><?=$value?></td>
</tr>
<?php
$i++; // Increment $i
}
?>
</table>

[Ashley_Sheridan]

The preferred method to check if a row is odd or even is to use the modulus (&#37😉 of a number when divided by 2:

$color = ($i%2) ? $color_one : $color_two;

Also, it's typical to set a class rather than use the bgcolor attribute, which is deprecated and results in a lot of extra code. Note also that you only need to set the class for the odd rows, because if you set a default for all the rows, just changing the odd rows gives you the alternate colour effect.

Lastly, you can do away with all this with a small bit of CSS code:

td {background-color: #fff;}
tr:nth-child(even) td {background-color: #bbcced;}

[peter2332]

Hi
CSS or anything else, apart from HTML and PHP can be used here.

[peter2332]

Hi Sherley,
your suggestion still doesn't work & u cant use CSS here.Only php & html to be used .

[Ashley_Sheridan]

What code do you have now?

(puts on best Leslie Nielsen voice) Don't call me Sherley 😉

[peter2332]

Hi
Sorry mistakenly called u Sherley.I have the same code as I gave u at first.Even ur altered code doesn't work.
Madam,I am saying that u cant use CSS or anything else apart from PHP & HTML.
Hope u now got me.

[Ashley_Sheridan]

I've altered your code in the way I first suggested and it works fine, that's why I asked what code you have now. Here is your code with the suggested amendment (also fixed some generaly formatting problems and replaced the short tags with the longer versions which are less prone to break on different servers)

<?php
// Define colors here
$color_one = '#FFFFFF';
$color_two = '#BBCCED';

$array = array('apple','pine','strawberry','pear','banana', 'cranberry','kiwifruit');

$i = 1;
?>

<!-- Table with alternate colors -->
<table width="100%" height=100 width=150 cellspacing="0">
<?php
foreach($array as $value)
{
	/*
	Check if ($i / 2) is an integer and determine which color to show.
	We will divide the number by 2
	*/

$color = ($i%2) ? $color_one : $color_two;
?>
<tr>
<td bgcolor="<?php echo $color; ?>"><?php echo $value; ?></td>
</tr>
<?php
$i++;
}
?>
</table>

Oh yes, and I'm not a madam!

[Bonesnap]

Ashley Sheridan;11003136 wrote:

tr:nth-child(even) td {background-color: #bbcced;}

The only issue with this is it's a CSS3 property and will not render in IE8 and under. The PHP method checking for the modulus is probably the safest bet.

[NogDog]

I would prefer to assign a class attribute based on the modulus result:

<?php
$counter = 0;
// start your loop, and then...
?>
<tr class='<?php echo ($counter++ % 2) ? 'odd' : 'even';?>'><td>...

Then you can assign the desired color(s) in your CSS for either or both of the classes used.

[laserlight]

peter2332 wrote:

I am saying that u cant use CSS or anything else apart from PHP & HTML.

You should be using CSS here because this is about presentation style. If you cannot use advanced CSS for fear of lack of browser support, then using PHP as a template language to set CSS classes would be a suitable alternative.