Im trying to return a record from my table with the following class...
class User {
public function getCredits($uid)
{
$rs = mysql_query("select bank from `users` where id = '$uid'");
$row=@mysql_fetch_object($rs);
return $row->bank;
}
}
I output this data with the following...
<?php print $User->getCredits(); ?>
Im not given anything however, as though my code cannot be read and my broweser just display a white page?
You have:
$rs = mysql_query("select bank from `users` where id = '$uid'");
but call the method with ->getCredits(), so $uid is going to be a null, and your query ends up being
select bank from users where id = ''
Assuming the id field is a standard integer/auto_increment, there will be NO records that have a blank id, so you get back an empty set. So fetch_object will return a boolean false to indicate a fetch failure (no records to fetch), and you blindly try to use that boolean false as your data elsewhere.
You are not passing the function a parameter:
<?php print $User->getCredits(); ?>
Pass it the proper $uid for the query. Also, see @BenjaminGruenbaum comment on deprecated extension.
Try this:
class User {
public function getCredits($uid)
{
$rs = mysql_query("select bank from `users` where id = '".$uid."'");
$row= @mysql_fetch_array($rs);
return $row['bank'];
}
}
@in your code. It's hiding any errors that might occur. My guess ismysql_fetch_objectis throwing an error and you're suppressing it.mysql_*functions. Instead go withmysqli_or PDO!mysql_*functions in new code. They are no longer maintained and are officially deprecated. See the red box? Learn about prepared statements instead, and use PDO or MySQLi - this article will help you decide which. If you choose PDO, here is a good tutorial. People who want this message: gist.github.com/MadaraUchiha/3881905var_dump()of$uid, the actual query string, or of$rowwould have revealed this problem immediately. The first thing you should always do when code is not behaving the way you expect it to is to debug. You should of course also have code to handle all possible edge cases (query failure, query returning 0 rows, etc.).