Почему не работает выборка WHERE в SQL запросе?

Question

[LenaOlegovna]

схожу с ума, не понимаю почему не работает выборка WHERE в SQL запросе, помогите найти ошибку, по моему запросу выводит все записи из таблицы

$cat = intval($_GET["cat"]);
$sql = "SELECT * FROM tour WHERE `category` = $cat ";
$query = $pdo->query($sql); 
$tours = $query->fetchAll(PDO::FETCH_ASSOC);

Заранее спасибо за любую подсказку

Comments

[Дмитрий Кузнецов]

вроде не нужно указывать кавычки столбцу, по которому делаете сортировку.

Попробуйте просто:

$sql = "SELECT * FROM tour WHERE category = " . $cat;

И вообще такие манипуляции с сырым SQL слишком опасны. Почитайте про SQL-инъекции.

[Akina]

Дмитрий Кузнецов,

вроде не нужно указывать кавычки столбцу, по которому делаете сортировку.

Извините, но это откровенная дезинформация. Квотирование имён объектов никак не связано с их типом и местоположением, и определяется совершенно иными факторами.

[Akina]

помогите найти ошибку,

Каков тип поля category?
Каково значение $cat?
Чему равно значение $sql?

[LenaOlegovna]

тип поля category - varchar(255)
значения слова

[LenaOlegovna]

Дмитрий Кузнецов, также выводит все категории

[Akina]

LenaOlegovna, Вы понимаете разницу между "значение" и "назначение, смысл"?

тип поля category - varchar(255)

Тогда вставляемый литерал следует сделать строковым:

$sql = "SELECT * FROM tour WHERE `category` = '$cat' ";

Но вообще такой способ - прямое приглашение к SQL injection. Используйте параметризованные запросы.

[LenaOlegovna]

Akina, так вообще ничего не выводит

[Akina]

Ну так, может, Вы всё же соизволите показать значения, которые содержатся в переменных $cat и $sql? Или мне предлагается высматривать их в ХШ?

[FanatPHP]

Чтобы не сходить с ума, надо не разглядывать код. И показывать его окружающим тоже нет смысла. Это не новая сумочка.

Чтобы понять, что не так с кодом, его ЗАПУСКАЮТ. И смотрят - как выглядит запрос, какие данные лежат в базе.

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

В любом случае, надо выполнять какие-то действия, высказывать предположения и проверять их, писать код и выполнять его. Банальная исследовательская работа.

Answer 1

[ThunderCat]

первая подсказка - использовать подготовленные выражения
вторая подсказка - вывести на экран ваш запрос, скопировать, вставить в консоль sql / пхпмайадмин и выполнить.

выводит все записи из таблицы

Могу предположить что данное поле не является уникальным значением и ничего удивительного что вы получаете ВСЕ записи, где категория = какому-то номеру.