计数行时发生Mysql错误

Question

尝试对表中的行进行计数时出现问题。

变量$ username_me等于会话用户名。他们中没有人是在if或else语句中。

我得到的错误是：

警告：mysql_num_rows（）预计参数1是资源，在/ home/* /的public_html/** /sidebar.php给上线21

布尔

第21行是我粘贴的代码位中的最后一行。

//Count unread oneliners 
$oneliners_sql = "SELECT * FROM oneliners WHERE to_user='$username_me' AND read=0"; 
$oneliners_query = mysql_query($oneliners_sql); 
$oneliners_num = mysql_num_rows($oneliners_query); 

Answer 1

.1。 read是mysql保留字，必须用反引号引用：

$sql = "SELECT * FROM oneliners WHERE to_user='$username_me' AND `read`=0"; 

.2。从不使用SELECT *来计算行数。改用SELECT count（*）。

.3。始终以这种方式运行所有查询以查看发生的任何错误。

//Count unread oneliners 
$sql = "SELECT count(*) FROM oneliners WHERE to_user='$username_me' AND `read`=0"; 
$result = mysql_query($sql) or trigger_error(mysql_error()." ".$sql); 
$row = mysql_fetch_row($result); 
$oneliners_num = $row[0]; 

从来没有使用die()为此目的，尽管所有这些愚蠢的建议。

.4。为这样一个通常的任务创建一个函数。

function getOne($sql) { 
    $result = mysql_query($sql) or trigger_error(mysql_error()." ".$sql); 
    if ($row = mysql_fetch_row($result)) { 
    return $row[0]; 
    } 
} 

那么，你就可以用一条线

$oneliners_num = getOne("SELECT count(*) FROM oneliners WHERE to_user='$username_me' AND `read`=0"); 

Answer 2

mysql_query返回假 instread MySQL的结果。所以，你的查询或连接有一些错误。您可以读取错误与mysql_error

Answer 3

• 你的代码是脆弱的SQL Injection。还有一个xkcd strip的说法:-)

如果magic_quotes_gpc的未启用（他们不应该，魔术引号是PHP的的许多设计错误之一，他们现在不建议使用），黑客（或最有可能的机器人！）可以自由改变您的查询造成您的网站的各种麻烦。

解决这一严重问题的正确方法是使用Prepared Statements, for instance with PDO。或者您可以使用mysql_real_escape_string：

$sql = "SELECT COUNT(*) FROM oneliners WHERE to_user='" . 
    mysql_real_escape_string($username_me) . "' AND `read`=0"; 

与mysql_real_escape_string麻烦的是，你已经记住经常使用它，而有准备的语句，这些问题都照顾你。另外，如果你不使用预处理语句，你可以使用intval来清理你的整数输入。

让我再次强调一下：消毒您的输入非常重要。你不应该信任来自用户的任何信息。

• 您的问题通过的mysql_query时查询失败返回FALSE造成的。如果查询失败，mysql_query返回FALSE而不是资源。您可以在资源上调用mysql_num_rows，而不是在FALSE上。 （从手册：mysql_query（）返回成功的资源，或错误的FALSE）。

• 该错误是由read字段名称引起的，因为是reserved word。尝试引用反引号（`）或甚至更好，重命名它。

• 要找出一个行是否存在与否可以用COUNT(*)或SELECT 1

随着COUNT查询将总是返回0或1的值（我想你已经一个唯一索引to_user），当然查询失败除外。

$sql = "SELECT COUNT(*) FROM oneliners WHERE to_user='" . 
    mysql_real_escape_string($username_me) . "' AND `read`=0"; 

随着SELECT 1如果行存在的查询将返回1，否则将不返回任何一行。

$sql = "SELECT 1 FROM oneliners WHERE to_user='" . 
    mysql_real_escape_string($username_me) . "' AND `read`=0 LIMIT 1"; 

要使用哪一个取决于您是否需要计算行数或者您只需要知道行是否存在。如果有独特的索引，它们在计算上应该非常相似。