Problem with arrays and looping?

jwer78

Well not sure what Im doing wrong here, so here goes. Im trying to make a dynamic bid module for my league website. People will place bids on players so players can have multiple bids. I am trying to pull the team whos total bid(fbid) is the highest on a given player, and then loop it to the next player; but somewhere I screwed up lol. Any help would be greatly appreciated.

$query = "SELECT * FROM nuke_fa_bids";
$r = mysql_query($query) or die("Could not query: " . mysql_error());
while ($row = mysql_fetch_array($r, MYSQL_ASSOC)) {
    $player = $row['Player'];
    if (!$result[$player] || $result[$player]['bid'] < $row['Fbid']) {
        $result[$player]['bid'] = $row['Fbid'];
        $result[$player]['team'] = $row['Team'];
        $result[$player]['position'] = $row['Position'];
    }
}

$count = 0;

echo '
    <table border="1" cellpadding="3" cellspacing="0">
        <tr>
            <th>Team</th>
            <th>Player Name</th>
            <th>Position</th>
            <th>Winning Bid</th>
        </tr>';

foreach ($result as $key => $value) {
    if (($count % 2) == "0") {
        $color = "D6CFCE";
    } else {
        $color = "0000FF";
    }

echo '
    <tr style="background-color:' . $color . ';">
        <td align="left"><img src="images/smalllogos/'. $value[team] .'.gif"></td>
        <td align="center" class="boldblacktext">' . $key . '</td>
        <td align="center" class="boldblacktext">' . $value[position] . '</td>
        <td align="center" class="boldblacktext">' . $value[bid] . '</td>
    </tr>';
    $count++;
}
echo '</table>';

If I echo $player it shows the players, If I echo $row['xxx'] it will show whatever I put, but however it will not echo $key, $value[xxx] or $result[$player]['xxx']. Again any help would be greatly appreciated. Also I should mention this was not how my original script worked(well didnt work lol) someone helped he tidy it up but now nothing at all shows except for the first table row with the column names. The person helped me touple shoot but everything seemed fine and it was working for him so I am hoping maybe to find some new ideas. Thanks again.

Weedpacket

Looks reasonable (except that you're missing the quotes in $value[team], etc.). What is it echoing if it's not echoing what it's supposed to?

Jack_McSlay

while ($row = mysql_fetch_array($r, MYSQL_ASSOC)) {
    $player = $row['Player'];
    if (!$result[$player] || $result[$player]['bid'] < $row['Fbid']) {
        $result[$player]['bid'] = $row['Fbid'];
        $result[$player]['team'] = $row['Team'];
        $result[$player]['position'] = $row['Position'];
    }
}

are you sure the database column names in the table are 'Fbid','Team'... array keys are case-sensitive on php

add this line after that loop to make sure the data is set to $result correctly

echo '<pre>'.print_r ($results).'</pre>';

jwer78

Yes the fields names are correct. Adding that line echos a 1 and a blank line...

jwer78

This is how I originally wrote this. My only problem was it wasnt looping. It was just returning 1 result.

$mysql_query = "SELECT Player from nuke_fa_bids group by Player";
$res=mysql_query($mysql_query);
while($result = mysql_fetch_array($res)){
$current_player=$result['Player'];
$current_id=$result['id'];

$mysql_query = "SELECT * from nuke_fa_bids where Player='$current_player' order by Fbid DESC LIMIT 1";
$res=mysql_query($mysql_query);
$num_rows = mysql_num_rows($res);
for($count=0;$count<$num_rows;$count++){
$team[$count]=mysql_result($res,$count,"Team");
$player_name[$count]=mysql_result($res,$count,"Player");
$player_pos[$count]=mysql_result($res,$count,"Position");
$fbid[$count]=mysql_result($res,$count,"Fbid");
}
}
$tdbgcolor = array("D6CFCE","0000FF");
    for($count=0;$count<$num_rows;$count++){
        echo "<tr>";
        echo "<td bgcolor=" . $tdbgcolor[$count%2] . " align=left class=boldblacktext><img src=images/smalllogos/". strtolower($team[$count]) .".gif></td>";
        echo "<td bgcolor=" . $tdbgcolor[$count%2] . " align=center class=boldblacktext>$player[$count]</td>";
        echo "<td bgcolor=" . $tdbgcolor[$count%2] . " align=center class=boldblacktext>$player_pos[$count]</td></tr>";

}

Maybe its easier just to fix the original way I tried to write it. Again any help is greatly appreciated.

Jack_McSlay

is this the entire code? if using print_r shows a '1' it means that the $result var is not an array at all, likely a string. try adding this before the while loop

$result = array();

jwer78

Yes that was the entire code. Added what you said and it wokrs perfect now. Much thanks.