foreach loop from mysql query

mysql_fetch_array just returns one row of the results, not all the rows. Your foreach loop is just looping over the columns in the first row of results.

If you want to process all the results, you should write:

while ($rc = mysql_fetch_array($results_course))

Then inside your loop, you use $rc['course_num'] to get the course from that row.

But I don't understand why you need that first loop at all. You're JOINing the hc_courses and hc_entries tables in the first query inside the loop. Why don't you just use that same query, but without the c.course_num = $rc condition, so it gets all courses at once instead of doing them one course at a time? Loop over those results, and start a new table every time the course number changes.

Here's the query for that:

SELECT c.course_name as course,
       e.distance as distance,
       e.score as score,
       e.time as time,
       e.user as User,
       c.course_num as course_num
FROM hc_entries e
JOIN hc_course c ON e.course=c.course_num
WHERE e.vehicle=$varVeh
ORDER BY course, score DESC

The PHP then looks like:

$last_course = null;

while ($row = mysql_fetch_assoc($results_HiScores) {
    if ($row['course_num'] !== $last_course) {
        // New course number, start a new table
        if ($last_course !== null) {
            // Close out last table, if any
            echo '</table>';
        }
        $last_course = $row['course_num'];
        echo "<table><tr><th>Best Scores for ".$vehName['Veh_name']."</th> </tr></table>";
        echo "<table border='1'>";
        echo "<tr><th>Course</th><th>Score</th><th>Distance</th><th>Player</th><th>Time</th></tr>";
    }
    echo "<tr>";  
    echo "<td>" .$row['course'] . "</td>";
    echo "<td>" .$row['score'] . "</td>";
    echo "<td>" .$row['distance'] . "</td>";
    echo "<td>" .$row['User'] . "</td>";
    echo "</tr>";
}
if ($last_course !== null) {
    echo "</table>";
}

And you shouldn't do the hc_vehicle_type query inside the loop. It doesn't use any variables that change during the loop, it's just looking up the name of $_POST['Veh_num']. Just do it once and reuse the result inside the loop.