Как правильно юзать уже заполненное статичное свойство?

Question

[Иван Иванов]

// где-то в дебрях класса App
self::$db = require_once 'config/db.php';

// КАК ИСПОЛЬЗОВАТЬ?

// 1
$db = App::$db;
self::$instance = new PDO($db['dsn'], $db['username'], $db['passwd'], $db['options']);

// 2
self::$instance = new PDO(App::$db['dsn'], App::$db['username'], App::$db['passwd'], App::$db['options']);

Я не до конца, наверное, понимаю суть статики, но мне (лично) кажется, что нужно юзать 1 варик, потому что хоть мы и не создаём экземпляр App и свойство db заполнено, но во втором варике мы 4 раза дергаем App::db, а не 1.

Есть какая-то разница при таких вариантах обращения к уже подключенному массиву?

Comments

[Дмитрий]

Зачем статика? Подключаем массив конфига, создаём инстанс в каком нибудь bootstrap.php или что в самом начале, и пользуемся прокидывая подключение куда надо

[Иван Иванов]

Дмитрий, в ответе DevMan я класс Db дал, у меня потом в базовой модели используется Db::pdo, Db::exec, мне просто получать данные массива.

[Дмитрий]

Иван Иванов, потом просто передаём объект пдо в модель, и сохраняем в свойство
И так же пользуемся, только через стрелочку

[Иван Иванов]

Дмитрий, а если на каких-то страницах нет работы с БД, то будет висеть в bootstrap (по примеру) инстанс, получится.

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

Answer 1

[DevMan]

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

в архитектурном плане оба варианта - дрянь.

Comments

[Иван Иванов]

константы?

[DevMan]

Иван Иванов, зависит от потребностей.

проблема вашего подхода в том, что app::$db вообще не нужен - подключение нужно создать один раз, сохранить и передавать куда нужно уже готовое.

app::$db имеет право на жизнь только, если содержит уже готовый конект, а не параметры подключения.

это в общем. в частном может быть пулл конектов.

[Иван Иванов]

DevMan,

Мне для этого подойдёт?

<?php


namespace core;


use PDO;

class Db
{
    public static $count_instances = 0;
    public static $count_sql = 0;
    public static $queries = [];
    private static $instance;

    private function __construct()
    {
        // $db = require_once 'config/db.php'; избавляемся от инклуда в этом месте
        self::$instance = new PDO(App::$db['dsn'], App::$db['username'], App::$db['passwd'], App::$db['options']);
    }

    public static function pdo($sql, $params = [], $class = 'stdClass')
    {
        return self::exec($sql, $params)->fetchAll(PDO::FETCH_CLASS, $class);
    }

    public static function exec($sql, $params = [])
    {
        self::$count_sql++;
        self::$queries[] = $sql;
        $stmt = self::getInstance()->prepare($sql);
        $stmt->execute($params);
        return $stmt;
    }

    private static function getInstance()
    {
        if (!self::$instance) {
            self::$count_instances++;
            new self();
        }
        return self::$instance;
    }

    public static function getLastInsertId()
    {
        return self::getInstance()->lastInsertId();
    }

    public static function getServerVersion()
    {
        return self::getInstance()->getAttribute(PDO::ATTR_SERVER_VERSION);
    }
}

[FanatPHP]

Нет, не подойдет

[Иван Иванов]

FanatPHP, константы?

[FanatPHP]

Какие еще "константы"? Для чего? Что ты собрался с ними делать?

[FanatPHP]

я про класс целиком если что, а не про твои страдания с конфигом

[FanatPHP]

а по поводу переменных DevMan уже всё сказал - что в лоб что по лбу.

[Иван Иванов]

FanatPHP,

Какие еще "константы"? Для чего? Что ты собрался с ними делать?

хранить и получать данные для подключения

[Иван Иванов]

FanatPHP,

я про класс целиком если что

и что в нём не так? обычный синглтон под AR)))

[FanatPHP]

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

Если у тебя весь код ООП, то попробуй писать вообще без статических костылей. Это труднее но зато не придется потом переписывать вообще всё.
Если процедурщина - то без разницы, говнякай как придется

[FanatPHP]

Иван Иванов, где транзакции? где выполнение нескольких подготовленных запросов?
но зато есть getServerVersion() :))))

[Иван Иванов]

FanatPHP, у меня App вот такой

<?php


namespace core;


use Error;

class App extends Error
{
    public static $debug;
    public static $routes = [];
    public static $db = [];
    public static $name = 'Название';
    public static $title = 'Описание';

