PHP add array to an object

Question

I am trying to query the database to:

Get a football match record
Get all the players that are part of that match

There are three tables: Matches | Players | Match_Players

Match_Players simply connects the two other tables together.

I am trying to return the result in JSON.

The problem is with the second query. It returns an Errant Query. This is what I have so far:

$matches = array();
if(mysql_num_rows($matchResult)) 
{
    while($match = mysql_fetch_assoc($matchResult)) 
    {
        $players_query = 
            "SELECT p.* FROM match_players mp 
            LEFT JOIN players p on p.id = mp.player_id
            WHERE mp.id = ".$match->player_id; // <--- This is the error

        $playersResult = mysql_query($players_query,$link) or die('Errant query: '.$players_query);
        $players       = array();

        if(mysql_num_rows($playersResult)) 
        {
            while($player = mysql_fetch_assoc($playersResult)) 
            {
                $players[] = $player;
            }
        }

        $match->$players = $players;    
        $matches[] = $match;
    }
}

Update

I have changed the second SQL statement by replacing $match->player_id to $match['player_id'];. However, I am not receiving the collection of players. All I can see is the match details. Any idea why?

Steve · Accepted Answer · 2014-04-30 22:17:24Z

2

Use

$match['player_id']

(array syntax) not

$match->player_id

(object syntax) or use

mysql_fetch_object

Though i expect you should do one query, not a loop

answered Apr 30, 2014 at 22:17

Steve

20.5k5 gold badges47 silver badges71 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

Subby Over a year ago

Please check the update. Though the error has gone, I can't see a returned result of the players.

Steve Over a year ago

@Subby same problem, change $match->$players = $players; to $match['players'] = $players;

furier · Accepted Answer · 2014-04-30 23:30:00Z

1

Shouldn't the query be like this?

"SELECT p.* 
FROM match_players mp 
RIGHT JOIN players p on p.id = mp.player_id
WHERE mp.match_id = ".$match->id;

or like this

"SELECT p.*
FROM players p, match_players mp, matches m
WHERE p.id = mp.player_id AND mp.match_id = m.id
AND m.id = ". $match->id;

created an sqlfiddle to try and check the queries and they are fine...

edited Apr 30, 2014 at 23:30

answered Apr 30, 2014 at 22:23

furier

1,9771 gold badge22 silver badges39 bronze badges

7 Comments

Subby Over a year ago

I just noticed that too JUST before you posted :) I still don't see the players coming through. If I run this code through PHPMYAdmin, the results come through A-OK.

Subby Over a year ago

It throws an errant query when I paste your update in.

furier Over a year ago

Ah, you made a left join which will give you only match_players stuff back... you need to do a right join

Subby Over a year ago

Errant query: SELECT p.* FROM match_players mp RIGHT JOIN players p on p.id = mp.player_id WHERE mp.match_id =

furier Over a year ago

this is probably a stupid question but you are putting an id after the = ? :P e.g. "SELECT p.* FROM match_players mp RIGHT JOIN players p on p.id = mp.player_id WHERE mp.match_id = 1"

|

Collectives™ on Stack Overflow

PHP add array to an object

2 Answers 2

2 Comments

7 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

2 Comments

7 Comments

Your Answer

Sign up or log in

Post as a guest

Related