Uniwersalna klasa Singleton, Napisana raz, wykorzystywana wiele... Opcje

[tompaw]

Witam forumowiczow.

W swoim projekcie chcialem wykorzystac klasy Singleton. Nie chce jednak w kazdej z nich tworzyc osobno tego samego mechanizmu (czyli funkcji get_instance). Postanowilam napisac klase Singleton, a pozostale tworzyc w oparciu o nia. Niestety, pojawil sie problem.

Klasa Singleton wyglada tak:

[PHP] pobierz, plaintext 
<?php
class Singleton {
 
	private static $instance = null;
 
	private function __construct() {
	}
 
	public static function GI() {
		if (self::$instance == null) {
			self::$instance = new Singleton;
		}
 
		return self::$instance;
	}
};
?>
[PHP] pobierz, plaintext 

Pozniej pojawia sie jakas prosta klasa pochodna i wywolanie jej metody:

[PHP] pobierz, plaintext 
<?php
class Controller extends Singleton {
 
// stuff here...
 
function test() {
	}
 
};
 
Controller::GI()->test();
?>
[PHP] pobierz, plaintext 

Oczywiscie ten kod zwroci blad:

Fatal error: Call to undefined method Singleton::test() in test.php on line x

i jest to zrozumiale. Problem tkwi w jednej linijce metody Singleton::GI(), a mianowicie:

[PHP] pobierz, plaintext 
<?
self::$instance = new Singleton;
?>
[PHP] pobierz, plaintext 

Przez to metoda ta jest na sztywno przypisana do klasy Singleton, ktora w moim przypadku jest klasa czysto wirtualna, tzn chce, aby sluzyla jako podstawa dla innych klas.

Moje pytanie brzmi: jak sprawic, aby tworzony byl obiekt nie klasy Singleton, ale klasy pochodnej od niej, na rzecz ktorej wywolywana jest metoda GI() ?

Probowalem dostac sie do nazwy tej klasy dwojako: przez get_class() oraz __CLASS__. Niestety, zawsze zwracana jest klasa Singleton.

Z gory dzieki za pomoc, pozdrawiam.

Ten post edytował tompaw 7.11.2004, 13:04:21

[bela]

[PHP] pobierz, plaintext 
<?php
class Singleton {
 
	private static $instance = array;
 
	private function __construct() {
	}
 
	public static function GI( $name ) {
		if (self::$instance[$name] == null) {
			self::$instance[$name] = new $name;
		}
 
		return self::$instance[$name];
	}
};
?>
[PHP] pobierz, plaintext 

moze cos takiegO ?

[PHP] pobierz, plaintext 
<?php
class Controller extends Singleton {
 
// stuff here...
 
function test() {
	}
 
};
 
Controller::GI()->test();
?>
[PHP] pobierz, plaintext 

zle sie odwołujesz powinienes

[PHP] pobierz, plaintext 
<?php
$object = object::GI();
$object->test();
 
?>
[PHP] pobierz, plaintext 

[tompaw]

Tylko ze z Twoja propozycja to trzeba by wywolywac funkcje:

ClassName::GI("ClassName")

tak ?

Dlaczego wywolanie Controller::GI()->test(); jest wg Ciebie zle ?

[hawk]

Odwołanie Controller::GI()->test() jest jak najbardziej poprawne.

To co bela666 proponuje to już nie będzie singleton.

@tomapaw: tego się raczej nie da zrobić. Nie tylko w php, w innych językach również. Chociaż w php miałoby to większe szanse powodzenia, bo php jest interpretowany.

[bela]

To co bela_666 proponuje to już nie będzie singleton.

a czemu ?

[hawk]

Bo to będzie Rejestr. Była jeszcze jakaś inna, mądrzejsza nazwa, ale nie zmienia to faktu, że rejestr to nie singleton. Inny wzorzec.

Bo Singleton służy do teo, aby była tylko jedna instancja klasy, a w twoim przykładzie w ogóle nie będzie żadnej instancji klasy Singleton.

Bo w zależności od tego co podamy, dostajemy obiekty różnej klasy, a singleton powinien zawsze dawać tą samą instancję.

Po prostu - inny wzorzec. Ani lepszy, ani gorszy, ale na pewno nie singleton. Co nie zmienia faktu że rejestry często są przy okazji singletonami.

[tompaw]

hm... rozwiazalem to w ten sposob:

[PHP] pobierz, plaintext 
<?
 
class Singleton {
 
	private static $instance = null;
 
	private function __construct() {
	}
 
	public static function &GetInstance($name) {
		if (self::$instance[$name] == null) {
			self::$instance[$name] = & new $name; 
		}
 
		return self::$instance[$name];
	}
};
 
class SingletonManager {
 
	function &GI($type) {
		eval(&#092;"$new_object = \".$type.\"::GetInstance(\"\".$type.\"\");\");
		return $new_object;
	}
 
}
 
?>
[PHP] pobierz, plaintext 

opinie, komentarze, itp ;>

[Vengeance]

a na ciorta tak ?

jak masz tą pierwszą klase to mozesz juz

Singleton::GetInstance('jakasklasa');
czyba nie ?

[bela]

hm... rozwiazalem to w ten sposob:

[PHP] pobierz, plaintext 
<?
 
class Singleton {
 
	private static $instance = null;
 
	private function __construct() {
	}
 
	public static function &GetInstance($name) {
		if (self::$instance[$name] == null) {
			self::$instance[$name] = & new $name; 
		}
 
		return self::$instance[$name];
	}
};
 
class SingletonManager {
 
	function &GI($type) {
		eval(&#092;"$new_object = \".$type.\"::GetInstance(\"\".$type.\"\");\");
		return $new_object;
	}
 
}
 
?>
[PHP] pobierz, plaintext 

opinie, komentarze, itp ;>

przecież hawk powiedział, że to nie singleton

[tompaw]

a na ciorta tak ?

jak masz tą pierwszą klase to mozesz juz

Singleton::GetInstance('jakasklasa');
czyba nie ?

jak tak teraz na to patrze to chyba rzeczywiscie =]

[hawk]

Tylko dlaczego w kółko pojawiają się wątki o singletonie, które kończą się tym, że singleton jest olewany i staje na czymś zupełnie innym?

Singleton - najbardziej nadużywany wzorzec wszechczasów.

[Ozzy]

http://zend.com/zend/week/week209.php => TLK: New overloading feature?