テーブル内のNUMBER行が検出されない

Question

私は現在、私のコースの一環としてニュースサイトのログインシステムを作っています。何らかの理由でif文で$ rows-> num_rows == 1を使用すると、常に "else"コードが実行されます。基本的には、これは、入力されている正しいユーザー情報に対応するテーブルの行を検出できないことを意味します。 HTMLコードに情報が入力されたときに実行されるPHPコードを次に示します。

<?php 
error_reporting(E_ALL); 
ini_set('display_errors', 1); 

//Connect to DB 
    include_once("db_connect.php")or die ("Couldnt connect to DB"); 
$username = $_POST['user']; 
$password = $_POST['password']; 

session_start(); 

if(trim($username) != '' and trim($password) != ''){ 

//Sanitizes whatever is entered 
    $username=stripslashes($username); 
    $password=stripslashes($password); 

    $username=strip_tags($_POST['user']); 
    $password=strip_tags($_POST['password']); 

    $username=mysqli_real_escape_string($conn,$username); 
    $password=mysqli_real_escape_string($conn,$password); 

//Checks whether Username exists   
$query = mysqli_query($conn, "SELECT * FROM user WHERE users='$username' 
AND password = '$password' ") 
or die(mysqli_error($conn)); 

$numrows=mysqli_num_rows($query); 

if($numrows > 0){ 

// echo "Record exists."; 

$_SESSION['login_user']=$username; // Initializing Session 

header("location: index.php"); // Redirecting To Other Page 
exit; 
} 

else { 
    echo "Username or password is incorrect."; 
} 
}else{ 
    echo "Please enter information"; 
} 
?> 

最後のif文で行が検出されないので問題が発生します。そして、はい、私のテーブルは1行のユーザー情報（ユーザー、パスワード）で埋められ、HTMLフォームもPOSTを使用します。

私はこの問題を少なくとも3時間は調査したが、まだ解決策を見つけることはできません。ここで

現在のエラーログです：

Warning: include_once(1): failed to open stream: No such file or directory in /home/vol9_7/byethost4.com/b4_18083024/htdocs/loginAuth.php on line 6 

Warning: include_once(): Failed opening '1' for inclusion 
(include_path='.:/usr/share/pear/') in  
/home/vol9_7/byethost4.com/b4_18083024/htdocs/loginAuth.php on line 6 

Notice: Undefined variable: conn in  
/home/vol9_7/byethost4.com/b4_18083024/htdocs/loginAuth.php on line 22 

Warning: mysqli_real_escape_string() expects parameter 1 to be mysqli, null  
given in /home/vol9_7/byethost4.com/b4_18083024/htdocs/loginAuth.php on line 
22 

Notice: Undefined variable: conn in 
/home/vol9_7/byethost4.com/b4_18083024/htdocs/loginAuth.php on line 23 

Warning: mysqli_real_escape_string() expects parameter 1 to be mysqli, null 
given in /home/vol9_7/byethost4.com/b4_18083024/htdocs/loginAuth.php on line 
23 

Notice: Undefined variable: conn in 
/home/vol9_7/byethost4.com/b4_18083024/htdocs/loginAuth.php on line 42 

Warning: mysqli_query() expects parameter 1 to be mysqli, null given in  
/home/vol9_7/byethost4.com/b4_18083024/htdocs/loginAuth.php on line 42 

Notice: Undefined variable: conn in 
/home/vol9_7/byethost4.com/b4_18083024/htdocs/loginAuth.php on line 43 

Warning: mysqli_error() expects parameter 1 to be mysqli, null given in 
/home/vol9_7/byethost4.com/b4_18083024/htdocs/loginAuth.php on line 43 

EDIT：フレッド-ii-答え​​を使用します。 include_once("db_connect.php")or die ("Couldnt connect to DB");がコードの先頭に移動しました。

第2に、新しいバージョンのif文が追加されて古いバージョンに置き換えられました。このステートメントはFred -ii- answerでも見ることができます。私は、テーブルと列 名前を混同しているので

第三に、SQLステートメントが修正されました。

最後に、error_reporting(E_ALL); ini_set('display_errors', 1); はフレッド-ii-答え​​の礼儀再び、エラーを見つけるのを助けるために追加されました。

Answer 1

以下の回答は、元の投稿https://stackoverflow.com/revisions/37284594/1に記載されています。編集としてマークしないで、コードの先頭にインクルードを2回移動し、追加編集としてマークしません。

---

あなたが

include_once("db_connect.php")or die ("Couldnt connect to DB"); 

それはあなたがmysqli_real_escape_string()である、DB接続を必要とする任意の関数を呼び出す前に配置する必要があり、ここで馬の前に送料を入れています。

データベースが大きくなるにつれ、同じユーザー名を持つ複数の人がいる場合は、if ($rows->num_rows >= 1)またはif ($rows->num_rows > 0)も使用する必要があります。有りうる。実際、私は昨日そのようなことを試していました。

また、各ヘッダーの後にexit;を使用してください。そうしないと、コードを実行し続けることができます。

また、クエリに対するエラーもチェックする必要があります。あなたはそれをやっていない。

これでも動作しない場合は、POST配列に対して使用している機能の一部に悪影響があり、有効な文字を取り除く可能性があります。削除する必要があるかもしれません。

準備されたステートメントを使用すると、これらのステートメントからすべて削除されます。

• http://www.php.net/manual/en/mysqli.quickstart.prepared-statements.php

エラーチェック（クエリが失敗しました）。

http://php.net/manual/en/mysqli.error.phpおよびhttp://php.net/manual/en/function.error-reporting.php のリンクを参照し、コードに適用してください。

---

パスワード

私はまた、あなたがプレーンテキストでパスワードを保存することができることに気づきました。これはお勧めしません。

使用次のいずれか：

• PHP 5.5のpassword_hash()機能。
• 互換パック（もしPHP < 5.5）カラムの長さについてhttps://github.com/ircmaxell/password_compat/

重要追記：

もし、あなたがpassword_hash()または互換機能パックを使用することを決定しないときhttps://github.com/ircmaxell/password_compat/（PHP < 5.5の場合）現在のパスワード列の長さが60未満の場合は、それ以上（またはそれ以上）に変更する必要があることに注意することが重要です。このマニュアルでは255の長さが推奨されています。

有効にするには、新しいハッシュを使用して列の長さを変更し、最初からやり直す必要があります。そうしないと、MySQLは自動的に失敗します。関心の

その他のリンク：

• CRYPT_BLOWFISH
• crypt()
• bcrypt()
• scrypt()
• On OPENWALL
• PBKDF2
• PBKDF2 on PHP.net
• PBKDF2 For PHP

---

編集：

変更し、このブロック：（あなたのメソッドは、$rows->num_rowsのために失敗することができる）

$query="SELECT * FROM user WHERE users='$username' AND password = '$password'"; 
$rows = mysqli_query($conn, $query); 

if ($rows->num_rows == 1){ 

    $_SESSION['login_user']=$username; // Initializing Session 

    header("location: index.php"); // Redirecting To Other Page 
} 

へ：

$query = mysqli_query($conn, "SELECT * FROM user WHERE users='$username' AND password = '$password' ") 
or die(mysqli_error($conn)) 
; 

$numrows=mysqli_num_rows($query); 

if($numrows > 0){ 

// echo "Record exists."; 

    $_SESSION['login_user']=$username; // Initializing Session 

    header("location: index.php"); // Redirecting To Other Page 
    exit; 
} 

し、ファイルの先頭にこれを配置します。

<?php 
error_reporting(E_ALL); 
ini_set('display_errors', 1); 

// rest of your code 

---

NOTA：

私はSELECT * FROM user WHERE users

あなたは右を選んだことを確認し、このかかわらずに質問していますあなたはそれらを偶然逆算していないことを確認してください。

Answer 2

私はこの

使用あなたに別のコードを示唆：これに代えて

if ($result = mysqli_fetch_array($rows,MYSQLI_NUM)){ 

：

if ($rows->num_rows == 1){