Использование реестра через wrapper без синглтона в PHP?

Question

[rumkin]

Насколько уместно использование подобной конструкции для использования реестра:

$___REGISTER = (object) array(); // Создаем StdClass для работы с массивом как с объектом

function &Reg()
{
    return $GLOBALS['___REGISTER'];
}

// Применение

Reg()->config = (object) array();
Reg()->config->host = 'habrahabr.ru';
Reg()->config->admin = '[e-mail]';
Reg()->config->name = 'Хабрахабр';

// ... и т.д. и т.п.

Использую для замены:

Register::set('config', array(/*... */));
$config = Register::get('config');

Какие подводные камни могут быть при использовании такой обертки? Использовали бы вы такой враппер в своих проектах? Где его 100% не нужно использовать?

Comments

[Сергей Береснев]

Чем синглтон не угодил для реестра?

Answer 1

[den1n]

Не использовал бы, т.к. глобальную переменную $___REGISTER можно случайно или намеренно изменить, а потом искать, где «собака порылась».
Если очень хочется использовать stdClass в качестве контейнера в реестре, то я бы предложил поступить так:

class Registry extends stdClass {
    private static $instance;

    private function __construct () {}
    private function __clone () {}

    public static function me ()
    {
        if (self::$instance === null)
            self::$instance = new self;
        return self::$instance;
    }
}

# Использовать так
Registry::me()->config = new stdClass;
Registry::me()->config->host = 'habrahabr.ru';
Registry::me()->config->admin = '[e-mail]';
Registry::me()->config->name = 'Хабрахабр';

Comments

[Arsen]

чтобы было поудобнее использовать можно развить вашу идею:

class stdClassExt extends stdClass {

	private function  __get( $name ) {
		$this->$name = new self;
		return $this->$name;
	}
}

class Reg extends stdClass {
	private static $instance;

	private function __construct () {}
	private function __clone () {}

	public static function me () {
		if ( self::$instance === null ) {
			self::$instance = new self;
		}
		return self::$instance;
	}

	private function  __get( $name ) {
		self::me()->$name = new stdClassExt();
		return self::me()->$name;
	}
}
# теперь не нужно париться с инициализацией
Reg::me()->config->host = 'habrahabr.ru';
Reg::me()->config->admin = '[e-mail]';
Reg::me()->config->name = 'Хабрахабр';

// а также вложенность можно быть любая
Reg::me()->config->localization->lang->abbreviation = 'ru';

print_r( Reg::me()->config );

[Сергей Береснев]

Ну, тут и есть синглтон, хотя авто хотел его почему-то избежать: )

[Алексей Шеин]

А что, теперь стало модно писать extends stdClass? Раньше как-то без этого обходились )

[den1n]

Не модно и не обязательно. Просто, так получилось.

[rumkin]

В данном случае Reg::me() не многим лучше чем Reg::get('config')->localization->…
потому что с помощью Reg() я пытаюсь уйти от использования громоздкой записи «Reg::».

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

Answer 2

[LoneCat]

Вы вполне можете использовать наследие процедурного php для этих целей, а именно — объявление статической переменной функции.

function Reg() {
  static $registry = null;
  if(null === $registry) {
    $registry = new stdClass;
  }
  return $registry;
}

Comments

[ha2bj]

без if:

function Reg() {
    static $registry = null;

    empty($registry) AND $registry = new stdClass;

    return $registry;
}

[LoneCat]

Синтаксически это конечно правильная конструкция, но логически, и большей частью идеологически — это мрак, несмотря на то что AND (&&) можно использовать вне условий — лучше так не делать. Если хочется сокращений — то тогда уж лучше так:

function Reg() {
  static $registry = null;
  return (null === $registry) ? new stdClass : $registry;
}

или в PHP 5.3 так:

function Reg() {
  static $registry = null;
  return $registry ?: new stdClass;
}

[Алексей Шеин]

LoneCat, вы в последнем примере потеряли присвоение экземпляра переменной $registry, у вас всегда новый объект возвращается.

[Алексей Шеин]

Если вы планируете тестировать собственный код (в смысле автоматизированно, например, с помощью PHPUnit), не используйте синглтоны, не говоря уже о глобальных переменных (которые вы привели в пример). Почитайте что такое тестируемый дизайн и принцип Dependency Injection. Если вы читаете по-английски, рекомендую вот эту ссылку misko.hevery.com/code-reviewers-guide/ и презентацию
www.procata.com/talks/phptek-may2007-dependency.pdf
Вот здесь есть описание компонентов Symfony в переводе на русский: madbee.ukr.su/solenko/dependency-injection/

[LoneCat]

Почему потерял? Это сокращенный тернарный оператор, его принцип заключается в том что в случае если переменная не приводится к false — он возвращает значение оной, если приводится — то соот-но вторую часть выражения, иже это аналог

return $registry ? $registry : new stdClass;

null в частности приводится к false, хотя в принципе правильнее будет изменить первую строку на

static $registry = false;

[Ves]

Такое ощущение, что вы не читали вопроса, а сразу выпалили заготовку про «вы просто не знакомы с оператором Элвиса».

Правильный вариант:
function Reg() {
static $registry;
return $registry ?: $registry = new stdClass;
}

> в принципе правильнее будет изменить первую строку на
Вовсе не нужно чего-либо писать дополнительно. Переменная по умолчанию инициализируется null, который отлично приводится к false.

[Алексей Шеин]

все правильно написал.

Answer 3

[Алексей Шеин]

Если вы планируете тестировать собственный код (в смысле автоматизированно, например, с помощью PHPUnit), не используйте синглтоны, не говоря уже о глобальных переменных (которые вы привели в пример). Почитайте что такое тестируемый дизайн и принцип Dependency Injection. Если вы читаете по-английски, рекомендую вот эту ссылку misko.hevery.com/code-reviewers-guide/ и презентацию
www.procata.com/talks/phptek-may2007-dependency.pdf
Вот здесь есть описание компонентов Symfony в переводе на русский: madbee.ukr.su/solenko/dependency-injection/

Comments

[rumkin]

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

[rumkin]

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

    $driver = new MailDriver(Reg()->config->mail);
    $mail = new Mail($driver);

[Алексей Шеин]

Они мне не товарищи, просто недавно наткнулся на этот перевод )
Реестр — это просто глобальный массив в глянцевой объектно-ориентированной обертке, собственно, ваша реализация это и показывает.
Имейте в виду, что он не спасает ни от одной проблемы, связанной с глобальными переменными.
Кстати, код Reg()->config->mail нарушает еще один принцип, называемый «Закон Деметры» (Law of Demeter), также известный как «Не разговаривай с незнакомцами» (Don't talk to strangers), в википедии про него хорошо написано.

[rumkin]

По принципу Деметры и сам реестр является неудачной реализацией хранения каких-либо данных, так как повышает увязку в противовес связанности.