    public static function run($true_false)
    {
        if (is_bool($true_false)) {
            session_start();
            self::$debug = $true_false ? false : true;
            Debug::run();
            self::$routes = require_once 'config/routes.php';
            self::$db = require_once 'config/db.php';
            Router::run();
        } else {
            die('Ошибка запуска!');
        }
    }
}

и в index.php

<?php

use core\App;

chdir(dirname(__DIR__));
spl_autoload_register(function ($class) {
    $file = str_replace('\\', '/', $class) . '.php';
    if (is_file($file)) {
        require_once $file;
    }
});
App::run(false); // false - dev, true - prod

[Иван Иванов]

FanatPHP,

где выполнение нескольких подготовленных запросов?

всё в базовой модели остальное: count, countBy, findAll, findAllBy, findOne, findOneBy, save, delete и т.д....

[FanatPHP]

Выполнение нескольких подготовленных запросов - это когда ты подготовил один раз а потом много раз выполняешь

[Иван Иванов]

FanatPHP, ну я пока что не сталкивался с таким, я понял про что ты, не было пока что такой надобности, да и объект из AR имеет у меня массив первоначальных данных, если изменились свойства-столбцы, то их начальные данные есть в этом массиве, он кстати используется при save() для сохранения только изменённых свойств, а то я видел как свой же айдишник сохраняют люди тупо сохраняя все свойства-столбцы, как-то так...

[Иван Иванов]

Mellorn,

class App extends Error

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

Согласен - эпичненько, но зачем нам лишний класс AppException ;)

// та-тах
throw new App('Вы не ввели или неправильно ввели данные');

// та-тах
try {
    // сюда метод с выбросом
} catch (App $app) {
    echo $app->getMessage();
}

[Иван Иванов]

Mellorn,

установил бы какой-либо из популярных фреймворков

тяжёлые, про фалкон читал, но не вникал сильно

[Vitsliputsli]

Иван Иванов, как уже написали:

class App extends Error - На человеческий переводится как мой App сплошной Error.

Если App наследует класс Error, значит App это разновидность Error. Ваше приложение разновидность ошибки? Причем здесь конструктор и что за класс AppException? Наследование подразумевает родство классов, а не просто объединение произвольных кусков кода. Только так, иначе код превратится в бессвязную мешанину.

установил бы какой-либо из популярных фреймворков
тяжёлые, про фалкон читал, но не вникал сильно

Тяжелые, потому что так говорят? Или у вас highload решение и вы боретесь за доли секунды?

[Иван Иванов]

App это разновидность Error

именно так, он также используется для реализации интерфейса Throwable

что за класс AppException?

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

class AppException extends Error{} // всё, это весь класс

зачем он мне, я App уже отнаследовал, у него нет конструктора, если я в App буду его создавать, то выглядеть он должен будет вот так

public function __construct($message = "", $code = 0, Throwable $previous = null)
{
    parent::__construct($message, $code, $previous);
}

у меня в App только статика

Тяжелые, потому что так говорят?

любой голый фреймворк генерит у меня страницу примерно 0.1 сек, у меня авторизованный скрипт с примерно около десятка запросов генерит страницу за 0.005, да мне и не нужен фв на 50 мегабайт, у меня скрипт весь с графикой около 5-ти мег

[Vitsliputsli]

App это разновидность Error?

именно так, он также используется для реализации интерфейса Throwable

У вас App базовый класс в точке входа, он устанавливает коннект к БД, причем здесь Error?! Вопрос не в том, как вы его используете, а зачем вы так его используете. Принцип single responsibility запрещает подобное надругательство, как смешение всего в одном.
Зачем передавать объект Error в конструктор? Ошибки генерируются путем выбрасывания Exception и соответствующий объект Throwable будет сгенерирован в месте выбрасывания. Что это за божественный класс Error, что вы хотите везде его всунуть?

любой голый фреймворк генерит у меня страницу примерно 0.1 сек, у меня авторизованный скрипт с примерно около десятка запросов генерит страницу за 0.005, да мне и не нужен фв на 50 мегабайт, у меня скрипт весь с графикой около 5-ти мег

пока ваш скрипт ничего не делает, разница огромна, но вы ведь его вероятно будете обогощать функционалом? Фреймворки не панацея, для простого скрипта они не нужны, для высоконагруженных решений - они недопустимые потери. Но в вашем случае в фреймворках можно почерпнуть правильные подходы (неидеальные подходы, может где-то спорные, но хотя бы не откровенно плохой код, идеального вообще в жизни мало).

[Иван Иванов]

