singleton i dziedziczenie Opcje

[ksiadz]

mam problem z singletonem przez dziedziczenie a mianowicie napisalem taki kod

[PHP] pobierz, plaintext 
<?php
 
class system
{
	protected static $instance;
 
	protected function __construct()
	{}
 
	public static function get_instance()
	{
		if( self::$instance === false )
		{
			self::$instance = new self();
		}
 
		return self::$instance;
	}
}
 
class innysystem extends system
{
	public function main_run()
	{
	}
}
 
$system = innysystem::get_instance();
$system->main_run();
 
?>
[PHP] pobierz, plaintext 

ale podczas pobierania referencji przez funkcje get_instance() tworzy sie nowy obiekt z tym ze jest to 'system' a nie 'innysystem' wiec jak to najprosciej naprawic by we wszystkich klasach potomnych nie trzebabylo wklepywac tej samej funkcji get_instance() ?

[dr_bonzo]

Niestety musisz recznie dopisac get_instance() dla kazdej z klas pochodnych. Jako ze nie przeciazyles tej metody w klasie pochodnej, get_instance() bedzie zawsze wywolywana dla obiektu klasy nadrzednej.

Ten post edytował dr_bonzo 20.05.2005, 19:24:57

[Imperior]

Co gorsza, nie da się (przynajmniej ja nie mogłem) pobrać nawet nazwy tej "rzeczywistej" klasy.

[dr_bonzo]

Nawet get_class() nie dziala

[ksiadz]

jedyne co mi sie udalo uzyskac to tylko to

[PHP] pobierz, plaintext 
<?php
 
class system
{
	protected static $instance = array();
 
	protected function __construct()
	{}
 
	public static function get_instance( $classname=__CLASS__ )
	{
		if( ! isset( self::$instance[ $classname ] ) )
		{
			self::$instance[ $classname ] = new $classname();
		}
 
		return self::$instance[ $classname ];
	}
}
 
class innysystem extends system
{
	public static function get_instance()
	{
		return parent::get_instance( __CLASS__ );
	}
 
	public function main_run()
	{
	}
}
 
$system = innysystem::get_instance();
$system->main_run();
 
?>
[PHP] pobierz, plaintext 

moze ktos ma lepszy pomysl na dziedziczenie singletona

[dr_bonzo]

No i chcesz stworzyc instancje klasy 'innysystem'

[PHP] pobierz, plaintext 
<?php
$blabla = innysystem::get_instance();
?>
[PHP] pobierz, plaintext 

co konczy sie wywolaniem:

[PHP] pobierz, plaintext 
<?php
new innysystem();
?>
[PHP] pobierz, plaintext 

Aby to wywolanie bylo mozliwe -- konstruktor klasy innysystem musi byc publiczny, a jak jest publiczny to mozna stworzyc obiekt w sposob tradycyjny, co burzy koncepcje singletona.

A poza tym skoro juz implementujesz metode get_instance() w klasie pochodnej to po prostu napisz ja tak jak w klasie 'system'.

[Bora]

[PHP] pobierz, plaintext 
<?php
class Container
{
   private static $container = array();
 
   private static $instance;
 
   private function __construct()
 
    public static function getInstance()
    {
        if( self::$instance === false )
        {
            self::$instance = new self();
        }
 
        return self::$instance;
    }
 
   public static function get( $name )
   {
      if( isset( self::$container[ $name ] ) )
      {
         return self::$container[ $name ];
      }
   }
   
   public static function set( $name, $value )
   {
      self::$container[ $name ] = $valule;
   }
}
class Module
{
   static $modules = array();
   
   public static function _get( $name )
   {
      $container = Container::GetInstance(); 
      $container->get($name);
   }
   
   public static function _set( $name, $value )
   {
      $container = Container::GetInstance(); 
      $container->get($name, $valule); 
   }
}
 
class Core extends Module
{
}
 
class Shop extends Module
{
}
 
$core = new Core;
$shop = new Shop;
 
$core->test = 'Testujemy';
 
echo 'Core: '.$core->test.'<br/>Shop: '.$shop->test;
?>
[PHP] pobierz, plaintext 

pisane w locie ale powinno zadzialac.

widze ze piszac w pospiechu pomylilem tematy: http://forum.php.pl/index.php?showtopic=33537

Ten post edytował Bora 2.08.2005, 13:07:55

[dag]

Hmm IMHO jest to błąd projektowy. Jeśli używać wzorca projektowego SINGLETON to tylko w ten sposób do jakiego jest zalecany, czyli do zwracania referencji obiektu i uniemożliwienie tworzenia nowych obiektów.

Jeśli już musisz dziedziczyć po jakieś klasie to lepiej rozbić tę klasę. IMHO będzie to lepsze rozwiązanie i czytelniejsze z projektowego punktu widzenia.

[NuLL]

A co sądzicie o pomyśle na singletonFactory :?: Chyba wiadomo o co chodzi