Почему возникает ошибка «Call to a member function fetch() on boolean»?

Question

[SteveJ42]

В методе модели класса делал запрос к базе и через цикл расставлял все по ячейкам массива и выходит какая то ошибка 'Uncaught Error: Call to a member function fetch() on boolean'
Искал в сети. пишут что ошибка в OpenServer но мне так не кажется.
Кто сталкивался с такой ошибкой?
Потом использую этот метод в другом контроллере

public static function getProducts($count = self::SHOW_BY_DEFAULT)
    {
        $db = Db::getConnection();
        $productsList = array();

        $result = $db->query('SELECT id, name, price FROM product '
                . 'WHERE status = "1"'
                . 'ORDER BY id DESC '                
                . 'LIMIT ' . $count);

        $i = 0;
        while ($row = $result->fetch()) {
            $productsList[$i]['id'] = $row['id'];
            $productsList[$i]['name'] = $row['name'];
            $productsList[$i]['price'] = $row['price'];
            $i++;
        }

        return $productsList;
    }

Comments

[FanatPHP]

во-первых, чтобы пдо сообщал тебе об ошибках, надо ему сказать об этом при соединении. а не вываливать ошибку прямо на экран, как тебе тут местные гении советуют
Во-вторых, у тебя в запросе SQL инъекция. чтобы её не было, надо использовать подготовленные запросы

[FanatPHP]

и в-третьих, вместо этого пересыпания песочка из куличков в ведерки и обратно, в пдо есть функция которая УЖЕ возвращает тебе массив,
return $result->fetchAll(PDO::FETCH_ASSOC);

[SteveJ42]

FanatPHP, Спасибо!

[FanatPHP]

что "спасибо"?
Ты понял как включить отображение ошибок без идиотского echo $db->error?
Тебе пхп написал в чем твоя ошибка?
Ты понял как избежать инъекции?
Ты понял как возвращать сразу массив?

[FanatPHP]

Ты, похоже, только киваешь "спасибо", но вообще не понимаешь что тебе говорят
Объясняю по буквам
Во-первых, надо поменять код функции Db::getConnection() чтобы она сообщала об ошибках как показано по ссылке. Если ты не умеешь открывать ссылки или не понимаешь что делает функция Db::getConnection();, можешь написать её содержимое здесь, я поправлю
Во-вторых, надо поменять код функции getProducts, вот так

public static function getProducts($count = self::SHOW_BY_DEFAULT)
{
    $db = Db::getConnection();
    $stmt = $db->prepare('SELECT id, name, price FROM product WHERE status = 1 ORDER BY id DESC LIMIT ?');
    $stmt->execute([$count]);
    return $stmt->fetchAll(PDO::FETCH_ASSOC);
}

В-третьих, надо убедиться что пхп показывает тебе ошибки. Хотя бы написать где-нибудь в в начале

ini_set('display_errors',1);
error_reporting(E_ALL);

Потом запустить свой код и посмотреть, КАКАЯ У ТЕБЯ ОШИБКА в запросе.

[SteveJ42]

FanatPHP, В точке входа в систему у меня прописан уже давно

ini_set('display_errors',1);
error_reporting(E_ALL);

Честно в PDO подробно не разбирался поэтому могу не понимать. в методе getConnection у меня просто путь к массиву с данным для соединения и подключение PDO по данным этого массива.
какую строку кода нужно добавить чтобы PDO показывал свои ошибки??

[FanatPHP]

SteveJ42, просто покажи код getConnection()

Answer 1

[Иван]

public static function getProducts($count = self::SHOW_BY_DEFAULT)
    {
        $db = Db::getConnection();
        $productsList = array();

        $st= $db->prepare('SELECT id, name, price FROM product '
                . 'WHERE status = "1"'
                . 'ORDER BY id DESC '                
                . 'LIMIT ' . $count);
        if ($st && $st->execute()) {
            $i = 0;
            while ($row = $st->fetch()) {
                $productsList[$i]['id'] = $row['id'];
                $productsList[$i]['name'] = $row['name'];
                $productsList[$i]['price'] = $row['price'];
                $i++;
            }
        }
        return $productsList;
    }

Comments

[Северное Сияние]

Пустой массив != ошибке
Пустой массив - это конкретное значение - товаров нет. А товары есть.

[SteveJ42]

FanatPHP объясните пожалуйста как лучше всего делать!

[FanatPHP]

SteveJ42, но я же все уже написал. Ты читал вообще?

Answer 2

[Дмитрий]

Ошибка очень красноречивая. Она означает, что вы пытаетесь вызвать метод fetch у значения типа Boolean, это означает что переменная result содержит значение типа boolean, то есть запрос вернул false

Чтобы этой ошибки не было, нужно писать

if ($result = $db->query('SELECT id, name, price FROM product '
                . 'WHERE status = "1"'
                . 'ORDER BY id DESC '                
                . 'LIMIT ' . $count)) {
    // твой код
} else {
    else echo($db->error);
}

Comments

[SteveJ42]

Дмитрий, а как работает условие которое вы написали?

[Дмитрий]

SteveJ42, Если result получает значение, отличное от false выполняется тот код, который вы хотите выполнить, в противном случае выполняется блок else, где вы обрабатываете ошибку

[FanatPHP]

SteveJ42, никак не работает, это нерабочий код

Answer 3

[Северное Сияние]

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

public static function getProducts($count = self::SHOW_BY_DEFAULT)
{
    $db = Db::getConnection();
    $result = $db->query('SELECT ....');

    if (!$result) {
        throw new \RuntimeException('SQL query error: ' . $db->error());
    }

    $productsList = [];
    while ($row = $result->fetch()) {
        // нет ни малейшего смысла перегонять из пустого в порожнее,
        // т.е. из одного массива в другой
        $productsList[] = $row;
    }

    return $productsList;
}

Comments

[SteveJ42]

Северное Сияние а чем собственно сама ошибка запроса?