[Resolved] while statement droping one record?

chriswheat

Hello,

This one is strange, for some reason the code only echo's 6 out of the 7 rows? This happens no matter how you sort it?

Heres the code
if (!$stats_sort){
$stats_sort = "lname";
}else{}
$sql = mysql_query("SELECT * FROM employee order by $stats_sort asc");
$result = mysql_fetch_array($sql);
if(mysql_num_rows($sql) > 0) {
// If there are rows, make the table
echo "<table width=\"100%\" border=\"0\" cellpadding=\"5\">
<tr>
<td><a href=\"?stats_sort=lname\">Employee Name</a></td>
<td><a href=\"?stats_sort=last_login\">Last
Login</a></td>
</tr>";
while($row=mysql_fetch_array($sql)) {
//
echo "<tr>
<td bgcolor=\"$row_color\">".$row['aero_id']." - ".$row['fname']."</td>
<td bgcolor=\"$row_color\">".$row['last_login']."</td>
</tr>";
}
echo "</table>";
//end the table
}

heres the link
http://www.aeroportal.net/pleasehelp/stats.php

as you can see when you click on last login it displays records 1,2,4,5,6,7 and leaves out #3

and when you click on Employee name it displays recods
1,2,3,5,6,7 and leaves out #4

Any ideas? This is driving me crazy.

Chris

Pig

When you do this:
$result = mysql_fetch_array($sql);
You are getting the first row and incrementing the internal pointer.

Then when you do this:
while($row=mysql_fetch_array($sql))
you are already on the second row.

Pig

Also, your naming conventions are all messed up. $sql should be a text string, $result should be the query result, and $row should be a single row from the resulting array.

You should be able to delete
$result = mysql_fetch_array($sql);
altogether.

chriswheat

I don't understand. "your naming conventions are all messed up" Can you post the correct code so that I can see the correct way to do this?

Chris

Pig

$sql = "SELECT * FROM employee order by $stats_sort asc";
$result = mysql_query($sql);
if( mysql_num_rows($result) == 0 ) echo("No matches found");
else
{
     echo("put <table> opener and header <tr> stuff here.");
     while( $row = mysql_fetch_array($result) )
     {
          //stuff here
     }
}

chriswheat

Thank You!!

Chris