Array, Implode, missing 1 value

MystedVeil

I am currently using this code to pull out a comma separated list of each Specialty that a given race can be. Something is going wrong as it is missing one Specialty, the first one listed.

$RaceID = $_GET['RaceID'];
$result = mysql_query("SELECT DISTINCT RaceSpec.SpecialtyID, Specialties.SpecID, Specialties.Specialty, RaceSpec.RaceID from RaceSpec, Specialties WHERE RaceSpec.RaceID='$RaceID' AND RaceSpec.SpecialtyID=Specialties.SpecID ORDER BY Specialties.Specialty");
 if (mysql_num_rows($result)) 
 while($row = mysql_fetch_array($result)) {
    $Specialty = array();

while($row = mysql_fetch_array($result)) {$Specialty[] = $row['Specialty'];}
echo '' . implode(', ',  $Specialty);
 }

Krik

Use [man]print_r[/man] or [man]var_dump[/man] to see if all the values are in the array to start with

And if the Specialty array is missing it, check the row array also

MystedVeil

14 out of 15 are showing, even though the 15th belongs there too. All 15 are in the db.

array(14) { [0]=> string(12) "Race 1" [1]=> string(10) "Race 2" [2]=> string(5) "Race 3" [3]=> string(10) "Race 4" [4]=> string(5) "Race 5" [5]=> string(8) "Race 6" [6]=> string(11) "Race 7" [7]=> string(5) "Race 8" [8]=> string(7) "Race 9" [9]=> string(6) "Race 10" [10]=> string(12) "Race 11" [11]=> string(6) "Race 12" [12]=> string(8) "Race 13" [13]=> string(7) "Race 14" }

it has however actually ignored the first alphabetically listed row

MystedVeil

replaced

$Specialty = array();

with

$Specialty = array($row['Specialty']);

and now it pulls up properly.

bradgrafelman

That's because you've got a while() loop within a while() loop. Why do you have two loops that retrieve data from the database? You should be using only one loop.

The reason you're missing the first row is that the outer while() loop retrieves that row without ever doing anything with it; the inner while() loop retrieves rows 2 to the end and echo's the data.

MystedVeil

Thank you, I removed that portion of the code and it is truly working properly.

bradgrafelman

Glad you got it working - don't forget to mark this thread resolved (if it is) using the link on the Thread Tools menu above.