php ログイン認証

Question

phpのログイン認証でてこづっていますので、ご教授いただきたいです。

$_POST["id"]をsetterに送り、

$pdo = $db->access();でデータベースに接続し、

$db::logining($pdo);でログイン認証をしています。

$db::logining($pdo);では、sqlを送り返ってきた結果が、
$_POST["id"];と同じであれば、ログイン成功、異なれば
loginForm.phpにページ移動するような仕様です。

今回はわかりやすくするため、IDだけのチェックにしています。

---login.php---

if($_POST != null){
    require_once 'logfor.php';
    require_once 'db.php';

    $log = new login();
    $log->setId($_POST["id"]);　---IDをpostで受取りセット
    $db = new db();
    $pdo = $db->access();　---Mysqlに接続
   $db::logining($pdo);　---メソッドの呼び出し
echo $arr;

}

---db.php

 function logining($pdo){

        $login = new login();
        $id = $login->getId();

        $sql = "SELECT * FROM MUSER WHERE ID = '".$id."'";
        $result = $pdo->query($sql);  ---結果をもらう
      // $arr = $result->fetch(PDO::FETCH_ASSOC);

        if((array)$result[0] == $id){　　
            ---帰ってきた結果がセットした$idと同じならページ移動
            ---コメントアウトを外した際は$resultを$arrに変更しています。
        header("Location:ログイン成功.php");
        exit();
        }else{
            header("Location:loginForm.php");
            exit();
        }

$id = $login->getId();にはちゃんと値が入っており、データベース接続も成功しています。
しかし、最後のif文が機能しておらず、データベースにない値でも、ログイン成功.phpに入ってしまいます。

また、コメントアウトを外した際は

Fatal error: Uncaught Error: Call to a member function fetch() on boolean in C:\コメントアウトの部分 Stack trace: #0 C:\loginForm.php(14): db::logining(Object(PDO)) #1 {main} thrown in C:\コメントアウトの部分

というエラー文が出力されます。

よろしくお願いします。

Answer 1

質問文の内容に間違いがないという前提 で、解説します。

まずエラーメッセージを読むと

Fatal error: Uncaught Error: Call to a member function fetch() on boolean 
in C:\コメントアウトの部分 Stack trace: 
#0 C:\loginForm.php(14): db::logining(Object(PDO)) 
#1 {main} thrown in C:\コメントアウトの部分

Call to a member function fetch() on boolean

の部分から、$arr = $result->fetch(PDO::FETCH_ASSOC); にエラーがあることがわかる。

boolean で fetch()を呼んでいますよってエラーなので、

$result = $pdo->query($sql); 
$arr = $result->fetch(PDO::FETCH_ASSOC);

$result が trueまたはfalse であることがわかる。

リファレンス（http://php.net/manual/ja/pdo.query.php）によると

PDO::query — SQL ステートメントを実行し、結果セットを PDOStatement オブジェクトとして返す

なので $result は PDOStatement オブジェクト であるはずですね。にも関わらず、$resultがboolean であるというのは、おかしいですね。
ということは、$pdo->query($sql) が正しい値を返していないということがわかります。

となると、query() が boolean を返す？
そんなことがあるのでしょうか？

リファレンスによると、

PDO::query() は、PDOStatement オブジェクトを返します。 失敗した場合は FALSE を返します。

と書かれていなす。じゃあ、$result は false なんだろうね。

ということは、$pdo って変数に正しく値はセットされてるの？と疑うことになる。
$pdo って変数はどこから来ているんだろ？
(（心の声）てゆーか、「データベース接続も成功」って書いているのは、単なる「思い込み」で嘘情報だろうな…)

function logining($pdo){

ああ、ここか。
ん？ $pdo には正しく値がセットされてる？
あ、コードが提示されてない…

詰んだ…

ちゃんとコード提示してね、となる。

Comments

[退会済みユーザー]

いつものやつ、また勝手に紹介しときましたｗ
フォローお願いします＾＾

[退会済みユーザー]

質問者にはオーバースペックだと思うわw

[rockstar]

Javaだとサクッと行けたのでPHPでも余裕だと調子乗ってました笑
今出先きてしまったので、明日の朝までにできなければデータベース接続貼らせていただきます。
申し訳ないです。

[rockstar]

te2jiさん
こういう紹介ありがたいです！
フォローさせていただきました！

[退会済みユーザー]

あの記事書いたの、Kosuke_Shibuya さんなんで、不明点あればこちらで質問どうぞｗ
ログイン認証に付随すべき機能（会員管理、アカウントロック、メール送信、ログ）を網羅的に解説してる記事ってあまりないんで、かなり実践的です。
初心者の頃、コード理解できなくても読み物として面白かったです。

[rockstar]

te2jiさん
ぜひ参考にさせてもらいますね！

[rockstar]

昨日の件、添付しましたので、ご確認いただければと思います。

Answer 2

認証スクリプトとして必要なレベルに至っていません。PHPでログイン機能を実装するチュートリアルあたりで、必要な処理を確認してください

間違い箇所は、多分以下

$result[0] == $id
→
$arr[0] === $id

Comments

[rockstar]

te2jiさん　ご回答ありがとうございます。
変更しましたが、変化ありませんでした。

[退会済みユーザー]

それはそうでしょうね＾＾；
$arr の中身、ちゃんと見てみるといいですよ。

[rockstar]

return で戻し、foreachでechoしてみましたが、何も表示されませんでした。。ヒントいただけませんか？

[退会済みユーザー]

たまたまidが整数値としてPOSTすれば通っちゃいそうな雰囲気があって、パスワード認証とかないのかなってモニョる。

[退会済みユーザー]

変数の中身の確認方法って、学習してます？
個人的には、var_dump() より、var_export() のほうが好き。

（余談）
処理を理解すると、今やろうとしている if 条件があまりきれいでないと気がつくと思います。
値が取れていれば、else には絶対行かない処理になっています。

[rockstar]

te2jiさん
ヒントいただいてましたね、確認してみます！

[rockstar]

m6uさん
パスワードの方はわかりやすくするために省いています。idさえできれば付け足すのもできるかなと笑

Answer 3

無事ログイン認証できました。下記は成功したコードです。

---loginForm.php---

if($_POST != null){
    require_once 'logfor.php';
    require_once 'db.php';

    $log = new login();
    $log->setId($_POST["id"]);
    $log->setPass($_POST["pass"]);---POSTで受け取った値をgetterに入れます。
    $db = new db();
    $db::logining($log);　---ログイン認証のメソッドを実行します。（下記参照）
}

 function logining($log){

        $db1 = self::access(); ---同じクラス内のデータベース接続メソッド（下記にあります。）

        $id = $log->getId();
        $pass = $log->getPass();
        $sql = "select * from muser where user_id = '".$id."' AND user_pass = '".$pass."'";　　---setterで値を受け取りsql文作成
        $result = $db1->query($sql);---結果をオブジェクトでもらい
        $arr = $result->fetch(PDO::FETCH_ASSOC);　---配列に変換？

        if($arr["USER_ID"] === $id AND $arr["USER_PASS"] === $pass){
            header("Location:main.php");
            exit();
        }else{
            header("Location:loginForm.php");
            exit();
        }

function access()
    {
        $pdo = new PDO('mysql:host=localhost;dbname=locshop;charset=utf8', 'root', '', [
            PDO::ERRMODE_EXCEPTION
        ]);

        if (! $pdo) {
            echo "データベース接続失敗" . PHP_EOL;
            echo "errno: " . mysqli_connect_errno() . PHP_EOL;
            echo "error: " . mysqli_connect_error() . PHP_EOL;
            exit();
        }

        echo 'データベース接続成功';
        $pdo->query('SET NAMES utf8');
        return $pdo;
    }

原因はlogin.phpとdb.phpで二度インスタンス生成しているため、db.phpで値を受け取ろうとしてもうまく受け取れず、SQL文がおかしくなりqueryの結果がbooleanになるのだとわかりました。（受け取った値はNULLになっていました。）

if文ですが、var_dumpしてみたらなぜだめなのかすぐにわかりました。

力を貸してくれた皆様に感謝しています。ありがとうございました。

Comments

[退会済みユーザー]

$sql = "select * from muser where user_id = '".$id."' AND user_pass = '".$pass."'";
SQLインジェクション脆弱性があります。

mysqli_connect_errno() でエラーは取れません。

$pdo->query('SET NAMES utf8'); いらない。

Answer 4

ミスです。
ほかの回答をご覧ください。

Answer 5

ミスです。
ほかの回答をご覧ください。