Laravel リレーション定義したデータを抽出する際に関して

Question

前提・実現したいこと

Laravelで図書館で借りた本をユーザが確認できる様なものを作成しています。
リレーション定義したものを取得することは以前の質問で解決することができたのですが、複数のレコードがあった際にそれを取得することができましたので問題はないのかと思っています。
しかし、当日分のデータをその借りた時間から取得するというコードがうまく動作しなくなってしまいました。当日分のデータは book_userテーブルのcreatedを指しています。

該当のソースコード

$user = User::with(['books'])->latest();
foreach ($user->books as $book) {
    dump($book->pivot->created_at);
    dump($book->book_name);
}

試したこと

下記の様にwhereで繋げてみました。
結果はエラーが表示されなかったのですが、当日のデータ以外も表示されてしまいました。

$user = User::with(['books' => function($q) {
    $q->where('book_user.created_at', '>', date('Y-m-d'));
}])->latest();
foreach ($user->books as $book) {
    dump($book->pivot->created_at);
    dump($book->book_name);
}

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

Laravel 7.28.1
PHP 7.X
MySQL

create_users_table.php

            $table->id();
            $table->string('name');
            $table->string('email')->unique();
            $table->timestamp('email_verified_at')->nullable();
            $table->string('password');
            $table->rememberToken();
            $table->timestamps();

create_book_user_table.php

            $table->id();
            $table->unsignedBigInteger('user_id');
            $table->unsignedBigInteger('book_id');
            $table->foreign('user_id')->references('id')->on('users')->onDelete('cascade');
            $table->foreign('book_id')->references('id')->on('books')->onDelete('cascade');
            $table->timestamps();

create_book_table.php

            $table->id();
            $table->char('book_name');
            $table->bigInteger('book_price');
            $table->text('book_detail');
            $table->timestamps();

App\User.php

<?php

namespace App;

use Illuminate\Contracts\Auth\MustVerifyEmail;
use Illuminate\Foundation\Auth\User as Authenticatable;
use Illuminate\Notifications\Notifiable;

class User extends Authenticatable
{
    use Notifiable;

    /**
     * The attributes that are mass assignable.
     *
     * @var array
     */
    protected $fillable = [
        'name', 'email', 'password',
    ];

    /**
     * The attributes that should be hidden for arrays.
     *
     * @var array
     */
    protected $hidden = [
        'password', 'remember_token',
    ];

    /**
     * The attributes that should be cast to native types.
     *
     * @var array
     */
    protected $casts = [
        'email_verified_at' => 'datetime',
    ];

    public function books(){
        return $this->belongsToMany("App\Book")->withTimestamps();
    }

}

App\Book.php

    <?php

    namespace App;

    use Illuminate\Database\Eloquent\Model;

    class Book extends Model
    {
        protected $table = "books";

        protected $fillable = [
            'book_name', 'book_price', 'book_detail',
        ];

        public function users(){
            return $this->belongsToMany("App\User");
        }
    }

Comments

[hentaiman]

勉強の為ならともかく、リレーション張ったからと言って無理に使う必要無いですよ

[yuki84web]

whereの条件に'>'とありますが、これの意味は理解されてますか？

Answer 1

Migrations

<?php

use Illuminate\Database\Migrations\Migration;
use Illuminate\Database\Schema\Blueprint;
use Illuminate\Support\Facades\Schema;

class CreateBooksTable extends Migration
{
    /**
     * Run the migrations.
     *
     * @return void
     */
    public function up()
    {
        Schema::create('books', function (Blueprint $table) {
            $table->id();
            $table->string('name');
            $table->bigInteger('price');
            $table->string('detail', 1000)->nullable();
            $table->timestamps();
        });
    }

    /**
     * Reverse the migrations.
     *
     * @return void
     */
    public function down()
    {
        Schema::dropIfExists('book');
    }
}

<?php

use Illuminate\Database\Migrations\Migration;
use Illuminate\Database\Schema\Blueprint;
use Illuminate\Support\Facades\Schema;

class CreateBookUserTable extends Migration
{
    /**
     * Run the migrations.
     *
     * @return void
     */
    public function up()
    {
        Schema::create('book_user', function (Blueprint $table) {
            $table->id();
            $table->unsignedBigInteger('user_id');
            $table->unsignedBigInteger('book_id');
            $table->timestamps();

            $table->foreign('user_id')->references('id')->on('users')->onDelete('cascade');
            $table->foreign('book_id')->references('id')->on('books')->onDelete('cascade');
        });
    }

    /**
     * Reverse the migrations.
     *
     * @return void
     */
    public function down()
    {
        Schema::dropIfExists('book_user');
    }
}

