Как сделать запрос с использованием 3 таблиц на Laravel?

Question

[kamilqiyasov]

Есть 3 класса:

class Brand extends Model
{
    protected $table = 'brands';

    public function categories()
    {
        return $this->hasMany(Category::class);
    }

    public function products()
    {
        return $this->hasManyThrough(Product::class, Category::class);
    }
}

class Category extends Model
{
    protected $table = 'categories';

    public function brand()
    {
        return $this->belongsTo(Brand::class);
    }

    public function products()
    {
        return $this->hasMany(Product::class);
    }

}

class Product extends Model
{
    protected $table = 'products';

    public function category()
    {
        return $this->belongsTo(Category::class);
    }

    public function brand()
    {
        return $this->belongsTo(Brand::class);
    }
}

Я объединил их, все замечательно, но как мне сделать запрос с объединением этих трех таблиц. Чтобы одним запросом я тянул все 3 таблицы и + добавил пару условий.
Проблема такая:
На странице мне нужно вывести продукты с категорией чпу которого равен remkomplekty и c брендом чпу = aro

На SQL я делаю так:

SELECT p.*
FROM products p, categories c, brands b
WHERE p.category_id = c.id 
AND c.slug = 'remkomplekty' 
AND c.brand_id = b.id
AND b.slug = 'aro'
ORDER BY quantity DESC, CODE

Подскажите, какой будет логичный и красивый код контроллера на "Языке Laravel".

Варианты с DB::table, DB::select не катят.

Answer 1

[Антон Антон]

Одним не получится, а вот по одному запросу на таблицу - да https://laravel.com/docs/5.8/eloquent-relationship...

Comments

[kamilqiyasov]

Это получается слишком много запросов будет.

[Антон Антон]

kamilqiyasov, ну тогда только QueryBuilder. Но на самом деле не так много. И часто большое количество джоинов обманывает оптимизатор и он выбирает неоптимальный план выполнения и один большой запрос дольше нескольких маленьких. Особенно если они в одном пакете и не теряется время на межпроцессное (и даже межсерверное) взаимодействие несколько раз на каждый запрос.

[Алексей Сундуков]

Антон Антон, если у вас сам запрос написан корректно и расставлены все индексы по fk, то один запрос с join отработает быстрее, чем Х мелких. Т.к. не будет оверхеда на взаимодействие приложения и базы.

[kamilqiyasov]

Алексей Сундуков, Ну вот я об этом, у меня все индексы расставлены. Но проблема в том, если запихнуть этот запрос в массив, то тянуть оттуда данные будет сложно. + если еще фильтры какие то добавить, то вообще запрос переписывать придется

[Антон Антон]

Алексей Сундуков, на план исполнения влияют не только индексы на ключи соединения, но и прочие условия. Да и при большом количестве джоинов тоже может быть грустно. При простом запросе без дополнительных условий и трех таблицах скорее всего один запрос будет чуть быстрее. При появлении условий, особенно на правые таблицы - не факт. Да еще и при "каскадном" джоине, когда а->b->c связь, а не a-> + a->c. В таком случае уже нужно смотреть замеры на реальных данных.

[Алексей Сундуков]

Антон Антон, с нормальным LEFT JOIN и условием в ON для увеличения селективности выборки из левой таблицы обычно проблем с JOIN не возникает. Даже при каскаде. Просто в левых таблицах должны быть более селективные условия максимально сужающие выборку. Конечно при условии наличия индексов по объединяющим полям.

Например в данном примере самой левой таблицей стоило бы поставить brands (как я понимаю там будет ровно одна запись), потом categories и только потом products. Но автор делает CROSS JOIN и это мне видится неправильным решением.

Answer 2

[Алексей Сундуков]

Попробуйте через with как это описано в Eager Loading: Eager Loading Multiple Relationships

Comments

[kamilqiyasov]

Я пробовал, 2 таблицы получилось. с 3 не понимает уже

[Алексей Сундуков]

kamilqiyasov, with может принимать callback в котором можно модифицировать запрос дописав туда нужные join и условия. Запрашивая products с categories при этом добавив внутри callback фильтр по brand.

[kamilqiyasov]

$products = Product::whereHas('category', function ($query) use ($categorySlug, $brandSlug) {
            $query->whereHas('brand', function ($query) use ($brandSlug) {
                $query->where('slug', $brandSlug);
            });
            $query->where('slug', $categorySlug);
        })
            ->orderBy('quantity', 'DESC')
            ->orderBy('code')
            ->get();

Через with нет, а вот через whereHas получилось

Answer 3

[kamilqiyasov]

$products = Product::whereHas('category', function ($query) use ($categorySlug, $brandSlug) {
            $query->whereHas('brand', function ($query) use ($brandSlug) {
                $query->where('slug', $brandSlug);
            });
            $query->where('slug', $categorySlug);
        })
            ->orderBy('quantity', 'DESC')
            ->orderBy('code')
            ->get();

Есть такой код, но при $products->category он делает новый запрос