Как сделать правильно sql запрос на выборку товаров из фильтра в laravel?

Question

[Юрка Блохин]

Добрый день, подскажите как сделать правильно выборку из базы данных?

Сейчас делаю так:

$ar_select_properties = [1,2];
Product::select('product.*')
->leftJoin('product_attribute', 'product_attribute.product_id', '=', 'product.id')
->where(function($query) use ($ar_select_properties) {
    if (isset($ar_select_properties)) {
        foreach ($ar_select_properties as $property) {
            $query->where('product_attribute.attribute_id', $property);
        }
    }
})->get();

SQL получается

select `product`.`id` from `product` left join `product_attribute` on `product_attribute`.`product_id` = `product`.`id`
where `product_attribute`.`attribute_id` = '1' and `product_attribute`.`attribute_id` = '2'

но в результате ничего не приходит, но если $ar_select_properties = [1] или $ar_select_properties = [2]

например:

select `product`.`id` from `product` left join `product_attribute` on `product_attribute`.`product_id` = `product`.`id`
where `product_attribute`.`attribute_id` = '1'

возвращаются нормальные данные.

Answer 1

[Константин Б.]

Есть два пути это сначала получить все товары и атрибуты, а затем работать с коллекцией. Благо в Laravel есть все для этого.

Либо фильтровать через красноречивого
Но для этого вам нужно сделать связь многие ко многим. Судя по вашему коду она у вас есть, и я надеюсь она описана правильно.

Product::with(['attributes' => function ($q) use ($ar_select_properties) {
$q->whereIn('id', $ar_select_properties);
}])->get();

Comments

[Юрка Блохин]

связи у меня такой не было, пытаюсь её делать в модели Product

public function attributes()
    {
    	return $this->belongsToMany('App\Models\ProductAttribute', 'product_attribute', 'id', 'attribute_id');
    }

потом делаю,

Product::with(['attributes' => function ($q) use ($ar_select_properties) {
$q->whereIn('id', $ar_select_properties);
}])->get();

но выскакивает ошибка: Syntax error or access violation: 1066 Not unique table/alias: 'product_attribute'

[Юрка Блохин]

получилось реализовать через коллекцию, но получается, что для каждого товара надо делать отдельный запрос к базе данных и проверять его свойства. сейчас в базе 300 товаров и получается 300 запросов только на свойства(

[Константин Б.]

Юрка Блохин: Есть же метод with и load. Слушай тебе бы сначала доку поучить, а потом уже писать браться

[Константин Б.]

Юрка Блохин: Product::with('attributes')->foo()->bar();

[Юрка Блохин]

Спасибо большое, разобрался как делать

Answer 2

[vimes7]

Ничего не приходит, потому что поле `attribute_id` не может быть одновременно быть равно 1 и 2. Используйте условие OR.

Comments

[Константин Б.]

Не OR, а $query->whereIn('product_attribute.attribute_id', $property); и без foreach

[Юрка Блохин]

Kostik_1993: если использовать $query->whereIn('product_attribute.attribute_id', $property);, то получается, что при фильтрации товары не убавляются, а наоборот - добавляются.

[Константин Б.]

Юрка Блохин: потому что нужно еще добавить groupBy('id') для продукта

[Константин Б.]

Юрка Блохин:

# вот это
foreach ($ar_select_properties as $property) {
      $query->where('product_attribute.attribute_id', $property);
}

# замени на это
$query->whereIn('product_attribute.attribute_id', $ar_select_properties );

# а перед этим 
->get();

# вставь это 
->groupBy('id')

#чтобы получилось вот это
->groupBy('id')->get();

[Юрка Блохин]

Kostik_1993: я попробовал, но результат такой-же. при добавлении ещё одного свойства в фильтр количество продуктов увеличивается, а надо чтобы лишние убирались.