I have the following function in a class :

<?php function RaceName()    { //Return the User Name   $id = $this->Id;   //Fetch the user race Id   $sql = "SELECT Race_Id FROM User WHERE Id=$id";   $result = $this->db->query($sql);   $row = $result->fetch();   $raceid = $row['Race_Id'];   //Fetch the race Name   $sql = "SELECT Name FROM Races WHERE Id=$raceid";   $result = $this->db->query($sql);   $row = $result->fetch();   return $row['Name'];  } ?>

I'm sure they is a way to have only one SQL statement, can anyone give me a hand?

This is what I tried, but it didn't work :

<?php $sql = "SELECT Name FROM Races, User WHERE User.Id=$id AND Id=User.Race_Id"; ?>

Any sugestion?

____________________________
A Stick give a wise man something to think about... and a fool, something to put in is mouth.

<?php $sql="SELECT Name FROM Races INNER JOIN User ON Races.Id=User.Race_Id WHERE User.Id=$id"; ?>

or

<?php $sql = "SELECT Name FROM Races, User WHERE User.Id=$id AND Races.Id=User.Race_Id"; ?>

maybe you only forgot to use Races.Id since the Id column appears in all two tables it's better to specify what table it belongs and also if Name is a field of all two tables use the Tabel.Field syntax

anyway the first syntax should be better and faster

____________________________
AndreaVB