Что не так тут?

Question

[Kristina8787]

Доброго дня товарищи. У меня загвоздка, есть класс Db , в нем прописано конструктором подключение , оно осуществляется, ошибки при вардампе не выдает, делаю вот такой запрос с главной страницы

require __DIR__.'/autoload.php';


$dbh = new \App\Db();
$sql = 'SELECT * FROM about';
$article = $dbh->query($sql,[]);
var_dump($article);die;

вот сам класс Db из которого берется метод запроса к БД

namespace App;

class Db
{
  protected $dbh;

  public function __construct()
  {
    $config = (include __DIR__.'/../../config.php')['db'];
    $this->dbh = new \PDO('mysql:host'.$config['host'].';dbname='.$config['dbname'], $config['login'], $config['password']);
  }

  public function execute(string $sql)
  {
    $sth = $this->dbh->prepare($sql);
    return $sth->execute();
  }

  public function query(string $sql, array $data)
  {
    $sth = $this->dbh->prepare($sql);
        if ($sth->execute($data)) {
            return $sth->fetchAll();
        } else {
            return false;
        }

  }
}

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

Comments

[galaxy]

ошибки отключены (error_reporting, display_errors)?

[FanatPHP]

На будущее, вопросам надо давать осмысленные заголовки

[Kristina8787]

galaxy, все включены

[Kristina8787]

FanatPHP, хорошо, просто торопилась

Answer 1

[FanatPHP]

Ну так ты в своем классе никому не сказала выводить ошибки. Вот он и молчит.

Вот тебе нормальный класс.

class Db
{
    public $dbh;

    public function __construct()
    {
        $config = (include __DIR__.'/../../config.php')['db'];
        $options = [
            \PDO::ATTR_ERRMODE            => \PDO::ERRMODE_EXCEPTION,
            \PDO::ATTR_DEFAULT_FETCH_MODE => \PDO::FETCH_ASSOC,
            \PDO::ATTR_EMULATE_PREPARES   => false,
        ];
        $dsn = "mysql:host=$config[host];dbname=$config[dbname];charset=utf8mb4";
        $this->dbh = new \PDO($dsn, $config['login'], $config['password'], $options);
    }

    public function query(string $sql, array $data = [])
    {
        $sth = $this->dbh->prepare($sql);
        $sth->execute($data);
        return $sth;
    }
    public function getAll(string $sql, array $data = [])
    {
        return $this->query($sql, $data)->fetchAll();
    }
}

- dbh сделана пуличной. lastInsertId() ты откуда будешь доставать?
- добавлено включение информирования об ошибках
- добавлена поддержка LIMIT
- добавлено выставление кодировки, чтобы потом не было вопросов "ой у меня крокозябры!!"
- убрана бессмысленная функция execute
- функция query сделана осмысленной и универсальной
- параметр $data сделан опциональным
- добавлена функция для получения массива строк

И запомни - создавать его надо один раз на всё приложение.

$dbh = new \App\Db();
$about = $dbh->getAll('SELECT * FROM about');
var_dump($about);die;

Comments

[Kristina8787]

у меня по заданию нужно было эти две функции, я так поняла первая чтобы добавлять,удалять,обновлять в БД данные,а вторая чисто выборку данных делать. . Но тот код что у меня прекрасно работал в предыдущем задании , почему сейчас не работает разобраться не могу. Безусловно ваш вариант лучше, но инфу об ошибках как включать я еще не знаю, и в задании этого нет, а где LIMIT добавлен не поняла,я так поняла в option.

[FanatPHP]

вот у тебя и есть две функции. первая выполняет любые запросы, а вторая чисто выборку делать. Что не так?
твоя execute была, мягко говоря, дурацкая. какой смысл делать prepare, если в execute ничего не
передаётся?

Что за задание? Откуда?
В любом случае делать надо нормально, а не только то, что в задании написано.

Что в итоге? Какая ошибка выводится?

[Kristina8787]

FanatPHP, с вашим кодом никакой ошибки нет, выборка дается в массиве. Сейчас мануал читаю по ошибкам,чтобы в будущем глупыми вопросами не докучать. Задание с курсов php pr-of-it . Суть метода execute была просто узнать тру или фолс при выполнении добавления или удаления записей в таблице.

[FanatPHP]

никогда не нужно узнавать, тру или фолс. это бессмысленная информация.
вместо этого функция должна возвращать $sth, чтобы получить сопутствующие запросу данные, например количество удаленных строк или же сами строки в случае запроса селект
удаление или добавление записей в таблице надо делать с помощью плейсхолдеров, а execute их не поддерживает.

Мануал по ошибкам читать не надо, там много всякой ерунды написано. Лучше докучать. Я очень люблю глупые вопросы и очень не люблю вороватых преподавателей, которые берут деньги но сами программирования не знают. В оличие от них я понимаю, что делает кажда строчка в моем коде, и могу это объяснить.

[FanatPHP]

Но почему раньше не работало - загадка.
а - дошло. в DSN знак = был пропущен.

[Kristina8787]

FanatPHP, я только код внутри $option слабо поняла, а в остальном прекрасно читабельно и доведено до ума. Преподователь толковый, я с торрента скачивала,вместе с ДЗ там лежит, только от этого препода научилась читать код более менее. А так три месяца учила у других, в пустую вообще было. Могу задание само скинуть если интересно посмотреть

