Реализация паттерна Singleton.

[Elser]

после многих танцев с бубном родил красивое решение
кому надо, кушайте наздоровье
P.S. у кого есть какие мысли и соображения готов обсудить

package {
 
	public class Singleton {
 
		private static var instance:Singleton = null;
		// грубо имитируем приватность конструктора
		private static var privater:Boolean = true;
 
		public function Singleton() {
			if(privater) {
				throw new Error("Use Singleton.getInstance() insted new Singleton() !");
			}
		}
 
		public static function getInstance():Singleton{
			if(!instance){ // ето будет выполнено только первый раз
				privater = false;
				instance = new Singleton();
				privater = true;
			}
			return instance;
		}
	}
}

заранее спасибо за здоровую критику

[-De-]

Можно чуть попроще, если вам надо. По моему скромному мнению, есть ошибки из-за невнимательности, от которых надо страховаться, а есть злодейство. Дёргать конструктор у синглтона - злодейство. Я бы даже выкинул проверку из getInstance из 95-99% синглтонов %)

package {
 
	public class Singleton {
 
		private static var instance:Singleton = null;
		// грубо имитируем приватность конструктора
 
		public function Singleton() {
			if(instance) {
				throw new Error("Use Singleton.getInstance() insted new Singleton() !");
			}
			instance = this;
		}
 
		public static function getInstance():Singleton{
			if(!instance){ // ето будет выполнено только первый раз
				new Singleton();
			}
			return instance;
		}
	}
}

upd: дада, согласен с замечаниями!

[Elser]

Ваш подход имеет право на жизнь, но он сырой в плане что первый инстанс можно породить как методом getInstance, так и конструктором. Во втором случае это будет работать что не есть хорошо. Моя первая реализация была такой же, но я пришел к тому что вызов конструктора извне Всегда должен генерировать исключение, а порождение синглтона, как и возврат ссылок на него должен выполнять один и тот же метод и это представьте должен быть только он. Вы не поверите, но мой подход таки более законченый.

[Mur4ik]

Мне вот такая реализация нравится

package 
{
	public final class Singleton
	{	
		private static var _instance:Singleton = new Singleton();
 
		public function Singleton()
		{
			if ( _instance ) throw new Error("Instantiation failed: Use Singleton.getInstance() instead of new Singleton().");
		}
 
		public static function getInstance():Singleton
		{
			return _instance;
		}
	}
}

[udaaff]

http://garbage-collector.ru/2007/10/...nscript_3.html

[Elser]

udaaff +1, спасибо

[dimarik]

Может что-нибудь по-сложнее обсудим? Медиатор, стратегию, состояние или посетителя? Право, синглтоны уже оскомину набили.

[Nirth]

Перестаньте делать синглтоны )) 21ый век, можно на мультитоны[1] переходить ^_^

[1] http://en.wikipedia.org/wiki/Multiton.

[Котяра]

Сообщение от dimarik

Может что-нибудь по-сложнее обсудим? Медиатор, стратегию, состояние или посетителя? Право, синглтоны уже оскомину набили.

Давайте)
Только, пожалуйста, не МВЦ)

[expl]

А может лучше обсудить "как правильно жить без синглтонов"?
Потому что из-за этой сволочи половину классов в проекте уже отдельно не протестируешь , не говоря о том, что иногда просто не понятно что ждать от этой глобальной точки в данный момент, надеясь что ее никто не поменял нечаянно.