【Laravel】退会していないのに「このユーザーは退会しました」と出てしまう、投稿ページにアクセスできない

Question

会員登録したユーザーが記事を投稿し、そのユーザーが退会を行った場合、記事の詳細ページに「このユーザーは退会しました」と表示させる仕組みにしてあるつもりなのですが・・・。

UsersController@show

public function show($id)
{
$user = User::find($id);
$lectures = $user->lectures()->orderBy('created_at', 'desc')->paginate(4);

    ~~$data = [~~
        ~~'user' => $user,~~
        ~~'lectures' => $lectures,~~
    ~~];        return view('users.show', $data);~~
~~}~~

当初、上記のUsersControllerを載せていましたが、今回の対象はLecturesControllerでした、申し訳ありません。

LecturesController@show

public function show($id) {

        $lecture = Lecture::find($id);
        $user = User::find($id);

        return view('lectures.show', [
            'lecture' => $lecture,
            'user' => $user,
        ]);
    }

lectures/show.blade.php

@extends('layouts.app')

@section('content')

<div class="container">
    <div class="head">レクチャー詳細</div>

        <div class="row ml-3 mt-5 pt-4">

            <div class="col-9 mx-auto">
                <article class="cont_inner">
                @if ($lecture)
                    @if ($lecture->image)
                    <figure class="show_post_thumbnail"><div class="new_area-img text-center"><img class="show_image" src="{{ asset('storage/'.$lecture->image) }}"></div></figure>
                    @else
                    <figure class="show_post_thumbnail"><div class="new_area-img text-center"><img src="../img/noimage.gif"></div></figure>
                    @endif
                    <section class="post_meta-all multiline line-8">
                <div class="pick_title mb-2">{{ $lecture->title }}</div>
                <time>{{ $lecture->updated_at }}</time>
                <div>
                @if ($lecture->category_name == 'ピッチング')
                    <span class="tag-pitching">{{ $lecture->category_name }}</span>
                @elseif ($lecture->category_name == 'バッティング')
                    <span class="tag-batting">{{ $lecture->category_name }}</span>
                @elseif ($lecture->category_name == '守備')
                    <span class="tag-fielding">{{ $lecture->category_name }}</span>
                @elseif ($lecture->category_name == '走塁')
                    <span class="tag-running">{{ $lecture->category_name }}</span>
                @elseif ($lecture->category_name == 'メンタル')
                    <span class="tag-mental">{{ $lecture->category_name }}</span>
                @elseif ($lecture->category_name == 'その他')
                    <span class="tag-other">{{ $lecture->category_name }}</span>
                @endif
                @if ($user === null)    
                <span class="d-inline-block ml-2 small text-info">投稿者：このユーザーは退会しました</span></div>
                @else
                <span class="d-inline-block ml-2 small text-info">投稿者：<a href="{{ route('users.show', ['id' => $user->id]) }}">{{ $user->name }}</a></span></div>
                @endif
                <div class="mt-3 mb-5">{{ $lecture->content }}</div>
                </section>
                @endif
                </article>
                @if (Auth::id() === $lecture->user_id)
                    <div class="text-center mt-4">
                            {!! Form::open(['route' => ['lectures.destroy', $lecture->id], 'method' => 'delete']) !!}
                                {!! Form::submit('削除する', ['class' => 'btn btn-danger btn-sm del_check']) !!}
                            {!! Form::close() !!}
                    </div>
                @endif
            </div>
@endsection

それと、「投稿する」のページにアクセスすると、エラーが出るようになりました。

「このユーザーは退会しました」と出るものを修正しようと試みる前は問題なくアクセスできていたので、何かの拍子にそうなってしまったのかと思うのですが、ルーティングやコントローラーのcreate部分は特に触っていないですし、一応確認してみても、個人的には何らおかしいところは見受けられませんでした。

---

web.php

// ユーザ機能
Route::group(['middleware' => ['auth']], function () {
    Route::resource('users', 'UsersController', ['only' => ['show']]);
    Route::get('lectures/create', 'LecturesController@create')->name('lectures.create');
    Route::post('lectures', 'LecturesController@store')->name('lectures.store');
    Route::delete('lectures/{id}', 'LecturesController@destroy')->name('lectures.destroy');
    Route::delete('users/{id}', 'UsersController@destroy')->name('users.destroy');
});

---

LecturesController@create

public function create() {
        
        $lecture = new Lecture;

        return view('lectures.create', [
            'lecture' => $lecture,
        ]);
    }

---

また、エラー内容を訳すと「非オブジェクトのプロパティ「user_id」を取得しようとしています」とのことで、tinkerにてあらためてLectureデータのカラムを確認しても、「user_id」は存在しています。

いろいろとすみません、よろしくお願いいたしますm(_ _)m

Comments

[yuuyu]

1.showが呼ばれているかの確認はしていますか？
2.User::findで該当する$idは存在していますか？
提示されているコードでは、UsersControllerがincludeされているかすら確認できないので、一度確認してみてください。

[Keight]

ご回答ありがとうございます。

＞ showが呼ばれているかの確認はしていますか？
これは、「web.php」の中の記述ということでしょうか？

＞ User::findで該当する$idは存在していますか？
存在していますね。
データを確認してみたところ、現在ログイン中のユーザー（自分自身）のidが 4 となっていて、

投稿者：<a href="{{ route('users.show', ['id' => $user->id]) }}">

この部分で直接指定しているのですが、なぜか「このユーザーは退会しました」と出てしまいます(・_・;)

[yuuyu]

＞これは、「web.php」の中の記述ということでしょうか？
質問文に「web.php」が一切出てきてないのですが、「lectures/show.blade.php」が実行されている範囲内、ですね。

