エルスと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

実際に文字列が空であるかどうかをチェックしています、結果セットではありませんか？

!empty("any string") 

は常にtrueを返します。

Answer 2

$sql変数には、空であるかどうかをチェックする前に割り当てられた文字列が含まれています。この場合は決して空ではありません。

おそらくあなたがしようとしているのは、クエリから返された結果の数を確認することですか？そのためには、最初にクエリを実行する必要があります。ここではmysqliのと（私の頭の上から）簡単な例です：

$mysqli = new mysqli("localhost", "my_user", "my_password", "my_db"); 

$id = $_GET["id"]; 
$sql = "SELECT * FROM book WHERE id = '".$id."' AND type = 'new'"; 

// Run query 
if ($result = $mysqli->query($sql)) 
{ 
    // We got results 
    var_dump($result); 
    $result->close(); 
} 
else 
{ 
    // No results 
    echo("Oops. Nothing here."); 
} 

SQLインジェクションを回避するために、私はPDOとどのように準備された文の記述方法についての学習をお勧めします。

Answer 3

$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>"; 
} 

それはTUは指定されたタイプ（ここではint）へのパラメータは、SQLインジェクションを防ぐためにGETキャストが重要です！

mysql_num_rowsを使用すると、クエリで返された行の数を確認できます。あなたはそれを確認するためにクエリを行う必要があります。 $sql変数が空であるかどうかを確認することは、文字列で常に「完全」であるため、無駄です。

Answer 4

$sqlは、決して設定した文字列ではありません。

$id=$_GET["id"]; 
$sql="SELECT * FROM book WHERE id = '".intval($id)."' AND type = 'new'"; 
$result = mysql_query($sql); 
if(!empty($result) and 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>"; 
}