Как получить связанные данные одним запросом?

Question

[PRodion]

Есть:

class YarnController extends Controller
{
    public function show($manufacturer, $collection = null)
    {
        if (!$collection) {
            $collections = Collection::whereHas('manufacturer', function($query) use ($manufacturer) {
                $query->where('id', $manufacturer);
            })->get();
        } else {
            $collections = Collection::whereHas('manufacturer', function($query) use ($manufacturer) {
                $query->where('id', $manufacturer);
            })->where('id', $collection)->get();
        }

        return view('yarn', compact('collections'));
    }
}

И

@foreach ($collections as $collection)
    <p>{{ $collection->manufacturer->name }} - {{ $collection->name }}</p>
@endforeach

Получается:

select * from `collections` where exists (select * from `manufacturers` where `collections`.`manufacturer_id` = `manufacturers`.`id` and `id` = '2')
select * from `manufacturers` where `manufacturers`.`id` = 2 limit 1
select * from `manufacturers` where `manufacturers`.`id` = 2 limit 1
select * from `manufacturers` where `manufacturers`.`id` = 2 limit 1

Коллекции производителя выводятся за один запрос, но вот для получения производителя каждый раз делается новый запрос.

Comments

[Рустам Садыков]

Зачем вы дублируете код? Используйте ->when после ->whereHas

[N]

Рустам Садыков, Тут сначала с "получением связей" бы разобраться))

[PRodion]

Рустам Садыков, не совсем понял о чем вы. Объясните для совсем зеленого.)

[Рустам Садыков]

PRodion,

$collections = Collection::whereHas('manufacturer', function($query) use ($manufacturer) {
    $query->where('id', $manufacturer);
})->when($collection, function ($q) use ($collection) {
    $q->where('id', $collection);
})->get();

И не нужно ни каких условий

[N]

Раз такая пьянка, многим будет полезно тоже:
20 Eloquent ORM трюков

[PRodion]

Рустам Садыков, круто. Может тогда подскажете как вернуть 404, если в базе 0 коллекций?

Answer 1

[N]

Collection::with('manufacturer')->whereHas...

https://laravel.com/docs/8.x/eloquent-relationship...

Comments

[PRodion]

N, спасибо! Похоже вы тут 24/7))

[N]

PRodion, Так кажется) из 24 я сплю, как раз, 7 :)