ログイン画面でのエラー解決

Question

実現したいこと

ブログサイトのログイン画面作成中に起きたエラー解決

前提

PHP Laravelでログイン画面のエラーメッセージの作成中if文のところでエラーが起きてしまった。

発生している問題・エラーメッセージ

Trying to get property 'password' of non-object

該当のソースコード

SessionController.php

<?php

namespace App\Http\Controllers;

use Illuminate\Http\Request;
use Illuminate\Support\Facades\Auth;
use App\User;

class SessionController extends Controller
{
    public function create(){
        return view('session.create', [
          'title' => 'ログイン',
        ]);
    }
    public function store(Request $request){
        $user = Auth::user();
        $id = Auth::id();
        $user = User::where('name', $request->name)->first();
        if($user->password !== $request->password){
            session()->flash('danger', 'ログインに失敗しました。');
            return redirect()->route('session.create');
        }
        session()->put('user_id', $user->id);
        return redirect()->route('posts.index');
    }
    public function destroy(){
        session()->flush();
        return redirect()->route('session.create');
    }
}

試したこと

20行目のif文の（）の中が原因ということはわかっているのですが、どう変えればよいのかわかりません。

補足情報（FW/ツールのバージョンなど）

書き方が間違っているのは自分でも重々承知なのですがこの書き方でやる場合どのようにしたら良いのか教えていただけると幸いです。
よろしくお願いします。

Answer 1

エラーメッセージ「Trying to get property 'password' of non-object」は、オブジェクトが存在しない場合に発生します。
これは、$userオブジェクトが見つからない場合に発生する可能性があります。

User::where('name', $request->name)->first(); この行でユーザーが見つからない場合、$userはnullになります。
この場合、$user->passwordを呼び出すと、エラーが発生します。
ユーザーが見つからなかった場合の処理を追加して、エラーを解決しましょう。

public function store(Request $request){
$user = Auth::user();
$id = Auth::id();
$user = User::where('name', $request->name)->first();

// ユーザーが見つからない場合の処理
if (!$user) {
    session()->flash('danger', 'ユーザーが見つかりません。');
    return redirect()->route('session.create');
}

if($user->password !== $request->password){
    session()->flash('danger', 'ログインに失敗しました。');
    return redirect()->route('session.create');
}
session()->put('user_id', $user->id);
return redirect()->route('posts.index');

}

ただし、上記のコードはパスワードの比較が平文で行われており、セキュリティ上の問題があります。
LaravelではHash::check()を使用して、ハッシュ化されたパスワードと平文パスワードの比較が推奨されています。
そのため、以下のように修正してください。

use Illuminate\Support\Facades\Hash;

// ...

public function store(Request $request){
$user = Auth::user();
$id = Auth::id();
$user = User::where('name', $request->name)->first();

// ユーザーが見つからない場合の処理
if (!$user) {
    session()->flash('danger', 'ユーザーが見つかりません。');
    return redirect()->route('session.create');
}

// パスワードのハッシュをチェック
if (!Hash::check($request->password, $user->password)) {
    session()->flash('danger', 'ログインに失敗しました。');
    return redirect()->route('session.create');
}

session()->put('user_id', $user->id);
return redirect()->route('posts.index');

}

これでエラーが解決し、セキュリティ上の問題も解決されます。

Comments

[退会済みユーザー]

ありがとうございました！