Models

<?php

namespace App;

use Illuminate\Database\Eloquent\Relations\BelongsToMany;
use Illuminate\Foundation\Auth\User as Authenticatable;
use Illuminate\Notifications\Notifiable;

class User extends Authenticatable
{
    use Notifiable;

    /**
     * The attributes that are mass assignable.
     *
     * @var array
     */
    protected $fillable = [
        'name', 'email', 'password',
    ];

    /**
     * The attributes that should be hidden for arrays.
     *
     * @var array
     */
    protected $hidden = [
        'password', 'remember_token',
    ];

    /**
     * The attributes that should be cast to native types.
     *
     * @var array
     */
    protected $casts = [
        'email_verified_at' => 'datetime',
    ];

    /**
     * @return BelongsToMany
     */
    public function books(): BelongsToMany
    {
        return $this->belongsToMany(Book::class)
            ->withTimestamps();
    }
}

<?php

namespace App;

use Carbon\Traits\Timestamp;
use Illuminate\Database\Eloquent\Model;
use Illuminate\Database\Eloquent\Relations\BelongsToMany;

class Book extends Model
{
    use Timestamp;

    protected $fillable = [
        'name',
        'price',
        'detail',
    ];

    /**
     * @return BelongsToMany
     */
    public function users(): BelongsToMany
    {
        return $this->belongsToMany(User::class)
            ->withTimestamps();
    }
}

factories

<?php

/** @var \Illuminate\Database\Eloquent\Factory $factory */

use App\User;
use Faker\Generator as Faker;
use Illuminate\Support\Str;

/*
|--------------------------------------------------------------------------
| Model Factories
|--------------------------------------------------------------------------
|
| This directory should contain each of the model factory definitions for
| your application. Factories provide a convenient way to generate new
| model instances for testing / seeding your application's database.
|
*/

$factory->define(User::class, function (Faker $faker) {
    return [
        'name' => $faker->name,
        'email' => $faker->unique()->safeEmail,
        'email_verified_at' => now(),
        'password' => '$2y$10$92IXUNpkjO0rOQ5byMi.Ye4oKoEa3Ro9llC/.og/at2.uheWG/igi', // password
        'remember_token' => Str::random(10),
        'created_at' => $faker->dateTime('-10 years'),
        'updated_at' => $faker->dateTime('-10 years'),
    ];
});

<?php

/** @var \Illuminate\Database\Eloquent\Factory $factory */

use App\Book;
use Faker\Generator as Faker;

$factory->define(Book::class, function (Faker $faker) {
    return [
        'name' => $faker->word,
        'price' => $faker->numberBetween(1000, 3000),
        'detail'=>null,
        'created_at' => $faker->dateTime('-10 years'),
        'updated_at' => $faker->dateTime('-10 years'),
    ];
});

Seeder

<?php

use Illuminate\Database\Seeder;

class UserSeeder extends Seeder
{
    /**
     * Run the database seeds.
     *
     * @return void
     */
    public function run()
    {
        factory(\App\User::class, 5)->create();
    }
}

<?php

use Illuminate\Database\Seeder;

class BookSeeder extends Seeder
{
    /**
     * Run the database seeds.
     *
     * @return void
     */
    public function run()
    {
        factory(\App\Book::class, 100)->create()->each(function (\App\Book $book) {
            $count = rand(0, \App\User::all()->count());
            $user_ids = \App\User::query()->inRandomOrder()->take($count)->pluck('id');

            for ($i = 0; $i < $count; $i++) {
                $start = now()->subYears(rand(0, 10))->subDays(rand(0, 365));
                $book->users()->attach($user_ids[$i], [
                    'created_at' => $start
                ]);
            }
        });
    }
}

UserController

<?php

namespace App\Http\Controllers;

use App\User;
use Illuminate\View\View;

class UserController extends Controller
{
    /**
     * Display a listing of the resource.
     *
     * @return View
     */
    public function index(): View
    {
        $users = User::with(['books' => function ($query) {
            $query->whereDate('pivot_created_at', now());
        }])->get();
        return view('users.index', compact('users'));
    }
}

Blade

views/users/index.blade.php

<!doctype html>
<html lang="ja">
<head>
    <meta charset="UTF-8">
    <meta name="viewport"
          content="width=device-width, user-scalable=no, initial-scale=1.0, maximum-scale=1.0, minimum-scale=1.0">
    <meta http-equiv="X-UA-Compatible" content="ie=edge">
    <title>Document</title>
