ユーザーごとの情報を出力する方法がわからない

Question

やりたいこと

・ユーザーごとにchecklist==1の場合とchacklist==0の場合をそれぞれ出力したい

名前	研修済	研修未完
user1	3	2
user2	1	0
user3	1	2

現状

・ユーザの名前は出力できているけど、「研修済」「研修未完」がuser1の情報がuser2,user3にも反映されている

名前	研修済	研修未完
user1	3	2
user2	3	2
user3	3	2

質問

※foreachで繰り返す中でuserテーブルの['id']とmemosテーブルの['user_id']が一致したうえでchecklist==1 or 0をカウントする処理？？
処理の方法を探してもなかなか記事も見つからないのでterataillで質問をさせていただきました

コード

AuthController.php

    public function userlist(){
        $user = \Auth::user();
        $userlist = User::paginate(10);
        $memolists = Memo::where('checklist','1')->where('status','1')->get();
        $memoyet = Memo::where('checklist','0')->where('status','1')->get();
        //論理削除　status==2は削除扱い
        return view('userlist',compact('user','userlist','memolists','memoyet'));        
    }    

userlsit.blade.php

@extends('layouts.app')

@section('content')
<div class="container">
    <div class="row justify-content-center">
        <div class="col-md-8">
            <div class="card">
                <table class="table">

                <h1>ユーザ一覧</h1>
                    <thead class = "table-dark">
                        <tr>
                        <th scope="col">ユーザー名前</th>
                        <th scope="col">研修済</th>
                        <th scope="col">研修未完</th>
                        </tr>
                    </thead>
                    <tbody>
                    @foreach($userlist as $list)
                    <tr>
                    <td>
                        <a href = "/userdetail/{{ $list['id'] }}" class="name" >
                                {{ $list['name'] }}
                            </a>
                    </td>

                    <th scope="row">
                    
                    <?php $i=0; ?>
                    @foreach($memolists as $memolist)
                        <?php $i++; ?>
                    @endforeach
                    <?php echo $i; ?>
                    </th>

                    <th scope="row">
                    <?php $i=0; ?>
                    @foreach($memoyet as $yet)
                        <?php $i++; ?>
                    @endforeach
                    <?php echo $i; ?>

                    </th>
                        

                        </tr>
                        @endforeach

                    </tbody>
                    
                    </table>

                </div>
    </div>
</div>
@endsection

テーブル定義情報

usersテーブル

id	name
1	test1
2	test2
2	test3

memosテーブル

id	user_id	status	checklist
1	1	1	0
2	1	1	0
3	1	1	1
4	1	1	1
5	1	1	1
6	2	1	1
7	3	1	1
8	3	1	0
9	3	1	0

Comments

[phper.k]

とりあえずテーブルの定義情報共有してもらわんことには回答してあげられんわ

[m.ts10806]

提示されたのは定義ではなくデータです。

Answer 1

User モデルに HasMany で Memo モデルへのリレーションを設定する

// App\Models\User

class User extends Model
{
    public function memos(): HasMany
    {
        return $this->hasMany(Memo::class);
    }

    public function memosCheckedTrue(): HasMany
    {
        return $this->memos()
            ->where([
                ['checklist', 1],
                ['status', 1],
            ]);
    }

    public function memosCheckedFalse(): HasMany
    {
        return $this->memos()
            ->where([
                ['checklist', 0],
                ['status', 1],
            ]);
    }
}

// App\Http\Controllers\UserController

class UserController extends Controller
{
    public function index(): View
    {
        $users = User::query()
            ->withCount(['memosCheckedTrue', 'memosCheckedFalse'])
            ->get();
        return view('', compact('users));
    }
}

<table>
    <thead class = "table-dark">
        <tr>
            <th scope="col">ユーザー名前</th>
            <th scope="col">研修済</th>
            <th scope="col">研修未完</th>
        </tr>
    </thead>
    <tbody>
        @foreach($users as $user)
        <tr>
            <td>{{ $user->name }}</td>
            <td>{{ $user->memosCheckedTrue_count }}</td>
            <td>{{ $user->memosCheckedFalse_count }}</td>
        </tr>
        @endforeach
    </tbody>
</table>

Comments

[yoshioka01]

上記のようにコードを書いたらできました！

Answer 2

「Memo」にユーザIDがあるのでしたらそれでも良いでしょうけど、関連するテーブルを別々で取るのは非常に非効率と思います。関係のないユーザー分も毎回ループしなければなりません。データ量が増えれば増える程、パフォーマンスが悪くなります。

まずはLaravelとしてどうするかではなくSQL１発でできないか考えてみましょう。
おそらく、こんな感じのものを組むことになると思います。

※テーブル定義提示されてないので仮

select
    user.id,
    user.name,
    (
        select
            count(*)
        from
            memo
        where
            memo.user_id = user.id
            and memo.checklist='1'
            and memo.status='1'
    ) as already,
    (
        select
            count(*)
        from
            memo
        where
            memo.user_id = user.id
            and memo.checklist='0'
            and memo.status='1'
    ) as yet
from
    user

調べた感じ、LaravelのクエリビルダでもSELECTにサブクエリを含むことが出来るようなので、Laravelの機能でも上記のようなSQL発行は可能に思います。

Comments

[yoshioka01]

pubic function userlist(){ //AccountController.phpのuselistメソッド

$sql = <<<SQL
select users.id, users.name, ( select count(*) from memos where memos.user_id = users.id and memos.checklist='1' and memos.status='1' ) as already, ( select count(*) from memos where memos.user_id = users.id and memos.checklist='0' and memos.status='1' ) as yet from users;
SQL;

return view('userlist',['sql'=>$sql]);
}

で行うとuserlist.blade.phpの方で@foreach($sql as $list)でデータが入ってないエラーが出てしまいます

[m.ts10806]

それ、ただの文字列を渡してるだけですよ。
実行結果のデータではないです。