1. PHP / Говнокод #9807

    +162

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    class Program implements ProgramProvider {
        
        public static function getInstance() {
                return new Program();
        }
}

    Гениальная реализация singleton!

    Запостил: viktor_poltorak, 29 Марта 2012

    Комментарии (29) RSS

    • ava VolDroN 29.03.2012 21:25 # 0

      > return new Program();
      разве будет работать?)
      Ответить

      • ava mindfreakthemon 29.03.2012 22:52 # +2

        о, ещё как
        Ответить

      • ava Steve_Brown 30.03.2012 10:32 # +1

        Ну так а не вызывай getInstance несколько раз! Вызови один раз, положи в глобальную переменную... с синглтонами работать не умеешь, что ли?

        :)
        Ответить

    • ava Lure Of Chaos 30.03.2012 00:12 # +4

      а кто сказал, что это должен быть singleton?
      Ответить

      • ava guest 30.03.2012 06:13 # −1

        Здравый смысл и getInstance?
        Ответить

      • ava Vasiliy 30.03.2012 13:49 # −1

        Как говорят американцы если что то выглядит как собака, лает как собака то это собака. А это сингелтон.
        Ответить

    • ava Bert 30.03.2012 08:06 # 0 

      class Program
{
	private static $_instance;

	public static function getInstance()
	{
		if(self::$_instance === null){
			// ...
			self::$_instance = new self();
		}
		return self::$_instance;
	}
}
      Ответить

      • ava Lure Of Chaos 30.03.2012 09:20 # +1

        а наследники этого синглтона таковыми уже не будут
        Ответить

      • ava roman-kashitsyn 30.03.2012 09:25 # +2

        Теперь внимание: что помешает написать new Program() там, где я захочу?
        Ответить

        • ava Bert 30.03.2012 10:18 # +1

          приватный конструктор например
          Ответить

          • ava roman-kashitsyn 30.03.2012 13:35 # 0

            На него и был намёк (не вижу в сниппете приватного конструктора). Он же нужен для предотвращения наследования (которое с синглтонами ну никак не сочетается).
            Ответить

            • ava Bert 30.03.2012 13:56 # 0

              для предотвращения наследования используется слово final
              Ответить

              • ava roman-kashitsyn 30.03.2012 14:13 # 0

                Oh, really? :)
                От использования приватного конструктора эффект будет тот же самый
                Ответить

                • ava Bert 30.03.2012 14:23 # 0

                  эффект далеко не тот же самый.
                  и если увидишь ебанутого, который зачем-то использует приватный конструктор вместо final, скажи, что он делает неправильно
                  Ответить

                  • ava roman-kashitsyn 30.03.2012 15:12 # 0

                    Прятать все конструкторы приходится тогда, когда класс сам осуществляет контроль над созданием своих экземпляров (например, имеет фабрику ala EnumSet в java). Есть приятный? побочный эффект - если все конструкторы приватные, то унаследоваться от этого класса нельзя. final без приватного конструктора означает, что детали реализации критичны для приложения (к примеру, это верно для String, Integer и т.п.), но клиент волен сам создавать инстансы класса.

                    Т.е. имеют смысл 2 разных юзкейса:
                    1. Класс сам следит за инстансами, наследоваться нельза автоматически (final не нужен) - наш Singleton.
                    2. Клиент создаёт инстансы, final нужен для уверенности в выполнении неких инвариантов - String из java (jvm полагается на то, что строки immutable).
                    Ответить

                    • ava Bert 30.03.2012 15:45 # 0

                      это конечно замечательно, но в php можно наследовать от класса с приватным конструктором
                      Ответить

                      • ava roman-kashitsyn 30.03.2012 15:57 # 0

                        Понял, спасибо за разъяснение. Ни в Java, ни в c#, ни в с++ (насколько я могу судить из своих скудных знаний об этом языке http://ideone.com/53c8q) такого сделать нельзя. Тогда, конечно, без final никак.

                        PHP такой PHP...
                        Ответить

                        • ava 3.14159265 30.03.2012 20:31 # 0

                          В пхп много чего разрешено явно и еще больше можно сделать немного извернувшись, а в результате имеем несбалансированное, кривое, трудноотлаживамое говно.Ява-скрипт тоже страдает от своей вседозволенности, но там хоть нет бремени совместимости с предыдущими версиями.
                          Хотя... Именно говно-то там легко отлаживать. Большими кучами.
                          Ответить

    • ava rO_ot 30.03.2012 12:51 # +3

      Синглтон не нужен
      Ответить

      • ava Bert 30.03.2012 13:26 # 0

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

      • ava Lure Of Chaos 30.03.2012 21:25 # 0

        в пхп - не особо
        Ответить

    • ava viktor_poltorak 30.03.2012 13:28 # 0

      Дальше по коду частенько встречается
      $program = Program::getInstance();
      или того хуже
      $someVar = Program::getInstance()->callSomeMethod();
      Так что это скорее задумывалось как singleton
      Ответить

    • ava myaut 31.03.2012 00:14 # +1

      Сигнлтон или не синглтон, но афтар нарушил логику именования методов, ибо это - createInstance()
      Ответить

    • ava guest8 09.04.2019 12:57 # −999

      показать все, что скрытоvanished
      Ответить

    Добавить комментарий