</head>
<body>
<div>
    <table>
        <thead>
        <tr>
            <th>ユーザーID</th>
            <th>ユーザー名</th>
            <th>メールアドレス</th>
            <th>本日貸出した本</th>
            <th>貸出日時</th>
        </tr>
        </thead>
        <tbody>
        @foreach ($users as $user)
            <tr>
                <td>{{ $user->id }}</td>
                <td>{{ $user->name }}</td>
                <td>{{ $user->email }}</td>
                <td>{{ $user->books->first()->name }}</td>
                <td>{{ $user->books->first()->pivot->created_at }}</td>
            </tr>
        @endforeach
        </tbody>
    </table>
</div>
</body>
</html>

Comments

[phper.k]

細かな調整はご自身でやってください。

[退会済みユーザー]

ご丁寧にありがとうございます！

一点気になったのですが、Modelのリレーション定義において以下の様に ::classとあったのですが、*2でも意味は変わらず明示的に記述されているのでしょうか？
return $this->belongsToMany(User::class)
*2 return $this->belongsToMany("App\User");

[phper.k]

「明示的に記述されているのでしょうか？」←どういう意味で聞いているんですか？
質問の意味がわかりません。

[退会済みユーザー]

return $this->belongsToMany(User::class);
return $this->belongsToMany("App\User");

前者と後者の違いがわからない為質問させていただきました。

[phper.k]

機能としては同じです。

dd(User::class);

として実行してみれば、どんな値になっているかわかるはず。

[phper.k]

https://www.php.net/manual/ja/language.oop5.basic.php#language.oop5.basic.class.class

[退会済みユーザー]

ありがとうございます！確認したところ変わりない様でした。

[退会済みユーザー]

度々すいませんが、複数ある時も1つのデータしか取得できない様なのですがご確認いただけると幸いです

[退会済みユーザー]

usersをdumpしたところ一つの配列しかありませんでした。

[phper.k]

それ、私がすること？
仕事じゃあるまいし、まして私に割り当てられたタスクじゃないんですよ

[phper.k]

> 複数ある時も1つのデータしか取得できない様なのです
> usersをdumpしたところ一つの配列しかありませんでした。

聞く前にそれが、回答者に問うて、適切かどうか判断しているんでしょうか？

> 複数ある時も1つのデータしか取得できない様なのです

「何が」複数あるのか？

> usersをdumpしたところ一つの配列しかありませんでした。

データが一つしかなけりゃ当然じゃない？ちゃんとデータはあるの？
前提条件を共有せずに聞くことかい？

[退会済みユーザー]

すいません。以前の質問よりご指摘いただいた貸し出し履歴、複数回あるときに正しく取得できるかどうかの話をしていました。質問本文にある以前のコードでは貸し出し履歴、複数回ある時は取得できていたのですが、回答していただいたコードでは貸し出し履歴、複数回ある時取得できていなかった為質問させていただいた次第です。

> 貸し出し履歴、複数回あるときに正しい貸し出し日時を取得できるかどうか確認してみてください。

[phper.k]

<td>{{ $user->books->first()->name }}</td>
<td>{{ $user->books->first()->pivot->created_at }}</td>

ここを適切に編集すれば良いということは気づけないですか？

[退会済みユーザー]

ありがとうございます。こちら本題の件は解決しましたのでベストアンサーにさせていただきます。

[退会済みユーザー]

> ここを適切に編集すれば良いということは気づけないですか？

現状見当がつかないです、、

[phper.k]

> 現状見当がつかないです、、

とりあえずコピペでやってる感じなんですかね・・・

[退会済みユーザー]

first() とすることでfirstメソッドは指定された真偽テストをパスしたコレクションの最初だけの要素を取得していることが原因かと思い、全ての要素を取得するコレクションとしてget()やall()など試しているのですがエラーとなってしまっている現状なのです、、

参考にしたドキュメント https://readouble.com/laravel/7.x/ja/collections.html#method-first

[phper.k]

ロシアンルーレットみたいなコードの書き方してるな・・

dd($user->books) でまず、$user->books がどのクラスなのか確認しなよ。
クラスがわかれば、そのクラスが持つメソッドが当たりつくじゃん。

[退会済みユーザー]

@foreach ($users as $user)
@foreach ($user->books as $book)
{{ $book->name }}
@endforeach
@endforeach

とすることで取得できました。

[phper.k]

LGTM

[退会済みユーザー]

大変失礼しました。先程のものではできず以下でできた様です。。

@php
foreach ($users as $user)
i = 0;
$user->books[i]['book_name']
i++;
endforeach
@endphp

[phper.k]

NO GOOD

[退会済みユーザー]

あまり良くない記述ということでしょうか？

[phper.k]

それ以前に、それは動かないでしょ。
ループの中で $i イテレータを回すのもナンセンスです。