Почему не работает функция foreach c PDO?

Question

[Николай]

Здравствуйте! Имеется простой код

# Get array of containers
$arr = file(www.'/names.db');

$checkQ = 'SELECT `name` FROM ' . $tb['table'] . ' WHERE `name`=?';
$check = $dbh->prepare($checkQ);

foreach($arr as $value) {
    echo '<p>' . $value.' :: ';

    $check->execute(['%' . $value . '%']);
    echo $check->rowCount();
}

который по идее должен выводить список строк из файла (в котором одно имя на строчку) а рядышком указывать, сколько совпадений нашлось по базе данных.
Но он не работает. Имя выводит правильно, но вот у каждого имени красуется 0, и только у последнего - 1 (и он действительно есть в базе)
если убрать foreach, и вместо $value подставлять какие-либо значения - всё работает.

В чём может быть проблема?

Comments

[FanatPHP]

Что такое www в этом коде? Константа? Строка? Плод воспаленного воображения?

Answer 1

[Оптимус Пьян]

А зачем тут foreach я чё-то не понял. Сделайте запрос 1 раз, результаты запроса в массив и выведите их из массива циклом уже как вам надо.
Потом при чём тут файл? PDO это прослойка для работы с MySQL (и не только)

$stmt = $db->prepare("SELECT * FROM `table` WHERE `name` = :name");
$stmt->bindParam(':name', $name_file);
$stmt->execute();
$arr = $stmt->fetchAll(PDO::FETCH_ASSOC);
$Num = count($arr);

Чему равен $Num столько у вас и совпадений. Взял этот код из рабочего проекта.

Comments

[Николай]

В смысле причём? При том, что этот код не работает х\
Ничего не понял что вы пишите, мне нужно узнать сколько совпадений в базе на строчку "имя" из файла.

[Николай]

Мне нужно знать сколько у каждого имени, что бы потом с этим работать (если 5 - то одно, если 3 - то другое)

[Оптимус Пьян]

@iNickolay вот этот запрос цифру вам и вернёт 5 так 5, 3 так 3

[Николай]

@marrk2 он возращает только ошибку:
"1064 You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '='Array'' at line 1' in "
Да и сами подумайте, вы массив файла присваиваете к переменной :name.

[Оптимус Пьян]

@iNickolay а, массив. Массив в PDO вы никак не передадите, там надо обёртку писать. Читайте тут про работу с массивами с PDO stackoverflow.com/questions/920353/can-i-bind-an-a...

[Николай]

@marrk2 вы издеваетесь? Мне надо просто сравнить данные массива с базой, а вы мне предлагаете писать обёртку, и какие-то сложные пути. По факту - нужно понять, почему не работает foreach, а не писать огромный костыль для маленького кода.

[Оптимус Пьян]

@iNickolay так это PDO ))) Абстракция над всеми БД. Я тоже поигрался с ним и пошёл на mysqli спокойно ))

[Николай]

c mysqli тоже не пашет, только что проверил. Что за 3.14здец?

[FanatPHP]

Тебе напишу проще. Аффтар, убейся ап стену. Код в ответе несовместим с реальностью.

[Оптимус Пьян]

@FanatPHP код работает баклан, сам въезжай в стройку )))

Answer 2

[FanatPHP]

Прекрасный, прекрасный пример того, что такое эти Q&A сайты.
Ну неужели жалкий ручеек тафика, состоящего из убогих и ламеров всех мастей, стоит того, чтобы терпеть это позорище?

Вопрос, который к ПДО имеет такое же отношение, как я - к балету, и - самое главное - ответы, прекрасно описываемые анекдотом про Вовочку "Я, конечно, не профессор..."

Вопрос, код в котором принципиально не будет работать, поскольку аффтару надо либо крестик снять, либо трусики надеть, и опеределиться - нечеткое у него сравение (и добавить LIKE) или четкое (тогда убрать процентики).

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

Ну и вишенка на торте - совершенно детская причина всех страданий - коварные переводы строк, возвращаемые file() по умолчанию.

Answer 3

[Yakov Akulov]

Переведите
$check = $dbh->prepare($checkQ);
внутрь цикла

Comments

[Оптимус Пьян]

С точки зрения работоспособности да, может и сработает, но что ж вы не предупредите человека запросы внутри цикла это злооооо! )

[Николай]

Не поможет, уже всё это пробовал.

[FanatPHP]

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

Answer 4

[Александр Кубинцев]

У вас запрос неправильный, судя по всему
Исправьте на

$checkQ = 'SELECT `name` FROM ' . $tb['table'] . ' WHERE `name` LIKE ?';

Да и лучше даже вот так:

$checkQ = 'SELECT count(`name`) FROM ' . $tb['table'] . ' WHERE `name` LIKE ?';

Пускай БД считает строки.

Answer 5

[Кирилл Захаров]

судя по процентам, вам name LIKE %$value% надо (ну или name = %$value%)?
тогда запрос скорее всего составлен херово и проценты нужно ставить в prepare, а не в execute

Comments

[Кирилл Захаров]

Ну или как вариант (из-за Array) - $value не всегда строка и лучше сделать var_dump(). Но скорее всего проще нормально сделать бинд и не париться

[FanatPHP]

Открою маленькую тайну: проценты надо ставить именно в execute

[Кирилл Захаров]

@FanatPHP Да, об этом написано в 6 примере php.net/manual/ru/pdo.prepared-statements.php и мне кажется это немного странным, так как с одной стороны, по идее, запрос должен брать значение и делать его шаблоном, а не наоборот, а с другой стороны - не всегда шаблоны вида %val%.
как-то так, хотя и пишут, что ок все stackoverflow.com/questions/4015198/how-to-do-like...

[FanatPHP]

Не надо путать шаблон подготовленного выражения и шаблон поиска по LIKE. Тогда ничего не будет казаться странным. А ещё лучше попробовать поработать с ПДО на практике. Это поможет не позориться перед другими.

[Кирилл Захаров]

@FanatPHP да я какбы год был на яве, а до этого с pdo напрямую дела не имел по разным причинам.
Поэтому мне глубого наплевать на условный позор - человек спросил, а я попытался ответить.

[FanatPHP]

Вот как раз пытаться отвечать на тему, в которой ты ни бельмеса не смыслишь, и есть позор. Безусловный.

[Кирилл Захаров]

@FanatPHP xkcd.com/386

[FanatPHP]

Прекрасное дополнение к моим выводам о ценности, авторитетности и полезности сайтов типа тостер ру.

[Кирилл Захаров]

@FanatPHP Мне кажется, что то, что вы напридумывали себе о тостере - не проблема тостера, ведь так?
Да и думать, что тостер внезапно станет нашим stackoverflow (я так понимаю, речь-то идет именно об этом?) и при этом упрекать всех в том, что их ответы на тостере полная лажа, а люди, задающие простые вопросы "хомячки и бабки-гадалки" - довольно странно, так как одно делает невозможным другое :)

При этом где-то я так или иначе согласен с вами.