[Resolved] Alternating row color

[cravoz71]

Any help please!!! Here is the short code, it is pulling the data from MySQL just fine, but the row color isn't alternating. Help PLEASE. Kind of new to the whole PHP MySQL world.

<?php
$connection = mysql_connect ("localhost", "root", "");
if ($connection == false){
echo mysql_errno().": ".mysql_error()."<BR>";
exit;
}

$query = "select * from anvils";
$result = mysql_db_query ("monetta_farrier_db", $query);
if ($result){
echo "<table width=415 border=0 cellspacing=2 cellpadding=2 align=left>";

$numOfRows = mysql_num_rows ($result);
for ($i = 0; $i < $numOfRows; $i++){
$row_color = ($row_count % 2) ? $color_one : $color_two;
$color_one = "EEEECC";
$color_two = "EEEECC";
$row_count = 0;
$part_number = mysql_result ($result, $i, "Part Number");
$description = mysql_result ($result, $i, "Description");
$price = mysql_result ($result, $i, "Price");

echo "<tr bgcolor=$row_color>
<td width=115>
<p align=left style='font-family: Verdana, Geneva, Arial, Helvetica, sans-serif; font-size: 8pt; margin-bottom: 0pt; margin-left: 0pt; margin-right: 0pt; margin-top: 0pt;'>
$part_number
</p>
</td>
<td width=255>
<p align=left style='font-family: Verdana, Geneva, Arial, Helvetica, sans-serif; font-size: 8pt; margin-bottom: 0pt; margin-left: 0pt; margin-right: 0pt; margin-top: 0pt;'>
$description
</p>
</td>
<td width=50>
<p align=left style='font-family: Verdana, Geneva, Arial, Helvetica, sans-serif; font-size: 8pt; margin-bottom: 0pt; margin-left: 4pt; margin-right: 0pt; margin-top: 0pt;'>
$price
</p>
</td>
</tr>";
$row_count ++;
}
echo "</table>";
}
else{
echo mysql_errno().": ".mysql_error()."<BR>";
}
mysql_close ();
?>

Thanks

CravoZ71

[drawmack]

add this to the head section of your document:

<style>
  td {
    font-family: Verdana, Geneva, Arial, Helvetica, sans-serif; 
    font-size: 8pt; 
    margin-bottom: 0pt; 
    margin-left: 0pt; 
    margin-right: 0pt; 
    margin-top: 0pt  

    text-align: left;
  }
  lightData {
    background-color: #EEEECC;
  }

  darkData {
    background-color: #DDDDBB;
  }
</style>

and here is the fixed up php code

<?php
$connection = mysql_connect ("localhost", "root", "");
if ($connection == false){
    echo mysql_errno().": ".mysql_error()."<BR>";
    exit;
} //end if

$query = "select * from anvils";
$result = mysql_db_query ("monetta_farrier_db", $query);
if ($result){
    echo "<table width=415 border=0 cellspacing=2 cellpadding=2 align=left>";

$numOfRows = mysql_num_rows ($result);
//moved from inside loop, see below comments for
//reasoning
$row_count = 0;
for ($i = 0; $i < $numOfRows; $i++){
    //now if you're already counting the rows with i
    //why introduce another variable named row_count?
    $row_class = ($i % 2) ? 'lightData' : 'darkData';
    //both of the color codes are the same and they
    //are defined after they are used, I'm changing 
    //to work off of styles.
    //$color_one = "EEEECC"; 
    //$color_two = "EEEECC";

    //now not only have you introcuded a redunant 
    //variable but you set it to 0 every time through 
    //the loop
    //$row_count = 0;
    $part_number = mysql_result ($result, $i, "Part Number");
    $description = mysql_result ($result, $i, "Description");
    $price = mysql_result ($result, $i, "Price");
    echo "<tr class=\"$row_class\">
  <td width=115>
      $part_number
  </td>
  <td width=255>
      $description
  </td>
  <td width=50>
      $price
  </td>
</tr>";
        $row_count ++; 
    } //end for
    echo "</table>";
} else {
echo mysql_errno().": ".mysql_error()."<BR>";
} //end if
mysql_close ();
?>

I am pretty sure that row_count can be removed now.

[cravoz71]

??? I like the control of the styles, never thought of doing it that way, but it isn't adding color to the alternating rows. Any more suggestions?

Thanks

CravoZ71

[drawmack]

in the style section I forgor the . befor the abstract class names they should be

.lightData and .darkData

[cravoz71]

Thanks for the help. You have just taught me something new!

Chris

[drawmack]

np please mark the thread resolved.