Почему не срабатывает фильтр в MySQL?

Question

[Dmitry2307]

Добрый день. Нужна помощь, не срабатывают фильтры для SQL, при запросе к базе данных, на страницу выводятся товары без применения фильтра, при этом при наборе такого же кода в phpMyAdmin - фильтры применяются.

function loadGoods() { //функция вывода товаров из БД
	$conn = connect();	
	$sql = "SELECT * FROM `smirnov` WHERE visible='1' ORDER BY '$sorting' ";
	$result = mysqli_query($conn, $sql);
	if (mysqli_num_rows($result) > 0) {
		$out = array();
		while($row = mysqli_fetch_assoc($result)) {
			$out[$row["id"]] = $row;
		}
		echo json_encode($out);
	} else {
		echo "0";
	};
	mysqli_close($conn);
};

$sorting = $_GET["sort"];
switch ($sorting) {
	case 'popular';
	$sorting = 'count DESC';
	$sort_name = 'Популярности';
	break;
	case 'cost-asc';
	$sorting = 'cost ASC';
	$sort_name = 'Возрастанию цены';
	break;
	case 'cost-desc';
	$sorting = 'cost DESC';
	$sort_name = 'Убыванию цены';
	break;
	case 'name-asc';
	$sorting = 'name';
	$sort_name = 'От А до Я';
	break;
	default:
	$sorting = 'id DESC';
	$sort_name = 'Нет сортировки';
	break;
}

Answer 1

[ThunderCat]

$sql = "SELECT * FROM `smirnov` WHERE visible='1' ORDER BY '$sorting' ";

$sorting откуда у вас берется?

Comments

[Dmitry2307]

$sorting подтягивается из url строки с помощью GET запроса
Но проблема не в переменной, т.к. запрос писал так же напрямую
$sql = "SELECT * FROM `smirnov` WHERE visible='1' ORDER BY cost ASC ";
и он не срабатывает, выводятся все товары из БД без сортировки

А если запрос "SELECT * FROM `smirnov` WHERE visible='1' ORDER BY cost ASC " ввожу в phpMyAdmin - все срабатывает

[ThunderCat]

$sorting подтягивается из url строки с помощью GET запроса

Угу, только каким "магическим" способом он оказывается внутри функции? (подсказка - читать про область видимости переменных).

Но проблема не в переменной, т.к. запрос писал так же напрямую

что наводит на мысль что выполняется не эта функция(что не отменяет предыдущего вопроса). сделайте так в своей функции:

$sql = "SELECT * FROM `smirnov` WHERE visible='1' ORDER BY '$sorting' "; 
var_dump($sql);
exit();

и посмотрите что вернется(скорее всего ничего, но по данному огрызку кода это не ясно точно).

Вообще хорошей практикой является делать return из функции значения, а не делать внутри нее echo. Тогда можно сначала посмотреть что там вернула функция, и после этого уже делать echo или выполнять другие операции.

[Dmitry2307]

ThunderCat, спасибо за ответ, с переменной разберусь дополнительно.

Но возникает тогда еще один вопрос?
SQL запрос выполняю без переменной
$sql = "SELECT * FROM `smirnov` WHERE visible='1' ORDER BY cost ASC ";
файл с базы данных приходит с фильтром по цене, но вывод осуществляется так же по id, без применения фильтра
Что может быть? Может ли влиять JSON.parse?

JS файл

function init(){ // загружаю товары на страницу
    $.post(
        "../core/core.php",
        {
            "action" : "loadGoods"
        },
        goodsOut
    );
}

function goodsOut(data){ // показываю товары на странице 
    data = JSON.parse(data);

    var out = '';
    for (var id in data){   
        var gart = data[id].art; if (gart == null) { gart = ""; }
        var gimg = data[id].img; if (gimg == null) { gimg = "nophoto.png"; }
        var gname = data[id].name;
        var gmat = data[id].mat;
        var gmarka = data[id].marka; if (gmarka == null) { gmarka = ""; }
        var gmarka2 = data[id].marka2; if (gmarka2 == null) { gmarka2 = ""; } else { gmarka2 = " / " + gmarka2}
        var gD = data[id].D; if (gD == null) { gD = ""; }
        var gcolor = data[id].color; if (gcolor == null) { gcolor = "-"; }
        var gcost = data[id].cost; 
        
        out += '<div class="col-4"> <div class="catalog__ramka">';
        out += '<span>Код: '+gart+'</span>';
        out += '<img src="../img/'+gimg+'" alt="'+gname+'">';
        out += '<a href="">'+gname+'</a>';
        out += '<p>Материал: '+gmat+'</p>';
        out += '<p>Марка: '+gmarka+gmarka2+'</p>';
        out += '<p>Диаметр: '+gD+' мм</p>';
        out += '<p>Покраска: '+gcolor+'</p>';
        out += '<b>'+gcost+' руб.</b>';
        out += '<button class="add_to_cart" data-art="'+id+'"><i class="fa fa-shopping-cart" aria-hidden="true"></i>В корзину</button>';
        out += '</div></div>';
    }

    $('#main_goods').html(out);
}

[Dmitry2307]

ThunderCat, Просмотрел ответ от JSON.parse - он действительно выдает Json файл без фильтра, можно ли это как то подкорректировать?

[ThunderCat]

Dmitry2307, скорее всего у вас там в качестве ключа идет id, соответственно массив всегда будет выдаваться в порядке роста цифровых ключей.

Answer 2

[AUser0]

Вместо:

function loadGoods() { //функция вывода товаров из БД
  $conn = connect();	
  $sql = "SELECT * FROM `smirnov` WHERE visible='1' ORDER BY '$sorting' ";

сделайте:

function loadGoods($srt) { //функция вывода товаров из БД
  $conn = connect();	
  $sql = "SELECT * FROM `smirnov` WHERE visible='1' ORDER BY {$srt}";

И функцию вызывайте с параметром: loadGoods($sorting)