php - Else and If mysql 和 php 错误

Question

我想在某些数据不存在时返回不可用，但我认为我的代码有问题

$id=$_GET["id"];
$sql="SELECT * FROM book WHERE id = '".$id."' AND type = 'new'";
if(!empty($sql))
{
    $result = mysql_query($sql);
    $row = mysql_fetch_assoc($result); 
    echo utf8_encode($row['bookreview']);
}
else
{
    echo "<div style='text-align: center; background-color: rgb(255, 255,255);'><font style='color: rgb(204, 0, 0);' size='+3'><span style='font-weight: bold;'>review not not available.</span></font><br>";
}

如果评论可用，则返回评论，如果不可用，则“评论不可用”不会回显。

Answer 1

$id  = (int) $_GET['id']; // important !
$sql = 'SELECT * FROM book WHERE id = ' . $id. ' AND type = "new"';
$result = mysql_query($sql);

if(mysql_num_rows($result) > 0) {
    $row = mysql_fetch_assoc($result); 
    echo utf8_encode($row['bookreview']);
} else {
    echo "<div style='text-align: center; background-color: rgb(255, 255,255);'><font style='color: rgb(204, 0, 0);' size='+3'><span style='font-weight: bold;'>review not not available.</span></font><br>";
}

重要的是将 GET 参数转换为指定类型(此处为 int)以避免 SQL 注入(inject)!

使用 mysql_num_rows，您可以检查您的查询返回了多少行。你必须做一个查询来检查它。检查 $sql 变量是否为空是没有用的，因为它只是始终“完整”的字符串 - 它包含您的查询语句。