PHP область видимости

Question

[daMage]

Есть система роутов. Каждой странице отвечает определенная функция и в этих функциях нужно иметь доступ к глобальной переменной: допустим, драйвер работы с бд - PDO. Писать global $var, передавать как параметр или определять переменную в теле функции не хочется. Можно было бы написать обертку и вызывать методы класса как статические, но меня вполне устраивает стандартный интерфейс класса PDO, не хотелось бы плодить код.

Answer 1

[Кирилл Платонов]

Подойдет шаблон Singleton + Registry.
Почитать здесь.

Comments

[Masterme]

разумеется, тут синглтон, ёба. вопрос в том, как к нему обращаться

[VovanZ]

@Masterme, через getInstance. А как ещё можно обращаться к синглтону?

[Masterme]

@VovanZ синглтон - это просто объект, существующий в единственном экземпляре. можно поместить его в глобальную переменную и обращаться по имени (захардкодить), либо через имя класса и getInstance, либо через глобальную функцию, возвращающую инстанс (что по сути одно и то же), либо через геттер (см. комментарий от @omun), либо иметь ассоциативный массив {название_таблицы => инстанс_драйвера_таблицы}, либо как-то ещё.

[VovanZ]

@Masterme, "Одиночка (англ. Singleton) — порождающий шаблон проектирования, гарантирующий что в однопоточном приложении будет единственный экземпляр класса с глобальной точкой доступа." (c) Вики

Это не "просто объект, существующий в единственном экземпляре".
Это шаблон, который предполагает:
1. Механизм порождения этого самого единтственного объекта.
2. Механизм контроля его единственности, который не даст создать больше одного объекта.
3. "Глобальную точку доступа", т. е. механизм получения этого самого единственного объекта.

[Dmitry]

@VovanZ поправь меня, если не так.
Контроль единственности синглтоном есть bad practice!

[Masterme]

@VovanZ 1 подразумевается, 2 можно запретить (но зачем?) через приватный конструктор, 3 означает либо "давайте везде захардкодим имя класса" либо "давайте, чтоб не хардкодить везде имя класса, натыкаем везде функций-оболочек"

[VovanZ]

@omun, в данном случае, мне это кажется меньшим злом =)
Твоё решение, основано на том, что "все функции в классах и у них есть общий предок", но не факт, что в данном случае это так. Моё решение более универсально.

@Masterme, нет ничего плохого в том, чтобы "хардкодить" им класса, если это фундаментальная часть приложения (такая, как объект доступа к БД).

Answer 2

[Dmitry]

Если все функции в классах и у них есть общий предок, то можно в него геттером запихать драйвер и в потомках использовать.

Comments

[Masterme]

да, я так и делаю
в контексте контроллера:
$this->Model_name->blabla();

[Masterme]

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

Answer 3

[egor_nullptr]

function give_me_pdo()
{
    static $pdo;
    if (!$pdo) {
        $pdo = new PDO('mysql:host=localhost;dbname=test', 'username', 'password');
    };
    return $pdo;
}
....
function handler_for_some_page()
{
    $db = give_me_pdo();
}

Answer 4

[Матвей Правосудов]

Кложуры с указанием контекста.

$pdo = new PDO;
Route::get ('/', function () use ($pdo) {
    $pdo->set();
});

Comments

[Dmitry]

1. Судя по всему вы запятую после '/' пропустили
2. Это не примеси, а анонимные функции с указанием контекста

[Матвей Правосудов]

@omun да, спасибо.

Answer 5

[VovanZ]

Я думаю, стоит написать обертку-синглтон.

Comments

[Dmitry]

...и получи лишнюю связность классов в подарок...

[Кирилл Платонов]

@omun глобальные переменные еще хуже. Предложите альтернативу, если отрицаете решение.

[Dmitry]

@BloodyHistory прям в кассу - goo.gl/QzxNmE

Answer 6

[Александр]

Вам надо почитать про шаблоны проектирования, например это.
Есть несколько вариантов избавления от глобальных переменных, один из них Service Locator, который по сути является тем, что было в первом ответе, то есть подвидом шаблона Registry. Делать самописный Dependency Injection Container наверное неразумно.

Ключевое отличие от использования глобальных переменных в том, что это гибче в плане управления и как следствии тестировании, когда может понадобиться эмуляция какого-то глобального сервиса

Answer 7

[sasha]

объяви эту переменную прям в роуте, а вообще это можно сделать хоть как хош

Answer 8

[Станислав]

Для начала тебе будет достаточно https://toster.ru/q/55530#answer_202248.

А вообще, погугли на тему инверсии зависимостей (IoC). Оно полезно.

Comments

[Masterme]

а ты сам погугли и предложи готовый вариант к прочтению

[Станислав]

@Masterme, а ты перейди на сельдереевую диету и заведи енота

[VovanZ]

@korchasa, жаль коммент плюсануть нельзя =)

Answer 9

[daMage]

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

function($handler)
function() {
  global $handler;
}
function() {
  $handler = get_instance();
}

только потому, что это занимает лишнее место в роуте. Хочется писать просто:

function() {
  $query = PDO::prepare();
}

без всяких предварительных записей.
Раньше я использовал обертку для PDO, но там выполнялись довольно тривиальные вещи. Когда же появилась необходимость писать более сложные запросы я его отбросил.
Желательно всё так и оставить:

DB::init('data');
function page1() {
  $query = DB::query();
}
DB::close();

Не знаю, может это не правильный вариант, повторюсь - я новичек.

Comments

[daMage]

Вариант с классом и наследованием понравился. Такое вроде практикуют во фреймворках. Во всяком случае, в CodeIgniter видел. Я бы использовал, но приложение небольшое и все функции лежат вне классов. А вот use ($var) очень понравился. Если честно, хотел именно этого, только вот не знаю как это применить к колбекам. То есть, чтобы не копировать дял каждой функции, а сделать на уровне роутера, при вызове call_user_func_array.