Правильно ли создаю сам себя?

Question

[bedolazhka]

class Db
{
    private static $instance;

    private function __construct($host, $name, $char, $user, $pass, $opt)
    {
        self::$instance = new PDO("mysql:host={$host};dbname={$name};charset={$char}", $user, $pass, $opt);
    }

    public static function setup($host, $name, $char, $user, $pass, $opt = [])
    {
        if (is_null(self::$instance)) {
            // self::$instance = new self($host, $name, $char, $user, $pass, $opt); // так пишут практически во всех примерах
            new self($host, $name, $char, $user, $pass, $opt); // я правильно понял, что self::$instance = тут и не нужно вообще?
        }
        return self::$instance;
    }
}

Answer 1

[FanatPHP]

Не трогал бы ты эту статику.
Целее будешь

Comments

[bedolazhka]

так там мне один раз вызвать Db::setup, просто в примерах дублируют присвоение свойству подключения, в конструкторе-то мы уже определили свойство, зачем в сетапе его присваивать, я так понимаю создаём экземпляр и возвращаем подключение полученное конструктором

[FanatPHP]

Вот и убери этот сетап совсем.
Откуда ты таких примеров набрался. Ты пишешь "везде пишут" но я такого говнокода не видел нигде.
Везде просто создается подключение и дальше пошал запросы делать.

[bedolazhka]

FanatPHP, это типа либа-синглтон так называемый, а в базовой модели получаю $this->connect = Db::setup(...);
и потом юзаю $this->connect для там разных запросов

[FanatPHP]

1. Это не синглтон, а говнотон.
2. В модель надо передавать соединение через конструктор.

[bedolazhka]

FanatPHP, ну как бы так и сделано

class Db
{
    private static $instance;

    private function __construct($host, $name, $char, $user, $pass, $opt)
    {
        self::$instance = new PDO("mysql:host={$host};dbname={$name};charset={$char}", $user, $pass, $opt);
    }

    public static function setup($host, $name, $char, $user, $pass, $opt = [])
    {
        if (is_null(self::$instance)) {
            new self($host, $name, $char, $user, $pass, $opt);
        }
        return self::$instance;
    }
}

abstract class Model
{
    protected $connect;

    public function __construct()
    {
        $this->connect = Db::setup($host, $name, $char, $user, $pass, [
            PDO::ATTR_ERRMODE => PDO::ERRMODE_EXCEPTION,
            PDO::ATTR_DEFAULT_FETCH_MODE => PDO::FETCH_OBJ
        ]);
    }
}

принцип же вроде соблюдён: одно подключение

не? я как-то так синглтон понимаю :)

[bedolazhka]

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

[FanatPHP]

Не понял. А откуда берутся все эти $host и прочее?

[bedolazhka]

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

if ($_SERVER['REMOTE_ADDR'] == '127.0.0.1') {
            $host = 'localhost';
            $name = 'voynushka';
            $char = 'utf8mb4';
            $user = 'mysql';
            $pass = 'mysql';
        } else {
            $host = '';
            $name = '';
            $char = '';
            $user = '';
            $pass = '';
        }
        $this->connect = Db::setup($host, $name, $char, $user, $pass, [
            PDO::ATTR_ERRMODE => PDO::ERRMODE_EXCEPTION,
            PDO::ATTR_DEFAULT_FETCH_MODE => PDO::FETCH_OBJ
        ]);

[FanatPHP]

И это все в конструкторе модели? Ты хорошо подумал?

[bedolazhka]

FanatPHP, это навскидку только начал делать, будет конфиг-файл с массивом параметров подключения инклудится перед $this->connect = Db::setup(), типа

$conf = require_once CORE . '/configs/config_db.php'; // файл с массивом параметров подключения
$this->connect = Db::setup($conf[] .......)

[FanatPHP]

тебе самому не смешно, в каждой модели инклюд файла делать делать? :)

[bedolazhka]

FanatPHP, в базовой только, несколько моделей не будут одновременно вызываться, которые от неё расширяются

[FanatPHP]

Наследуются классы а не объекты.
у тебя довольно странное представление про ООП, но в реальности наследование - это как будто ты руками в каждом наследнике прописал код, который лежит в родителе.

То есть по факту каждый инстанс модели будет инклюдить файл.

Если кратко, то это ужас.

[bedolazhka]

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

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

если не так, то подскажите, как не инклудить при перезагрузке страницы, ну я ссылку нажал вызвал другой контроллер на него обновилась страница

[FanatPHP]

при чем здесь перезагрузка страницы? не о ней речь.

речь про модели. Моделей в обычном приложении создается МНОГО.
Я не знаю, что у тебя за приблуда, но обычно в приложении создаются десятки или сотни моделей. На один вызов страницы.
Но даже если у тебя примитивный круд, коорый строго работает только с однйо моделью, все равно, инклюд и коннект прямо в модели это ужас.
убери ты хоть параметры подключения внутрь класса.

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

[bedolazhka]

FanatPHP, я уже понял примерно что к чему, а так-то да у меня всё по одному.

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

если я, например, создал в контроллере модель и ещё другую модель, то у меня второй инклуд произойдёт?

[FanatPHP]

произойдет, разумеется.
почему он не должен произойти?
В конструкторе написано - сделать инклюд. Вот он и будет делать

[bedolazhka]

FanatPHP, require_once - PHP проверит, включался ли уже данный файл, и если да, не будет включать его еще раз

[FanatPHP]

офигеть какая здравая идея :)
если файл не загрузится то ты свои параметры подключения и не получишь. будет "назло мамке отморожу уши" )))

Ну и вообще, я говорил не конкретно про файл, а про подход в целом. не важно, сработает или нет, сам факт наличия инклюда в конструкторе сотен объектов это не гуд

если уж на то пошло, кто все эти _once - это костыль. ужный только тогда когда у программиста такой бардак в коде, что он не знает что куда загружается. А в нормально спроектированном приложении никаких _once не нужно

[bedolazhka]

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

[FanatPHP]

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

попробкй освоить Симфони сначала - он здорово мозги прочищает.
а потом уже свой шедевр ваяй

[bedolazhka]

FanatPHP, спасибо за "шедевр" )

Answer 2

[gian_tiaga]

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

Comments

[bedolazhka]

во всех примерах в конструкторе свойству self::$instance присваивается подключение И потом в методе setup свойству self::$instance присваивается self::$instance с присвоением подключения

зачем 2 раза свойству self::$instance присваивать, если мы вызываем Db::setup, у нас свойству self::$instance уже конструктором присваивается подключение и мы его возвращаем

вот в чём вопрос: один где-то нацарапал, а остальные размножили по сети годами примеры такие лежат