Как правильно делать запросы к БД?

Question

[aopil]

Я создал Class в котором у меня все функции по работе с БД.

Вот так выглядит:

<?php
class Functions {

    private $DB;

    public function __construct()
    {
        include("config.php");
        try {
             $this->DB = new PDO($DSN, $DB_USER, $DB_PASS, $OPTIONS);
        } catch (PDOException $e) {
             // ошибка при подключении
            file_put_contents(__DIR__ . '/logs/error_connect_DB.txt', date('Y-m-d h:i:s') . 'Connect failed error:' . $e->getMessage() . PHP_EOL, FILE_APPEND);
            die();
        }
    }

    public function __destruct()
    {
        $this->DB = null;
    }

    public function GetDetailsByRequestId($requestId)
    {
        // PDO REQUEST
    }
}
$object = new Functions();
?>

Каждый раз, когда необходимо использование БД я подключаю файл:

include_once("function.php");
                $function = new Functions();

И использую функции в этом файле для работы с БД.

Вопрос: Насколько правильно такое использование БД? или такая конструкция ошибочна и несет какие-то последствия?

Answer 1

[Stalker_RED]

вместо file_put_contents лучше error_log() или выброс исключения.

Каждый раз, когда необходимо использование БД я подключаю файл:

до PSR-4 с описанием автозагрузки вы еще не дошли?

В итоге вы придете к тому, что будет базовый класс БД с подключением, логированием, и еще какими-то общими штуками, а от него будут наследоваться классы МОДЕЛИ, и уже в них будет выборка из конкретных таблиц.

Comments

[aopil]

Stalker_RED получаю иногда ошибки вот такого рода:

[28-Aug-2020 06:40:40 UTC] PHP Warning:  Error while sending QUERY packet. PID=2362 in /var/www/ttt/function.php on line 84
[28-Aug-2020 06:40:41 UTC] PHP Fatal error:  Uncaught exception 'PDOException' with message 'SQLSTATE[HY000]: General error: 2006 MySQL server has gone away' in /var/www/ttt/function.php:84

Может ли это быть из-за такого типа подключения как делаю я? Или скорее всего связаны с чем-то другим?

[Stalker_RED]

aopil, Почему возникает Error while sending QUERY packet в MariaDb?

Answer 2

[FanatPHP]

Я создал Class в котором у меня все функции по работе с БД.

Это очень недальновидное решение.
Я понимаю, что на данном этапе ты думаешь что на сайте будет 3 запроса и ты все три можешь положить в один класс.
Но это не так.
Запросов будет много и их надо разделять по областям ответственности, которые называют в разном контексте моделями или доменами

Поэтому у тебя не должно быть никакого класса Functions
Должен быть класс DB, в котором будет два метода - конструктор, в котором создаётся соединение с БД, и метод для выполнения подготовленного запроса. Например такой

Дальше ты делаешь классы-модели для работы с определенными наборами данных. В самом простом варианте - с одной таблицей в БД.
И передаёшь экземпляр класса DB в конструктор.

file_put_contents и die(); - абсолютно бессмысленные телодвижения
если не писать try..catch, то РНР сделает абсолютно то же самое: залогирует ошибку и остановит выполнение скрипта

Comments

[aopil]

FanatPHP, получается я должен взять и создать файл database.php в котором содержится:

Далее, у меня есть файл index.php, который выглядит вот так:

class Test {
    public function Name($param)
    {
        // где-нибудь тут нужно выполнить запрос к БД.
    }
}
$object = new Test();
$object->Name();

Подскажите пожалуйста, как я должен делать запрос в БД и подключение файла database.phpв файле index.php

[FanatPHP]

ну, начнем с того, что в индексе не должно быть никакого class Test {
код класса должен лежать в другом месте и инлюдиться в индекс

а как делать запрос написано по ссылке:

$db = new DB();
$object = new Test($db);
$object->name();

[FanatPHP]

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

[aopil]

FanatPHP, с чего должен начинаться index.php в таком случае? Я создал там Class и в нем функция, которая запускается по крону.

[FanatPHP]

ну я же написал - с инклюда файлов с кодом использующихся в нем классов
впрочем, если у тебя файл запускается по крону, то он не должен называться index.php
и ты можешь написать свой класс прямо в нем, хотя это некрасиво и не принято

[Stalker_RED]

aopil,

с чего должен начинаться index.php в таком случае? Я создал там Class и в нем функция, которая запускается по крону.

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

Для такого зачастую достаточно заинклудить конфиг, подключение к бд, и можно сразу писать логику (если ее не на 3000 строк, конечно).