Хорошая идея или плохая идея? Использование переменной статического класса для хранения глобального

Question

У меня есть класс, который я использую повсюду в своем коде. Он содержит настройки и другие основные функциональные возможности. Вот что я делаю сейчас, чтобы использовать класс.

$settings = new Settings(); 
$settings->loadSettings(); 

Тогда, когда мне нужно кода в некоторой случайной функции в другом классе, я что-то вроде этого:

function abc() { 
    global $settings; 
    $variable = $settings->a; 
} 

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

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

Settings::$settings = new Settings(); 
Settings::$settings->loadSettings(); 

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

function abc() { 
    $variable = Settings::$settings->a; 
} 

Хорошая идея или плохая идея?

Answer 1

Ну, это, вероятно, улучшение на глобальном уровне, потому что оно решает все уродливые проблемы, связанные с глобальными причинами. Как избавиться от глобального оператора, как правило, хорошо! То, что вы делаете, не отличается от одноэлементного шаблона, хотя оно значительно проще. (См. Раздел «Синглтон» по адресу http://php.net/manual/en/language.oop5.patterns.php для получения дополнительной информации о шаблоне.) Ваше решение почти наверняка прекрасно подходит для ваших целей.

С другой стороны, могут быть лучшие способы достижения той же цели, что и разделить ваш код больше. То есть каждый класс становится более способным к использованию в другом проекте без существенного перекодировки. Один из способов сделать это было бы «впрыснуть» настройку объект в каждый класс:

class ABC { 
    private $settings; 

    public function __construct($settings) { 
     $this->settings = $settings; 
    } 

    public function someMethod() { 
     $variable = $this->settings->a; 
    } 
} 

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

Этот процесс, в котором вы «вводите» объект в другой объект, который зависит от него, называется инъекцией зависимостей. Существуют и другие, более сложные способы сделать это, включая сложные контейнеры. См. http://fabien.potencier.org/article/11/what-is-dependency-injection для интересного набора руководств по этому вопросу. Вероятно, они связаны с вашими текущими потребностями, но могут помочь либо сейчас, либо в будущем.

Answer 2

Кажется, вы ищете Singleton. В основном идея состоит в том, чтобы иметь класс, который имеет открытый статический метод getInstance(), который возвращает экземпляр самого класса. При первом вызове метода он хранит экземпляр в частном свойстве, и все позже он возвращает сохраненный экземпляр. Таким образом, всякий раз, когда вы вызываете Settings :: getInstance(), вы гарантированно получаете копию того же объекта. Затем вы можете сохранить настройки в этом объекте.