Какой выбрать вариант избавления от глобальных переменных в ООП PHP приложении?

Question

[Brainbao]

1. Есть основной движок, написанный в процедурном стиле, которым занимаются другие разработчики. Он изобилует глобальными переменными.
2. Есть мое дополнение для этого движка, которое я стараюсь писать в ООП стиле.
Вопрос: чтобы не писать в каждом методе каждого класса global $variable, $variable1 и т. д., какой из способов оптимальнее:
1) Сделать трейт и добавлять его в классы, в которых требуется доступ к глобальным переменным, с подобным методом:

public function &__get($name)
	{
		return $GLOBALS[$name];
	}

2) Или в каждом методе добираться до глобальных переменных с помощью:

$variable = $GLOBALS['variable'];
$variable1 = $GLOBALS['variable1'];
// и т. д.

3) Или реализовать простенький DJ-контейнер и добавить туда $GLOBALS в качестве зависимости, чтобы в нужных местах обращаться к ячейке контейнера?
4) Ваши варианты?

Comments

[Adamos]

Вы не пробовали задуматься, как потом будете избавлять этот код от привязки к глобальным переменным, если вдруг движок исправится и перейдет от говнокода к best practices? Судя по варианту 1 - даже не пробовали...

[paldraken]

Как вариант используя DI инжектить нужные переменные в конструктор своих классов которые вы пишете в ООП стиле. (при условии что их нужно только "читать")
Если их надо еще записывать, то наверно делать это через отдельный сервис.

[Brainbao]

paldraken, да, некоторые надо записывать. Пока остановился на трейте с двумя методами:

public function global(string $name)
{
	return $GLOBALS[$name] ?? false;
}

public function setGlobal(string $name, $value)
{
	$GLOBALS[$name] = $value;
}

[paldraken]

Brainbao, по сервисам я имел ввиду что-то типа такого.

<?php
class MyOopClass
{
    public function __construct(
        private LegacyGlobalVars $globalVars
    ) { }
    public function method()
    {
        $this->globalVars->getVar1();
    }
}

interface LegacyGlobalVars {
    public function getVar1(): string;
    
    public function setVar1(string $var1);

    public function getVar2(): string;

    public function setVar2(string $var1);
}
class LegacyGlobalVarsImpl implements LegacyGlobalVars {
    public function getVar1(): string
    {
        return $GLOBALS['var1'];
    }

    public function setVar1(string $var1)
    {
        $GLOBALS['var1'] = $var1; 
    }
    ...
}

Работать с этими переменными через какую то абстракцию. Если это возможно то логически их разделить на несколько Интерфейсов и инжектить раздельно. Тогда сразу будет понятно, от чего конкретный класс зависит и при рефакторинге избавляться от них постепенно

Answer 1

[FanatPHP]

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

В первом случае выкинуть весь этот ад и делать по-человечески, передавая переменные в конструктор/методы в качестве параметров.