[FanatPHP]

Да интересно. [e-mail]

в опциях все интуитивно же. ну кроме эмуляции
\PDO::ATTR_ERRMODE => \PDO::ERRMODE_EXCEPTION, - сообщать об ошибках
\PDO::ATTR_DEFAULT_FETCH_MODE => \PDO::FETCH_ASSOC, - возвращать ассоциативный массив по умолчанию
\PDO::ATTR_EMULATE_PREPARES => false, - в LIMIT можно использовать плейсхолдеры, LIMIT ?,?. Без этого будет ошибка

[Kristina8787]

FanatPHP, ага, нашла) вот постоянно так, пять минут пишешь код, пол дня ошибку ищешь(

[Kristina8787]

FanatPHP, на счет LIMIT ничего не поняла(

[FanatPHP]

Скажи этому преподавателю, что он профнепригоден
Можешь сослаться на меня.

[FanatPHP]

$about = $dbh->getAll('SELECT * FROM about LIMIT ?, ?', [$limit, $start]);

Без \PDO::ATTR_EMULATE_PREPARES => false будет выдавать ошибку.

[Kristina8787]

FanatPHP, я его знать не знаю) я с торрента качала) а почему так критично? я второй курс тоже проходила, но с первого раза создала только полочки в голове,а вот второй раз прохожу и полочки заполняются уже осознанными знаниями.

[FanatPHP]

Критично то, понимает ли человек то о чём говорит, или просто как попугай повторяет заученные фразы.
делать так чтобы query принимала $sql и $data, а execute - только $sql это непонимание.
отсутствие информации о том, как увидеть ошибки - это критично
работа программиста всегда проходит в режиме получения обратной связи от программы. Чтобы не ходить вот так уокруг да около, не зная в чем проблема.

[Kristina8787]

FanatPHP, ближе к концу второго курса за исключения тема идет

[FanatPHP]

я могу заранее сказать что там написана 100% туфта.
исключения не надо изучать. они работают сами по себе, и лучше всего если не лазить в них руками.

и речь не про исключения а про ошибки вообще. исключение это просто удобный вариант представления ошибки.

[Kristina8787]

FanatPHP, то есть то что в $option это все что надо чтобы отслеживать ошибки PDO ?

[FanatPHP]

я выше писал про то что в $option

[Kristina8787]

FanatPHP, я поняла, я имею ввиду помимо этого еще отслеживания ошибок в PDO есть?

[FanatPHP]

помимо этого ничего не нужно

[FanatPHP]

в РНР у целом надо как говорилось выше error_reporting и при разработке - display_errors

[Kristina8787]

FanatPHP, а display_startup_errors?

[FanatPHP]

я никогда её не ставлю. если пхп нормально настроен то она не нужна.
А если ненормально то все равно в логах смотреть

[Kristina8787]

FanatPHP, я тут подсмотрела в одном из решений последнего задания что я вам показала метод запроса через класс обьекта, я его полностью поняла, он не сложный, но не понимаю как к этому методу автор пришел, просто взять и увидеть этот метод и понять его это одно,а вот его придумать к применению другое, может вы мне обьясните?

public function query(string $sql, array $data = [], $class)
    {
        $sth = $this->dbh->prepare($sql);
        $sth->execute($data);
        $dbArray = $sth->fetchAll();

        $ret = [];
        foreach ($dbArray as $row){
          $item = new $class;
          foreach ($row as $key => $value){
            if (is_numeric($key)){
              continue;
            }
            $item->$key = $value;
          }
          $ret[] = $item;
        }
        return $ret;
    }

[FanatPHP]

он пришел к этому коду от неграмотности. В ПДО уже есть всё, что он старательно выписывыает вручную

public function getObjects(string $sql, array $data = [], $class = 'stdClass')
{
    return $this->query($sql, $data)->fetchAll(PDO::FETCH_CLASS, $class);
}

[Kristina8787]

FanatPHP, query в этом методе этот метод опеределенный до этого?

public function query(string $sql, array $data = [])
    {
        $sth = $this->dbh->prepare($sql);
        $sth->execute($data);
        return $sth;
    }

[FanatPHP]

Да, разумеется.
Главная причина, по которой мы используем объекты - это не чтобы выглядеть круто, а чтобы не писать один и тот же код по 800 раз.
поэтому код выполнения запроса мы пишем в функцию, и потом все время используем её в других функциях.

[FanatPHP]

исправил опечатку в коде

[Kristina8787]

FanatPHP, поменяла, получилось, только пару моментов пришлось исправить

public function getObjects(string $sql, array $data = [], $class = 'stdClass')
{
    return $this->query($sql, $data)->fetchAll(PDO::PDO::FETCH_CLASS, $class);
}

PDO два раза написано, и \ перед ним не стоит,из за которого прога его за класс считать начинает.
А как на свой сайт транслировать расписание к примеру киноафишы с другого сайта?

[FanatPHP]

написать в адресной строке браузера как на свой сайт транслировать расписание киноафиши

[Kristina8787]

FanatPHP, да мне гугл выдает только афишы всякие, а по програмированию пока ниодной ссылки

[FanatPHP]

значит надо менять поиск добавлять ключевые слова.

[Kristina8787]

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