＞この部分で直接指定しているのですが
「このユーザーは退会しました」という表示が出ているということは、この行にたどりついておらず、
@if ($user === null) がtrueになっている可能性が高いと思われます。

[Keight]

あ、web.phpの中の記述に問題がある可能性があるのなら、web.phpの内容を載せようかと思った次第ですm(_ _)m
showが呼ばれているかというのは、どういうことなんでしょうか？
すみません、知識不足で????

@if ($user === null) がtrue・・・(´ε｀；)
ログインできており、記事の書き込みもでき、その書き込みをtinkerで確認したところ、ユーザーのidは 4 となっていますが、どうしてnullになるのか、怪しいところはどこか見受けられますでしょうか？

[yuuyu]

＞showが呼ばれているかというのは、どういうことなんでしょうか？
showは関数なので、関数を呼び出さないと内部は実行されないため、どこかでshow関数を呼び出しているか、という意味です。

あと今気付いたのですが、$user変数に入れているのは $user = User::find($id); の行のみでしょうか？
phpは基本的に関数の外と中で変数の共有をしないので、もしかしたら呼び出していても値のセットはできていないかもしれません。
例：
$a = "test";
function func(){
$a = "sample";
}
func();
echo $a; // "test"が出力される

[Keight]

＞ showは関数なので、関数を呼び出さないと内部は実行されないため、どこかでshow関数を呼び出しているか、という意味です。

web.php でこのように書いておりますが、関係ないでしょうか？

Route::resource('users', 'UsersController', ['only' => ['show']]);

＞ あと今気付いたのですが、$user変数に入れているのは $user = User::find($id); の行のみでしょうか？
phpは基本的に関数の外と中で変数の共有をしないので、もしかしたら呼び出していても値のセットはできていないかもしれません。

destroy のほうでも $user = User::find($id); としていますね。

public function destroy($id) {

$user = User::find($id);

if (\Auth::id() === $user->id) {
$user->delete();
}

return back();
}

コントローラーのshowのところで dd($user->id);で確認をすると、 4 と返ってはきます。

[m.ts10806]

質問は編集できますので適宜追記してください。
コメント欄はデフォルト非表示故、大事な情報が伝わらない可能性も高いです。

[yuuyu]

＞web.php でこのように書いておりますが、関係ないでしょうか？
すみません、Laravelについて詳しくはないので、その関数が何をしているかは分からないのですが、
＞コントローラーのshowのところで dd($user->id);で確認をすると、 4 と返ってはきます。
これをみる限り、おそらく呼び出しはできているので関係あるかもしれないです。

＞destroy のほうでも $user = User::find($id); としていますね。
この２箇所でしょうか？destroyも関数なので、これだけならスコープの問題な気がします。

[Keight]

表示させたいビューをよくよく考えてみると、今回の対象はUsersControllerのshowではなく、LecturesControllerのshowでした、すみません。
ただ、どちらにしてもやはり「このユーザーは退会しました」と出ていることに現状変わりはないですね。

「LecturesController@show」

public function show($id) {

$lecture = Lecture::find($id);
$user = User::find($id);

return view('lectures.show', [
'lecture' => $lecture,
'user' => $user,
]);
}

Answer 1

Lectureモデル内でUserの定義をしていたので、LecturesController@showで「$user = User::find($id);」のお記述を消し、
ビュー（lectures/show.blade.php）の「$user->id」の部分を「$lecture->user->id」変更すると、投稿者名が正しく表示されるようになりました。
また、投稿ページ（lectures/create.blade.php）にアクセスを試みた際、レクチャー詳細ページ（lectures/show.blade.php）に飛ぼうとする症状については、ルーティングの順序が原因でした。
ルーティングのルールとして、上にあるものが優先されるとのことですが、showを同じgetであるcreateよりも上で記述していたのがよくなかったようです。

Answer 2

return view('users.show', $data);

lectures/show.blade.php

showはshowでもディレクトリが違ってます

---

追記

@if ($lecture)
@endif

$lecture がある場合、のifブロックを抜けてから

@if (Auth::id() === $lecture->user_id)

$lecture を参照している
この時、$lectureがnullである可能性がある

Comments

[Keight]

ありがとうございます。
確認し直すと、LecturesControllerのほうでこのように記述しておりました。

public function show($id) {

$lecture = Lecture::find($id);
$user = User::find($id);

return view('lectures.show', [
'lecture' => $lecture,
'user' => $user,
]);
}

それでもやはり「このユーザーは退会しました」と出ています。
あと、なぜか今度は投稿フォームのページにアクセスした際に下記エラーが出るようになっていることに気付きました。

Trying to get property 'user_id' of non-object (View: E:\dev\htdocs\Laravel\bbl\resources\views\lectures\show.blade.php)

tinkerでLectureのデータのカラムを見ても user_id は存在していますし、ルーティングも下記で合っていると思うのですが。

Route::get('lectures/create', 'LecturesController@create')->name('lectures.create');

[Keight]

追記に対して・・・

@endifの位置を変え、

@if (Auth::id() === $lecture->user_id)
<div class="text-center mt-4">
{!! Form::open(['route' => ['lectures.destroy', $lecture->id], 'method' => 'delete']) !!}
{!! Form::submit('削除する', ['class' => 'btn btn-danger btn-sm del_check']) !!}
{!! Form::close() !!}
</div>
@endif

この部分まで包んでみました。
すると、「投稿する」のページにアクセスしてエラーは出なくなりましたが、本来表示させたい投稿フォームのページではなく、 lectures.show のページにアクセスされてしまうようです。
「投稿する」の部分は lectures.create に飛ばしたく、そのようにルーティング指定もしているのですが・・・。
https://gyazo.com/bff87ee9599b83a7c825580399938aed