2016-02-16 100 views
-4

我正在檢查以確認memberid和recipeid都不爲空。這樣,如果兩者都具有價值,它將允許成員將成熟的廣告添加到列表中,如果兩者都有值,則會告訴他們他們添加的配方已經在列表中。在此工作一段時間需要幫助,並嘗試從asp經典轉換到php閃爍的微笑。需要查找兩條記錄中的一條是否爲空

$check_fav = mysql_query("SELECT memberid, recipeid FROM favorites WHERE (memberid = '".$memberid."' AND recipeid = '".$id."')"); 

If ($check_fav == TRUE){....... 
+1

[您的腳本存在SQL注入攻擊風險。](http://stackoverflow.com/questions/60174/how-can-i-prevent-sql-injection-in-php) –

+1

請[停止使用' mysql_ *'functions](http://stackoverflow.com/questions/12859942/why-shouldnt-i-use-mysql-functions-in-php)。 [這些擴展](http://php.net/manual/en/migration70.removed-exts-sapis.php)已在PHP 7中刪除。瞭解[編寫]​​(http://en.wikipedia.org/ wiki/Prepared_statement)語句[PDO](http://php.net/manual/en/pdo.prepared-statements.php)和[MySQLi](http://php.net/manual/en/mysqli.quickstart .prepared-statements.php)並考慮使用PDO,[這真的很簡單](http://jayblanchard.net/demystifying_php_pdo.html)。 –

+1

喜歡「微笑的笑容」。 –

回答

1

mysql_query()永遠不會返回true。如果發生錯誤,它將返回結果對象或false

那麼這就是你打算做測試:

if ($check_fav !== false && mysql_fetch_row($check_fav) !== false) { ... 

但在這之前,請移動到mysqli的功能(或對象符號),而不是mysql_,因爲後者已被棄用很長一段時間,並且不再支持PHP7。你也容易受到SQL注入的影響。改爲使用prepared statements

+0

我的主人trincot你是一個拯救生命的人,謝謝! – askChuck

+0

10-4會做謝謝 – askChuck

相關問題