Rozszerzalny singleton Opcje

[Orzeszekk]

[PHP] pobierz, plaintext 
<?php
/**
 * Rozszerzalny singleton.
 *
 * <b>Użycie</b>
 * <ul><li>W każdej klasie pochodnej utwórz protected static $instance</li>
 * <li>Konstruktor kazdej klasy pochodnej musi byc chroniony lub prywatny</li></ul>
 */
class Singleton
{
   /** To jest do celow testowych */
   protected $nazwa;
   protected static $instance;
   protected function __construct()
   {
   }
   public static function get()
   {
      if (!(static::$instance))
      {
         $className = get_called_class();
         $className::$instance = new $className;
      }
      return static::$instance;
   }
   public function identify()
   {
      echo $this->nazwa;
   }
}
class Pierwsza extends Singleton
{
   public static $instance;
   protected function __construct()
   {
      $this->nazwa = "pierwsza";
   }
}
class Druga extends Singleton
{
   public static $instance;
   protected function __construct()
   {
      $this->nazwa = "druga";
   }
}
class Trzecia extends Druga
{
   public static $instance;
   protected function __construct()
   {
      $this->nazwa = "trzecia";
   }
}
$pierwsza = Pierwsza::get();
$druga = Druga::get();
$trzecia = Trzecia::get();
$trzecia->identify();
$pierwsza->identify();
$druga->identify();
?>
[PHP] pobierz, plaintext 

Jak go ulepszyc bym nie musial deklarowac w kazdej klasie public static $instance?

jesli usune deklaracje tych zmiennych statycznych z klas to wtedy tworzy mi singleton zamiast klasy tej co trzeba (IMG:style_emoticons/default/biggrin.gif) (ew w przypadku klasy trzecia utworzy mi klase druga).

Ten post edytował Orzeszekk 7.11.2011, 11:46:45

[Sephirus]

Hmm no nie dziwne - jeśli nie dasz w każdej klasie $instance to będziesz się z każdej odwoływał zawsze do tego samego (IMG:style_emoticons/default/smile.gif) oto w sumie chodzi w staticach (IMG:style_emoticons/default/haha.gif)

Moim zdaniem powinno się dawać statyczne $intance dla każdej klasy - sama funkcja typu "getInstance" może być dziedziczona. To i tak mniej a na bank będzie działać.

Drugim rozwiązaniem jest zrobienie statycznej tablicy "instancji" - wtedy w get() odwołuj się do $instance[self::$nazwa] - To powinno troszkę pomóc i nie byłoby potrzeby dawać $instance w tych klasach. (IMG:style_emoticons/default/wink.gif)

ALBO:

Zupełnie inne podejście (IMG:style_emoticons/default/smile.gif) Po co robić rozszerzalny singleton, z którego musiała będzie dziedziczyć każda klasa by była singletonem...

Można zrobić inaczej (IMG:style_emoticons/default/wink.gif) Zróbmy tak by dowolna klasa mogła być traktowana jako singleton ;P Zróbmy fabrykę singletonów (IMG:style_emoticons/default/wink.gif)

[PHP] pobierz, plaintext 
class singleton
{
	protected static $instance = array();
 
        private function __construct() {}
 
	public static function getInstance($className)
	{
		if(!isset(self::$instance[$className]))
			self::$instance[$className] = new $className;
 
		return self::$instance[$className];
	}
}
 
 
class a
{
	public function identify() { echo 'jestem A!'; }
}
 
class b 
{
	public function identify() { echo 'jestem B!'; }
}
 
class c 
{
	public function identify() { echo 'jestem C!'; }
}
 
$a = singleton::getInstance('a');
$b = singleton::getInstance('b');
$c = singleton::getInstance('c');
 
$a->identify();
$b->identify();
$c->identify();
[PHP] pobierz, plaintext 

Zamiast używać wtedy a::getInstance(); używamy singleton::getInstance('a'); - to daje nawet więcej możliwości ;P

Co Ty na to?

Ten post edytował Sephirus 7.11.2011, 12:34:02