PHP数组或SQL错误遍历foreach循环

Question

I have what I suspect is either an array error or a mysql query error. I have printed all the variables in the function, they return the correct result, and tried the same query in phpmyadmim which returns the correct result.

The use case is pretty simple

1. Get $_GET['gameID'] and query DB
2. Get returned result from 1st query and query DB for a 2nd time with week number and sport.
3. return an array with all results form second query

PROBLEM

The current function is only returning the last row of the table instead of returning all rows.

Picture of table

Output result

Code

function displayTeamsByGameID($gameID)
{
    global $db;
    //now prevents late submissions
    $sql = "SELECT * FROM bru_schedule WHERE gameID = :gameID";
    $stmnt = $db->prepare($sql);
    $stmnt->bindValue(':gameID', $gameID);
    $stmnt->execute();
    if ($stmnt->rowCount() > 0) {
        $games =  $stmnt->fetch();
        print $weekNum = $games['weekNum'];
        print $sport = $games['sport'];
    $sql = "SELECT * FROM bru_schedule WHERE weekNum = :weekNum AND sport = :sport";
    $stmnt= $db->prepare($sql);
    $stmnt->bindValue(':weekNum', $weekNum);
    $stmnt->bindValue(':sport', $sport);
    $stmnt->execute();
    if($stmnt->rowCount() > 0){
     $matches= $stmnt->fetchAll();
        foreach ($matches as $match) {
             $match[] = array('gameID' => $match['gameID'], 'weekNum' => $match['weekNum'], 'gameTimeEastern' => $match['gameTimeEastern'], 'homeID' => $match['homeID'], 'visitorID' => $match['visitorID'], 'sport' => $match['sport'], 'venue' => $match['venue'], 'tournament' => $match['tournament'], 'spread' => $match['spread']);

                 }//foreach
            }//statment weekNUm Sport
    }//statment gameID
        if (!isset($match)) {
            return false;
        }
        return $match;
}//function

If anyone could give my code a quick scan and or provide some advice it would much appreciated.

Answer 1

You are overwriting the $match variable since you have used the same variable in the foreach statement and within the foreach body for creating the result array. Use a different variable. See the code below. I have used a new variable $result for the result.

function displayTeamsByGameID($gameID)
{
    global $db;
    //now prevents late submissions
    $sql = "SELECT * FROM bru_schedule WHERE gameID = :gameID";
    $stmnt = $db->prepare($sql);
    $stmnt->bindValue(':gameID', $gameID);
    $stmnt->execute();
    if ($stmnt->rowCount() > 0) {
        $result = array();
        $games =  $stmnt->fetch();
        print $weekNum = $games['weekNum'];
        print $sport = $games['sport'];
    $sql = "SELECT * FROM bru_schedule WHERE weekNum = :weekNum AND sport = :sport";
    $stmnt= $db->prepare($sql);
    $stmnt->bindValue(':weekNum', $weekNum);
    $stmnt->bindValue(':sport', $sport);
    $stmnt->execute();
    if($stmnt->rowCount() > 0){
     $matches= $stmnt->fetchAll();
        foreach ($matches as $match) {
             $result[] = array('gameID' => $match['gameID'], 'weekNum' => $match['weekNum'], 'gameTimeEastern' => $match['gameTimeEastern'], 'homeID' => $match['homeID'], 'visitorID' => $match['visitorID'], 'sport' => $match['sport'], 'venue' => $match['venue'], 'tournament' => $match['tournament'], 'spread' => $match['spread']);

                 }//foreach
            }//statment weekNUm Sport
    }//statment gameID
        if (!isset($result)) {
            return false;
        }
        return $result;
}//function

Answer 2

function displayTeamsByGameID($gameID)
{
    global $db;
    //now prevents late submissions
    $sql = "SELECT * FROM bru_schedule WHERE gameID = :gameID";
    $stmnt = $db->prepare($sql);
    $stmnt->bindValue(':gameID', $gameID);
    $stmnt->execute();
    if ($stmnt->rowCount() > 0) {
        $games =  $stmnt->fetch();
        print $weekNum = $games['weekNum'];
        print $sport = $games['sport'];
    $sql = "SELECT * FROM bru_schedule WHERE weekNum = :weekNum AND sport = :sport";
    $stmnt= $db->prepare($sql);
    $stmnt->bindValue(':weekNum', $weekNum);
    $stmnt->bindValue(':sport', $sport);
    $stmnt->execute();
    if($stmnt->rowCount() > 0){
     $matches= $stmnt->fetchAll();
        foreach ($matches as $match) {
             // 【don't reassign value to $match, it's confused! Use an new variable name】
             $ret[] = array('gameID' => $match['gameID'], 'weekNum' => $match['weekNum'], 'gameTimeEastern' => $match['gameTimeEastern'], 'homeID' => $match['homeID'], 'visitorID' => $match['visitorID'], 'sport' => $match['sport'], 'venue' => $match['venue'], 'tournament' => $match['tournament'], 'spread' => $match['spread']);

                 }//foreach
            }//statment weekNUm Sport
    }//statment gameID
        if (!isset($ret)) {
            return false;
        }
        return $ret;
}//function