Как вы оптимальнее фильтруете данные по многим параметрам?

Question

[Mokhirjon Naimov]

Сразу скажу что решение надо которое могу использовать в shared hosting'е, то есть обычных PHP & MySQL (без Radis и.т.д.)!

Делаю интернет-магазин, как вы знаете товары надо фильтровать по многим параметрам (ну для удобство пользователя). Например для 3-ох параметров (color, size, brand) мое решение такое (ну просто ужасное):

if (isset($color) && isset($size) && isset($brand)) {
    // where color = $color and size = $size and brand = $brand
} elseif (isset($color) && isset($size) && ! isset($brand)) {
    // where color = $color and size = $size
} elseif (isset($size) && isset($brand) && ! isset($color)) {
    // where size = $size and brand = $brand
} elseif (isset($color) && isset($brand) && ! isset($size)) {
    // where color = $color and brand = $brand
} elseif (isset($color) && ! isset($size) && ! isset($brand)) {
    // where color = $color
} elseif (isset($size) && ! isset($color) && ! isset($brand)) {
    // where size = $size
} elseif (isset($brand) && ! isset($color) && ! isset($size)) {
    // where brand = $brand
}  else {
    // здесь по умолчанию
}

А теперь представьте что надо фильтровать товары по 10-15 параметрам :) Аж голова кружится...ку
Вот и хочется все это оптимизировать (узнать про практичное решение). Уверен многие сталкивались этой проблемой, поделитесь решением.

В частности решение нужно для Laravel 5.1. Но обычное PHP & SQL тоже будет танком для меня :)

Answer 1

[Вячеслав Плиско]

А в чём проблема, зачем такие сложные ифы, в простом варианте можно обойтись простми

if ($color = \Input::get('color')) {
   $model->where('color', $color);
}
if ($size = \Input::get('color')) {
   $model->where('size', $size);
}
//....

Можно оптимизировать засунув фуильтры в массив

foreach($filters as $filter) {
     if ($value = \Input::get($filter)) {
        $model->where($filter, $value);
    }
}

и т.д.
Если у вас нормальная база с EAV и фасетами, то полезно будет конструкция ::has
Почитайте доку по фреймворку, чтобы не было геммороя из-за незнания работы его конструкций, так понимаю про то, что where складыватся в and вы похоже не знали.

Comments

[Mokhirjon Naimov]

Вячеслав Плиско спасибо то что искал, тока еще надо проверить...

[Mokhirjon Naimov]

Route::get('check', function()
{
	$items = App\Goods::all();

	echo "Total of goods: " . $items->count() . "";

	if ($size = \Input::get('size')) {
		$items = $items->where('size', $size);
	}

	if ($color = \Input::get('color')) {
		$items = $items->where('color', $color);
	}

	if ($price = \Input::get('price')) {
		$items = $items->where('price', '<=', $price);
	}

	if ($amounts = \Input::get('amounts')) {
		$items = $items->where('amounts', '<=', $amounts);
	}

	echo "Total of goods after the filtering: " . $items->count();
});

Можете объяснить почему не работает?

[Mokhirjon Naimov]

Сделал так, теперь работает. Спасибо за ответ!

Route::get('check', function()
{
	$items = new App\Goods;

	echo "Total of goods: " . $items->count() . "";

	if ($size = \Input::get('size')) {
		$items = $items->where('size', $size);
	}

	if ($color = \Input::get('color')) {
		$items = $items->where('color', $color);
	}

	if ($price = \Input::get('price')) {
		$items = $items->where('price', '<=', $price);
	}

	if ($amounts = \Input::get('amounts')) {
		$items = $items->where('amounts', '<=', $amounts);
	}

	echo "Total of goods after the filtering: " . $items->count();
});

[Вячеслав Плиско]

Mokhirzon Naimov: модель накапливает состояния, поэтому не надо каждый раз делать присваивание. достаточно
$items->where('amounts', '<=', $amounts);
$items->where('amounts', '<=', $amounts);
Если бы не ифы, то можно писать в текучем стиле
$items->where('amounts', '<=', $amounts)->where('amounts', '<=', $amounts);

[Mokhirjon Naimov]

Вячеслав Плиско проста когда я раньше деле с all() не работало без присваивание, теперь попробую без присваивание

Answer 2

[Станислав Почепко]

Сделайте себе конструктор BD запроса, который будет собирать вам запрос по кусочкам. Когда вызываете метод where($field, $value) - то он генерирует строку для блока запроса SQL WHERE.
$sql .= "`field` = 'value'"

Comments

[Mokhirjon Naimov]

Станислав Почепко Это в смысле для Laravel? Или вообще...

Answer 3

[Денис Котляров]

Ух, ивправду скрипт тяжеленький:)
Так подсказать не могу, но еслиб я использовал Mysql, то обычного WHERE ему хватилобы.

Comments

[Mokhirjon Naimov]

Денис Котляров WHERE хватает, но параметрами проблемы. Но если что уже подсказали ответы для Laravel и для голова SQL.