problem with joins...

[m00nz00mer]

hey im having some problems with this join...

 

$reviewedByQuery =

"SELECT nbc_user.userName

FROM nbc_user

INNER JOIN nbc_bookreview

ON nbc_user.userID = nbc_bookreview.userID

WHERE nbc_bookreview.bookISBN = '".$_GET['bookID']."'";

 

$reviewedByQueryResult = mysql_query($reviewedByQuery) or die("Query error: ".mysql_error());

$reviewedBy = mysql_result($reviewedByQueryResult, 0);

 

and im getting this error:

 

Warning: mysql_result() [function.mysql-result]: Unable to jump to row 0 on MySQL result index 10 in /var/www/vhosts/numyspace.co.uk/web_users/home/~unn_p185724/public_html/bookinfo.php on line 198

Reviewed by:

 

anyone know why im getting this error? cheers.

 

[revraz]

Check to see if rows were returned before using mysql_result.  Sounds like no rows were returned so Row 0 doesn't exist.

[m00nz00mer]

is there a quick way to do that?

[revraz]

$reviewedByQueryResult = mysql_query($reviewedByQuery) or die("Query error: ".mysql_error());

 

if (mysql_num_rows($reviewedByQueryResult )) > 0 {

$reviewedBy = mysql_result($reviewedByQueryResult, 0);

}

else {

echo "No rows found";

}

 

[craygo]

$reviewedByQueryResult = mysql_query($reviewedByQuery) or die("Query error: ".mysql_error());
$num_rows = mysql_num_rows($reviewedByQueryResult);
if($num_rows > 0){
$reviewedBy = mysql_result($reviewedByQueryResult, 0);
} else {
echo "No rows returned";
}

 

Damn got beat again

 

Ray

 

[m00nz00mer]

ah yes it works!!! just didnt return anything... how can i make it so if there are no returns it displays a message???

[revraz]

Both of our examples have an Else that will display if No Rows were found.

 

ah yes it works!!! just didnt return anything... how can i make it so if there are no returns it displays a message???

[m00nz00mer]

Ok i see what u mean but when i try the following it works when there is data in row0 but when there is no data, i still get an error?

 

$reviewedByQuery =

 

"SELECT nbc_user.userName

FROM nbc_user

INNER JOIN nbc_bookreview

ON nbc_user.userID = nbc_bookreview.userID

WHERE nbc_bookreview.bookISBN = '".$_GET['bookID']."'";

 

$reviewedByQueryResult = mysql_query($reviewedByQuery);

$reviewedBy = mysql_result($reviewedByQueryResult, 0);

 

$reviewedByQueryResult = mysql_query($reviewedByQuery) or die("Query error: ".mysql_error());

$num_rows = mysql_num_rows($reviewedByQueryResult);

 

if($num_rows > 0){

//Display Review Query

echo "Reviewed by: <div class='detailTextGrey'>".$reviewedBy."</div>";

 

} else {

 

echo "No rows returned";

}

[revraz]

Both our examples show your mysql_result inside the IF statement, you are taking it out.  Use the code we gave you above and modify.

[m00nz00mer]

ok i have tried this ... but im not not retreving the correct info... im im now getting Resource id #10 instead of a value..

 

this is the code now...

 

$reviewedByQueryResult = mysql_query($reviewedByQuery) or die("Query error: ".mysql_error());

$num_rows = mysql_num_rows($reviewedByQueryResult);

 

if($num_rows > 0){

//Display Review Query

echo "Reviewed by: <div class='detailTextGrey'>".$reviewedByQueryResult."</div>";

 

} else {

 

echo "No rows returned";

}

 

[revraz]

Now you left out your mysql_result

[m00nz00mer]

ok so i tried this...

 

$reviewedByQueryResult = mysql_query($reviewedByQuery);

$reviewedBy = mysql_result($reviewedByQueryResult, 0);

$num_rows = mysql_num_rows($reviewedByQueryResult);

 

if($num_rows > 0){

//Display Review Query

echo "Reviewed by: <div class='detailTextGrey'>".$reviewedBy."</div>";

 

} else {

echo "No rows returned";

}

 

and it returns the value but it still gives an error when northing is returned?

[revraz]

$reviewedByQueryResult = mysql_query($reviewedByQuery);

$num_rows = mysql_num_rows($reviewedByQueryResult);

       

if($num_rows > 0){

$reviewedBy = mysql_result($reviewedByQueryResult, 0); // HERE

 

//Display Review Query

echo "Reviewed by: <div class='detailTextGrey'>".$reviewedBy."</div>";

       

} else {

echo "No rows returned";

}

 

[m00nz00mer]

thank you, thought i was being stupid!