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

Question

[survivor2005]

Делаю интернет магазин электроники на чистом пхп. Дошел до фильтров и встрял.
Суть думаю стандартная, есть фильтры у товаров в виде чекбоксов. Например хочу выбрать ноутбуки у которых 6 гигов памяти, 8 гигов и с процессором АМД, ну и т.д.
Есть таблица properties. В ней названия свойств.

Есть таблица значений этих свойств property_values

И таблица товаров.


В общем, когда жмут на чекбоксы, приходит массив на обработку. Естественно, я не могу знать заранее сколько и чего придет, не знаю правильно или нет, пришлось сделать сперва строку запроса циклом, затем сформировать массив подстановок (тоже не знаю, дает ли это безопасность в данном случае). Я просто ничего другого не придумал. Может все делается по-другому. Вот код:

public static function findFilteredProducts(array $arr=[])
    {
        $request = '';
        
        if(!empty($arr)) {
            foreach($arr as $id => $value) {
                $request .= ' AND property_values.property_id=:'.$id;
                foreach($value as $val) {
                    $request .= ' AND property_values.value=:'.$val;
                }
            }
        }
        if(!empty($arr)) {
            foreach($arr as $id => $value) {
                $placeholder[':'.$id]=$id;
                foreach($value as $val) {
                    $placeholder[':'.$val]=$val;
                }
            }
        }
      
        
        $db = new Db;
        return $db->query('SELECT product.* FROM ' . self::TABLE . ', property_values WHERE property_values.product_id=product.id '.$request,
        $placeholder);
    }

Из вышеуказанного кода выходит такой результат:

( //Массив подстановок
    [:86] => 86
    [:8] => 8
    [:87] => 87
    [:AMD] => AMD
)
PDOStatement Object
( //Строка запроса
    [queryString] => SELECT product.* FROM product, property_values WHERE property_values.product_id=product.id  AND property_values.property_id=:86 AND property_values.value=:8 AND property_values.property_id=:87 AND property_values.value=:AMD
)

Работает только если выбрать 1 чекбокс, более одного не пашет уже, не выдает товары.

Comments

[FanatPHP]

Ох, горе ты луковое
Надо же хоть немного понимать SQL, который ты пишешь
Ну откуда у тебя в property_values возьмется строка, в которй value одновременно и 8, и AMD?

Под каждое условие нужен отдельный джойн

[Slava Rozhnev]

Очень хотелось бы видеть в вопросе DDL & DML запросы вместо картинок или ссылки на fiddle. По типу:

create table properties (
    id int auto_increment primary key,
    ....
    ....
);

insert into properties values (....) ....

fiddle

[survivor2005]

FanatPHP, Да честно скажу, жутчайший пробел в SQL, учу его на ходу). Спасибо, ты как всегда злой красавчик)). В общем надо объединять запросы с помощью join. Я попробую) Не факт что получится у лукового горе)))

[Slava Rozhnev]

FanatPHP, не обязательно. Можно один с подсчетом подходящих строк

Answer 1

[Slava Rozhnev]

Вот вариант построения запроса:

<?php
$request = [
	87=>'AMD',
	86=>'8'
];

$filter = implode(' or ', array_fill(0, count($request), '(pv.property_id = ? and pv.value = ?)'));

$filter_values = [];

foreach($request as $key=>$val) {
	$filter_values[] = $key;
	$filter_values[] = $val;
}

$query = 'select p.* from products p
	join property_values pv on p.id = pv.product_id
	where  ' . $filter . ' 
	group by p.id, p.name
	having count(distinct pv.id) = ' . count($request);

Share PHP code

Comments

[survivor2005]

Спасибо большое, попробую, отпишусь!

[FanatPHP]

лучше делай через джойны.
group by - это не та операция которую стоит пихать без разбора в запросы просто от скуки или от лени

[FanatPHP]

не говоря уже о том что здесь запрос сначала выбирает кучу ненужных данных, а потом только отфильтровывает нужные