0

Я тут наговнокодил, но выдаёт ошибку:

Fatal error: Call to a member function execute() on boolean in /site.org/index.php on line 19

Вот моё поделие:

<?php

    $db = new PDO('sqlite:data.db');
    $termin = $_GET['termin'];
    $stmt = $db->query('SELECT * FROM articles WHERE title LIKE %$termin% OR text LIKE %$termin%');
    $stmt->execute();
    $result = $stmt->fetchAll();

?>

<form action="post">
    <input type="text" name="termin" />
    <input type="submit" name="submit" />
</form>
<?php if (empty($termin)): ?>
    <p>Missing term!</p>
<?php elseif (strlen($termin) < 3): ?>
    <p>The term must be at least 3 characters!</p>
<?php elseif ($result < 1): ?>
    <p><?= $termin ?> not found.</p>
<?php else: ?>
    <?php foreach ($result as $row): ?>
        <p>ID: <?= $row['id'] ?></p>
        <p>Date: <?= $row['date'] ?></p>
        <p>Title: <?= $row['title'] ?></p>
        <p>Text: <?= $row['text'] ?></p>
        <p><a href="?id=<?= $row['id'] ?>">More…</a></p>
        <br />
    <?php endforeach ?>
<?php endif ?>
6
  • Потому что $stmt у вас FALSE. А почему оно FALSE, читайте в документации php.net/manual/ru/pdo.query.php
    – andreymal
    11 янв 2019 в 9:32
  • execute - вызывается для подготовленных запросов. query - выполняет запрос сразу. в $stmt лежат уже результат из базы 11 янв 2019 в 9:32
  • Надобно вместо query использовать prepare? А как тогда синтаксически сформулировать запрос?
    – Linne
    11 янв 2019 в 9:33
  • Надобно разобраться, как работает query, почитав документацию, ссылку на которую я дал выше
    – andreymal
    11 янв 2019 в 9:34
  • Хотя вообще лучше действительно использовать prepare (обязательно разобравшись в нём, разумеется)
    – andreymal
    11 янв 2019 в 9:35

2 ответа 2

2

Правильный подход:

// подготавливаем запрос к выполнению
$stmt = $db->prepare('SELECT * FROM articles WHERE title LIKE ? OR text LIKE ?');
// выполняем
$stmt->execute(['%' . $termin . '%', '%' . $termin . '%']);
2

такой вариант, судя по всему, тоже подойдет

$termin = $_GET['termin'];
$sql = 'SELECT * FROM articles WHERE title LIKE :termin OR text LIKE :termin';
$stmt = $db->prepare($sql);
$stmt->execute(['termin' => "%$termin%"]);
$result = $stmt->fetchAll();

Кроме этого, важно стартовать PDO вот таким кодом

$options = array(
    PDO::ATTR_DEFAULT_FETCH_MODE => PDO::FETCH_ASSOC,
    PDO::ATTR_ERRMODE => PDO::ERRMODE_EXCEPTION,
);
$db = new PDO('sqlite:data.db', null, null, $options);

чтобы он мог сообщать об ошибках запроса

4
  • Но вроде же нельзя один и тот же плейсхолдер :termin более одного раза использовать?
    – u_mulder
    11 янв 2019 в 10:26
  • Хотя ман сообщает что "You cannot use a named parameter marker of the same name more than once in a prepared statement, unless emulation mode is on."
    – u_mulder
    11 янв 2019 в 10:28
  • Ну я попробовал, вроде бы в обоих вариантах работает. Но сейчас почитал, лучше его избегать. Судя по всему, нативный режим для SQLite PDO не поддерживает, но вдруг включат 11 янв 2019 в 11:36
  • Спасибо большое за информацию! Для меня это весьма ценно, я нуб почти совсем. И кроме того, даже не подозревал, что ПДО надо начинать таким кодом, теперь везде изменил, где необходимо.
    – Linne
    11 янв 2019 в 11:54

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.