Ошибка в запросе?

Question

[rennameduser_3432]

$sql_part .= "AND price > {$_GET['priceMin']} AND price < {$_GET['priceMan']}";

Текст ошибки: SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near '' at line 1

что не так?

UPD:
Весь запрос

$sql_part .= "AND price > {$_GET['priceMin']} AND price < {$_GET['priceMan']}";

$sql_part = "AND id IN (SELECT product_id FROM attribute_product WHERE attr_id IN ($filter) GROUP BY product_id HAVING COUNT(product_id) = $cnt)";

 $rows = \R::getAll("SELECT id, articul, category_id, brand_id, title, alias, content, price, old_price, status, keywords, description, img, hit, novinki  FROM product WHERE status = '1' AND category_id IN ($ids) $sql_part LIMIT $start, $perpage");

Comments

[Денис Юрьев]

это весь запрос?

[Rsa97]

Выведите весь собранный запрос и смотрите в нём.

Answer 1

[FanatPHP]

1. Надо включать отображение ошибок РНР.
Тогда ПХП тебе бы прямым текстом сказал, что в нет никакой переменной "priceMan". Хотя если смотреть на полный запрос, то скорее отсутствует parpage
2. Надо использовать для запросов подготовленные выражения
Тогда даже при отсутствии переменной не будет ошибки запроса. Не говоря уже про взлом сайта

Даже такая параша, как тухлая фасоль, и то умеет в замещение переменных в запросе

$params = [
'min' => $_GET['priceMin'],
'max' => $_GET['priceMax'],
'cnt' => $cnt,
'start' =>$start, 
'perpage' =>$perpage,
]
$idparam = '';
foreach ($ids as $i => $item)
{
    $key = ":id$i";
    $idparam .= $idstr? "," : "" . $key;
    $params[$key] = $item; 
}
$filterparam = '';
foreach ($ids as $i => $item)
{
    $key = ":f$i";
    $filterparam .= $filterparam? "," : "" . $key;
    $params[$key] = $item; 
}



$sql = "SELECT id, articul, category_id, brand_id, title, alias, content, price, old_price, status, keywords, description, img, hit, novinki  
FROM product 
WHERE status = '1' AND category_id IN ($idparam) 
AND price > :min AND price < :max
AND id IN (SELECT product_id FROM attribute_product WHERE attr_id IN ($filterparam) 
GROUP BY product_id 
HAVING COUNT(product_id) = :cnt)
LIMIT :start, :perpage"

$rows = \R::getAll($sql, $params);

Comments

[rennameduser_3432]

Текст ошибки: SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near ') GROUP BY product_id HAVING COUNT(product_id) = NU...' at line 5

[FanatPHP]

ну я так и думал что у тебя пустой массив фильтров. Как раз хотел добавить условие
Но я думаю и сам справишься

[rennameduser_3432]

FanatPHP,

$filter = '';
        if(!empty($_GET['filter'])){
            $filter = Filter::getFilter();
        }

сделал же

[FanatPHP]

ну и где здесь проверка, что $filter пустой?

[FanatPHP]

и что за говнокод в Filter::getFilter();?

[rennameduser_3432]

FanatPHP, Я делаю человеку это не мой код я сам не понимаю что тут творится

[Денис Юрьев]

rennameduser_3432, я вот ничего не понимаю в звукозаписи, и когда у меня спрашивают как лучше записать звук - я не берусь, а говорю что ничего в этом не понимаю

нахрена вы поползли в php и mysql, если ничего не понимаете?

[rennameduser_3432]

FanatPHP,

if(!empty($filter)) {
            $sql = "SELECT id, articul, category_id, brand_id, title, alias, content, price, old_price, status, keywords, description, img, hit, novinki  
            FROM product 
            WHERE status = '1' AND category_id IN ($ids) 
            AND price > :min AND price < :max
            GROUP BY product_id 
            HAVING COUNT(product_id) = :cnt)
            LIMIT :start, :perpage";
        }
        else {
            $sql = "SELECT id, articul, category_id, brand_id, title, alias, content, price, old_price, status, keywords, description, img, hit, novinki  
            FROM product 
            WHERE status = '1' AND category_id IN ($ids) 
            AND price > :min AND price < :max
            AND id IN (SELECT product_id FROM attribute_product WHERE attr_id IN ($filter) 
            GROUP BY product_id 
            HAVING COUNT(product_id) = :cnt)
            LIMIT :start, :perpage";
        }

[rennameduser_3432]

FanatPHP, SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near ') GROUP BY product_id HAVING COUNT(product_id) = 0)...' at line 5

не пойму что не так ._.

[FanatPHP]

тебе уже писали, чтобы ты вывел запрос
но хотя бы $filter можно вывести было?
чтобы не гадать на кофейной гуще, а ГЛАЗАМИ ПОСМОТРЕТЬ что там не так?

[rennameduser_3432]

FanatPHP,

SELECT id, articul, category_id, brand_id, title, alias, content, price, old_price, status, keywords, description, img, hit, novinki FROM product WHERE status = '1' AND category_id IN (78,79,80,81,82,83,84,85,64) AND price > :min AND price < :max AND id IN (SELECT product_id FROM attribute_product WHERE attr_id IN () GROUP BY product_id HAVING COUNT(product_id) = :cnt) LIMIT :start, :perpage

[FanatPHP]

!empty($filter) в каком случае сработает?

[rennameduser_3432]

FanatPHP, если не пусто

[rennameduser_3432]

FanatPHP, SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near ') LIMIT 0, 9' at line 6

[FanatPHP]

это потому что дебильный редбин не выключает режим эмуляции
надо посмотреть в интернете, как включить

[rennameduser_3432]

FanatPHP, SQLSTATE[42S22]: Column not found: 1054 Unknown column 'product_id' in 'group statement'

$sql = "SELECT id, articul, category_id, brand_id, title, alias, content, price, old_price, status, keywords, description, img, hit, novinki  
            FROM product 
            WHERE status = '1' AND category_id IN ($ids) 
            AND price > {$_GET['priceMin']} AND price < {$_GET['priceMax']}
            GROUP BY product_id 
            HAVING COUNT(product_id) = {$cnt}
            LIMIT {$start}, {$perpage}";