У вас App базовый класс в точке входа, он устанавливает коннект к БД, причем здесь Error?

при том, что в точке входа не класс, а статический метод App::run )))

и в том методе нет коннекта к БД, там подключается массив с данными для коннекта и он помещается в статическое свойство self::$db )))

Error тут при том, что App реализует интерфейс Throwable и используется для выброса своих исключений )))

пока ваш скрипт ничего не делает, разница огромна

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

в фреймворках можно почерпнуть правильные подходы

а кто сказал, что они правильные)))

[Иван Иванов]

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

сообщество пользователей всего лишь фреймворка или сообщество программистов?

почему тогда фреймворков множество, если каждый из них написан "правильно" разными программистами с разным подходом?

видимо каждый думает что именно он прав)))

Application - это application, а исключения - это исключения.

статический метод - это статический метод, экземпляр - это экземпляр)))

если вы игнорируете любые советы?

в вопросе нет такого слова - посоветуйте

[DevMan]

Иван Иванов, если та наследуешь свой главный класс от класса ошибок - это уже сигнал.

ООП - это не про "налепили классов как пришлось", а про продуманные связи.

даже имплементация интерфейса, вместо наследования, хоть и смотрелась бы дико, но не на столько.

[Иван Иванов]

DevMan,

App::run(); // запустил скрипт

var_dump(new App()); // не вижу в распечатке ни одного своего статического свойства и метода

вот так и живём, а не "налепили классов как пришлось" ;)

или может быть какие-то разработчики каких-то там фв так не делают и это другие разработчики принимают за эталон?)

или может быть есть чёткое указание в документации по PHP - нельзя класс со статическими свойствами и методами юзать для выброса наследованием, надо наделывать кучу пустых классов и юзать их?

[Иван Иванов]

DevMan,

Раз

<?php


namespace app\models;


use core\App;
use core\base\Model;

class VeteranRiddle extends Model
{
    public $number;
    public $last_number;
    public $last_winner;
    public $prize;

    public function handler(User $user, $number)
    {
        if (!preg_match('~^[1-9]\d{8}$~', $number)) {
            throw new App('Вы не ввели или неправильно ввели данные');
        }
        if ($number != $this->number) {
            if (!$this->last_number) {
                $this->last_number = $number;
                $this->save();
                throw new App('Вы не угадали, но ввели ближайшее число');
            }
            if ($number == $this->last_number) {
                throw new App('Вы не угадали, но повторили ближайшее число');
            }
            if (abs($number - $this->number) < abs($this->last_number - $this->number)) {
                $this->last_number = $number;
                $this->save();
                throw new App('Вы не угадали, но ввели новое ближайшее число');
            }
            throw new App('Вы не угадали, попробуйте ещё раз');
        }
        $this->number = mt_rand(100000000, 999999999);
        $this->last_number = null;
        $this->last_winner = $user->login;
        $this->save();
        $user->gold += $this->prize;
    }
}

Два

<?php


namespace app\controllers;


use app\models\VeteranRiddle;
use core\App;

class ClubController extends AppController
{
    public function __construct($route)
    {
        parent::__construct($route);
        $this->userArea();
    }

    public function indexAction()
    {
        $this->setTitle('Клуб офицеров');
        $user = $this->user->getAfter();
        $this->setVars(compact('user'));
    }

    public function veteranRiddleAction()
    {
        try {
            $veteran_riddle = VeteranRiddle::findOne(1);
            if ($this->formButton('veteran_riddle')) {
                $number = filter_input(INPUT_POST, 'number');
                $veteran_riddle->handler($this->user, $number);
                $prize = $this->numericDeclension($veteran_riddle->prize, ['золото', 'золота', 'золота']);
                $this->alert('success', 'Вы угадали число и получили ' . $prize);
            }
        } catch (App $app) {
            $this->alert('danger', $app->getMessage());
        }
        $this->setTitle('Клуб офицеров: Загадка ветерана');
        $user = $this->user->getAfter();
        $this->setVars(compact('user', 'veteran_riddle'));
    }
}

Какие неумолимые условия запрещают мне использовать тут App для выброса?

[DevMan]

Иван Иванов, никакие, кроме логики.

[Иван Иванов]

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

Answer 2

[FanatPHP]

Никак не использовать.
Учить ООП и писать нормальный код, а не обезьянить кривой yii 1

Comments

[Иван Иванов]

не пользователь фреймворков, не знаю, как там сделано

есть разница: забрать один раз весь массив из статики или 4 раза брать по ключу из статики значение?