Zebra

[Asday]

This code:

 

<?php
$j=0;
function shade($mode)
{
if ($mode == 0)
{
	echo '</tr><tr style="background-color:#DDDDDD">';
	$j = 1;
}
else
{
	echo '</tr><tr>';
	$j = 0;
}
}

$i=0;

foreach(scandir("upld194") as $x)
{
if ($i == "2")
{
	shade($j);
	//echo '</tr><tr>';
	echo '<td><a href="upld194/' . $x . '">' . '<img src="upld194/' . $x . '">' . '</a></td>';
	$i = 0;
}
else
{
	echo '<td><a href="upld194/' . $x . '">' . '<img src="upld194/' . $x . '">' . '</a></td>';		$i++;
}
}
?>

 

which is meant to shade every other row in grey, shades every row but the first grey.  Why?

[sasa]

try

function shade(&$mode)
{
if ($mode == 0)
{
	echo '</tr><tr style="background-color:#DDDDDD">';
	$mode = 1;
}
else
{
	echo '</tr><tr>';
	$mode = 0;
}
}

[ToonMariner]

needs to be the other way around

 

<?php

$j=0;

function shade($mode)

{

if ($mode == 1)

{

echo '</tr><tr style="background-color:#DDDDDD">';

$j = 0;

}

else

{

echo '</tr><tr>';

$j = 1;

}

}

 

[socratesone]

Is there a reason why you need to separate it into another function? Wouldn't this be easier?

foreach(scandir("upld194") as $x){
  if($altRow==0){
    echo '</tr><tr style="background-color:#DDDDDD">';
    $altRow=1;
  }else{
    echo '</tr><tr>';
    $altRow=0;
  }
  echo "<td><a href="upld194/' . $x . '">' . '<img src="upld194/' . $x . '">' . '</a></td>";
}