中間テーブルに入っているデータをControllerから検索・抽出したいです。

Question

実現したいこと

多人数チャット機能を作成しているのですが、room@showページでチャットを作成しているのですが、多人数チャットということでmemberという名前で中間テーブルを作成しまして、room@showメソッドでroom_idとroom->id、member_idとuser->idが一致したmember変数を作成しまして、showページでその変数が存在したらチャットを表示、無かったら「入室できていません」と表示するようにしたのですが、中間テーブルに無い新しく作ったアカウントでもチャットが表示されたので、controllerの記述またはshowページのifが機能していない可能性があるので原因を知りたいです。

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

中間テーブルにないユーザーもチャットを閲覧できるようになっています。

該当のソースコード

Schema::create('rooms', function (Blueprint $table) {
            $table->id();
            $table->string('name');
            $table->string('room_code');
            $table->string('room_password');
            $table->unsignedBigInteger('host_id')->constrained();
            $table->timestamps();
        });

public function up(): void
    {
        Schema::create('members', function (Blueprint $table) {
            $table->id();
            $table->unsignedBigInteger('room_id')->constrained();
            $table->unsignedBigInteger('member_id')->constrained();
            $table->timestamps();
        });
    }

public function show($id)
    {
        $user = Auth::user();
        $room = Room::find($id);
        $member = Member::where('room_id', '=', $room->id)
                        ->where('member_id', '=', $user->id)
                        ->get();
        $messages = Message::where('room_id', '=', $room->id)
                           ->get();
        return view('rooms/show', ['user'=>$user, 'room'=>$room, 'member'=>$member, 'messages'=>$messages]);
    }

<x-app-layout>
    <x-slot name="header">
        <h2 class="font-semibold text-xl text-gray-800 leading-tight">
            {{ $room->name }}
        </h2>
    </x-slot>
    {{$member}}
    @if(isset($member))
        <div class="romm-chat">
            <div class="room-chat-message">
                
            </div>
            <div class=="room-chat-form">
                <form method="POST" action="{{ route('message.store') }}">
                    @csrf
                    
                    <!-- Name -->
                    <div>
                        <x-text-input id="content" class="block mt-1 w-full" type="text" name="content" :value="old('content')" required autofocus autocomplete="content" />
                        <x-input-error :messages="$errors->get('content')" class="mt-2" />
                    </div></br>
                    <input type="hidden" id="user_id" name="user_id" value="{{$user->id}}">
                    <input type="hidden" id="room_id" name="room_id" value="{{$room->id}}">
                    
                    <div class="flex items-center justify-end mt-4">
                        <x-primary-button class="ms-4">
                            提案
                        </x-primary-button>
                    </div>
                    
                </form>
            </div>
        </div>
    @else
        入室許可を得ていません。
    @endif
</x-app-layout>

試したこと

controllerからのデータの検索方法を調べたのですが、whereが不適切なデータの検索の仕方だったりするでしょうか。

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

PHP8
Laravel10

Comments

[Eggpan]

$memberの内容は dd($member) などで確認されていますか？
データ取得してなさそうなので、常にクエリビルダーのオブジェクトが入っている状態じゃないですかね

[FKM]

$member = []であってもtrueを返しますね、その判定文だと。

[student]

isset($member)で試してみたのですが、同じ作動をしました。nullの方も試したのですが、同じくでした。
もしかしてcontrollerの方の$memberの検索の仕方がいけないのでしょうか？

[FKM]

まずは$memberにどんな値が代入されているか、var_dump($member)で調べてみては？そこが問題ないのであれば、モデルからSQLを呼び出してみてください。

https://qiita.com/y_sone/items/1bffdcf1b1232cae1ea0

[student]

すでにmemberテーブルで作成されているユーザーでview側で{{$member}}とdd($member)を試しました。
まずdd($member)で中身を確認したところ、全てnullになっているのでもしかしたらControllerでうまく情報を検索できていないかもしれません。
次にview側で{{$member}}で表示されるかを試したところ、htmlspecialchars(): argument #1 ($string) must be of type と表示され、ループ文を使うと表示されたので配列になっていました。
Controllerの検索方法を見直してみます。

[Eggpan]

dd($member) をコントローラの1行目とかに書いてませんか？
$memberに代入した直後はnullではないはずです。

同様に$messagesの代入も誤りです。これはどちらかというと未使用なのに取得しているというのもありますが・・。

メソッドの呼び方が正しくなくて、SQLが実行されてないはずです。変数がどうなっているかくらいは調査出来るようになりましょう。

[student]

->get()でshowページに{{$member}}を表示することが出来ましたが、相変わらず@ifで表示切替を出来ませんでした。
@isset($member)も試しましたが、同様の結果でした。
dd($member)もnullではありませんでした。すみませんでした。

[Eggpan]

今のコードだと、SQLのレコードがない場合でも$memberに値が入っているから @if($member) は常にtrueとなっていることは理解されていますか？
そもそも $member は何を入れたいんでしょうか

[student]

すみません。理解を出来ていませんでした。
$memberにはroom_idはroom->idと、member_idはuser->idと一致する2つの条件を満たしているmemberレコードを取得したいです。

[FKM]

だとするとSQLを吐き出してみて、それをデータベース環境で実行してみるといいと思います。

[student]

すみません。
controller側でif文のnull判定を行うとかは駄目ですか？

[FKM]

その判定を行うオブジェクトがどんな形でデータが入っているか確認するためには、SQLを叩いてみる必要があるからです。そこがクリアしてるなら、一番てっとり早い方法はis_countable関数でカウント可能なオブジェクトかどうか判定するという手があります。

null判定だけをおすすめしない理由はSQLが構造上間違ってる場合でも、SQLは正確で検索条件に適合した値が存在しない場合でも、同じ結果になる恐れがあるからです。

Answer 1

何故 @if(isset($member)) の条件を常に通過してしまうのか

まず、今回のコードでは、 $member には下記の結果が入っています。

$member = Member::where('room_id', '=', $room->id)
    ->where('member_id', '=', $user->id)
    ->get();

get() メソッドはSQLを実行し、結果を Illuminate\Support\Collection インスタンスとして返却します。
件数が0個であっても null が返却されるわけではなく、Collectionインスタンスとして返されます。
そのため、 isset($member) の結果は常にtrueとなります。

どうすれば良いのか

今回のケースですと $member は「レコードが1件見つかる、もしくは見つからなくて0件」のどちらかのはずです。
このような場合、Laravelではfirst() メソッドを使います。

$member = Member::where('room_id', '=', $room->id)
    ->where('member_id', '=', $user->id)
    ->first();

first() メソッドはレコードが見つかった場合、Modelのオブジェクトを返却します。レコードが見つからない場合、null を返却します。

なんらかのデータを取り扱うアプリケーションにおいて、この様なクエリビルダーの使い方は基礎の基礎なので、細かな違いをしっかりと把握しましょう。

また、if 文がどのような場合にtrueとなるか、みたいな話はLaravel以前にPHPとしての基礎です。

参考：
https://readouble.com/laravel/10.x/ja/queries.html

余談

isset部分ですが、次のように書くとより簡潔になります。

@isset($member)
    ....
@else
    ....
@endisset

Comments

[student]

すみません。解決しました。
わかりやすい解説でした。ありがとうございました。