Почему PHP PDO возвращает только 1 строку?

Question

[Андрей Корехов]

имеется код:

$sql = "SELECT * FROM request_journal WHERE addres = '$this->house' AND kv = '$this->flat' ORDER BY date DESC";
        $queryResult = $this->db->query($sql);
        $current = $queryResult->fetchAll(PDO::FETCH_ASSOC);

SQL строка после замены переменных получается такой
SELECT * FROM request_journal WHERE addres = 'Ленина 63-2' AND kv = '60' ORDER BY date DESC

Если забиваю ее в консоль бд получаю несколько строк результатов
Однако в скрипте возвращается только 1 строка.

С чем данное может быть связано?

Comments

[alexalexes]

Во-первых, никогда не вставляйте переменные внутрь запроса соединением строк, особенно, если они поступают от пользователя и никак не заэкранированы. Это дорога к sql инъекциям, лучше сразу отказаться от такой практики. Безопасно и правильный путь для обучения - это сразу научиться использовать пару функций prepare -> execute для выполнения запроса: https://www.php.net/manual/ru/pdo.prepare.php
Во-вторых, как выводите содержимое $current?

[runapa]

Заменить переменные не в консоли пробовали ?

[Егор]

Dump $sql сделайте.
Правильно, конечно, сделать prepared statements

[Андрей Корехов]

Егор,
Дамп sql - SELECT * FROM request_journal WHERE addres = 'Ленина 63-2' AND kv = '60' ORDER BY date DESC

попробовал с тем же запросом сделать pg_query($conn, $sql) в результат выпали все строки попавшие под параметры.
Где то в PDO у меня неправильные настройки если это вообще возможно.

[Андрей Корехов]

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

[Андрей Корехов]

alexalexes, содержимое вывожу банальным var_dump.
тут данные проверяются на этапе ввода через switch (специфика задачи =) ) и любое не совпадение с case прерывает скрипт.

Но вы правы надо в привычку вводить через prepare -> execute

Answer 1

[Евгений]

<?php
$q = $db->prepare("SELECT * FROM request_journal WHERE addres = ? AND kv = ? ORDER BY date DESC");
$q->execute([$this->house, $this->flat]);
$data = $q->fetchAll();
print_r($data);

Если это выведет одну строку, то там одна строка :)

Comments

[Андрей Корехов]

Вот в таком виде вывело весь результат.
Спасибо.

Хотя вопрос почему без prepare возвращает только одну строку все равно открытым остается )

[Pro_Code]

Prepared request(подготовленные запросы) - это правильно))

[Pro_Code]

Андрей Корехов, работайте с плейсхолдерами(они же подготовленные запросы), если хотите защититься от sql-инекций. Евгений продемонстрировал правильный подход...

[Евгений]

Андрей Корехов, не знаю, у меня и ваш код выдает несколько записей, что в mysql, что в postgres

Answer 2

[FanatPHP]

Если ПДО возвращает одну строку, это значит что запрос нашел строку.
Если запрос должен вернуть больше, то проблема либо во входящих данных, либо в способе, которым ты проверяешь количество строк.
Если волшебным образом всё исправилось, это значит что исправились входящие данные или способ которым ты проверяешь количество строк.

Ваш К.О.