Не понимаю что не так в моём классе для работы с БД?

Question

[Денис Пабло]

<?php


namespace App\Core;


use App\Core\Databases\DB;
use App\Core\Databases\MySQL;
use App\Core\Promises\CorePromise;
use App\Core\Promises\ExtremeCorePromise;
use GuzzleHttp\Psr7\Request;

class Core
{

    /**
     * @var ExtremeCorePromise
     */
    private $promise;

    /**
     * @var DB | null
     */
    private $DB = null;

    public function __construct()
    {
        $CorePromise = new CorePromise();
        $CorePromise->setRequest(
            new Request(
            $_SERVER['REQUEST_METHOD'],
            $_SERVER['HTTP_HOST'],
            getallheaders()
        ));

        $CorePromise->setConfig(new Configuration());

        $this->promise = new ExtremeCorePromise($CorePromise);
        if ($this->promise->getConfig()->getPromise()->getDatabases()->getMySQL()->getEnabled())
        {
            $this->DB = new DB(new MySQL($this));
        }
    }


    /**
     * @return DB
     */
    public function getDB()
    {
        return $this->DB;
    }


    /**
     * @return ExtremeCorePromise
     */
    public function getPromise() : ExtremeCorePromise
    {
        return $this->promise;
    }
}

Код который выводит данные

var_dump($Core->getDB()->getRow('SELECT * FROM users WHERE id= :id'), [
    "id" => 0
]);

Выдаёт вот это:

array(0) { } array(1) { ["id"]=> int(0) }

сама бд

Answer 1

[Максим]

Подозреваю, что ошибка где-то здесь

$CorePromise->setConfig(new Configuration());

$this->promise = new ExtremeCorePromise($CorePromise);
if ($this->promise->getConfig()->getPromise()->getDatabases()->getMySQL()->getEnabled()) {
    $this->DB = new DB(new MySQL($this));
}

Так как вы в качестве конфига передаёте новый экземпляр класса без самих данных. Трудно понять, что у вас не так. Я вижу, что для работы с БД вы используете свои классы. Как всё это реализовано останься только догадываться.

Так же, возможно, что-то не так в фильтрации условия и работы метода:
$Core->getDB()->getRow("...");

Comments

[Денис Пабло]

я забыл добавить ещё один класс, может здесь ещё что-то?

<?php


namespace App\Core\Databases;


use PDO;
use PDOStatement;

class DB
{
    /**
     * @var PDO
     */
    private $connection;

    public function __construct(iDriverDB $driver)
    {
        $this->connection = $driver->getConnection();
    }

    /**
     * @param string $query
     * @param array $params
     * @return bool
     * @deprecated
     */
    public function query(string $query, array $params) : bool
    {
        $stmt = $this->connection->prepare($query);
        $this->bindParams($stmt, $params);
        return $stmt->execute();
    }

    /**
     * @param string $query
     * @param array $params
     * @return bool|PDOStatement
     */
    public function db_query(string $query, array $params = [])
    {
        $stmt = $this->connection->prepare($query);
        if (empty($params))
        {
            $stmt = $this->bindParams($stmt, $params);
        }
        $stmt->execute();
        return $stmt;
    }

    /**
     * @param PDOStatement $statement
     * @param array $params
     * @return PDOStatement
     */
    private function bindParams(PDOStatement $statement , array $params) : PDOStatement
    {
        foreach ($params as $param => $valueParam)
        {
            print "<p>".$param." => ".$valueParam."</p>";
            $statement->bindValue(":".$param, $valueParam);
        }
        return $statement;
    }

    /**
     * @param string $query
     * @param array $params
     * @return mixed
     */
    public function getColumn(string $query, array $params = [])
    {
        return $this->db_query($query, $params)->fetchColumn();
    }

    public function getRow(string $query, array $params = [])
    {
        return $this->db_query($query, $params)->fetchAll(PDO::FETCH_ASSOC);
    }
}

[Максим]

Денис Пабло, попробуйте продебажить каждую функцию и сами поймёте где ошибка. На первый взгляд с телефона ничего особенного найти не могу. Оттестировать работу данного файла выше легко. Просто в методе вызовите сначала без фильтров по параметрам:

public function db_query(string $query, array $params = [])
    {
        $stmt = $this->connection->prepare($query);
        $stmt->execute();
        return $stmt;
    }

А в SQL запросе уберите условие. Идите от простого. А вообще хорошо бы тесты иметь) Но если результат будет такой же, то нужно искать не здесь.

[Денис Пабло]

Максим, Выяснил XD я не в ту функцию параметры вписывал

var_dump($Core->getDB()->getColumn('SELECT name FROM users WHERE id=:id', [
    "id" => 2
]));

[Максим]

Денис Пабло, я тоже сейчас это заметил)) Хотел написать, но вы успели первее) Был бы с не с телефона - думаю, сразу бы заметил))

[Максим]

Денис Пабло, ещё чисто экстатическое предложение. Вы везде используете camelCase, а в методе db_queryПочему то решили сделать snake_case. Я бы сделал dbQuery

[Денис Пабло]

Максим, спасибо за пометку)