Witam,

Zastanawiam się czy da się jakoś inaczej stosować singleton niż tak:

[PHP] pobierz, plaintext 
<?php
$db = DB::getInstance();
$user = User::getInstance();
$session = Session::getInstance();
?>
[PHP] pobierz, plaintext 

Czy jest jakiś krótszy sposób? Jak robicie to u siebie, referencję? Bo to jest trochę męczące, żeby dodawać ten kod w każdej metodzie.

Pozdrawiam

Ja robię najczęściej referencję, ale nie zawsze - zależy jak mi wygodniej, czasem przekazuję obiekt jako parametr

Pozdrawiam.

Może ktoś zarzuci jakiś przykład?

ja mam ten sam problem i teraz wymyslilem cos takiego:

[PHP] pobierz, plaintext 
<?php
	class GetClass {
	private $class = array();
 
	static public $oInstance = false;
		public static function gi() {
 
			if( self::$oInstance == false ) {
			self::$oInstance = new GetClass;
			};
 
		return self::$oInstance;
 
		}
 
 
		public function createObj($obj,$name = FALSE) {
 
			if(!$name) {
			$name = $obj;
			}
 
		$name = strtolower($name);
		$this->class[$name] = new $obj;			
 
 
		}
 
 
		public function addObj($refToObj,$name = FALSE) {
 
			if(!$name) {
			$name = get_class( $refToObj );
			}
 
		$name = strtolower($name);
 
		$this->class[$name] = $refToObj;			
		}
 
 
		public function __get($objName) {
 
		return $this->class[ strtolower($objName) ];
		}
 
	}
 
 
 
 
	class Q {
	public $text;
	}
 
 
	$gc = GetClass::gi();
 
	$gc->createObj('Q');
		$gc->q->text = 'hello hello';
	echo $gc->q->text;
 
 
?>
[PHP] pobierz, plaintext 

wtedy tylko jedna klasa jest singielem a reszta juz nie musi byc. nie wiem czy to dobry sposob, bo jak by nie patrzec zawsze jeden antywzorzec zostaje, ale plusem jest to ze we wszystkich innych klasach ktore powinny miec tylko jeden obiekt mozna pominac statyczny fragnent kodu.
pozdrawiam

Ja stworzyłem coś takiego, może już jest taki wzorzec, ale to rozwiązanie mi się podoba:

[PHP] pobierz, plaintext 
<?php
 
class Kernel {
	public $foo;
 
	static $instance;
 
	public function __construct() {
		$this->foo = new Foo;
	}
 
	public static function getInstance() {	
		if(!self::$instance) {
			self::$instance = new Kernel;
		}
		return self::$instance;
	}
}
 
class Foo {
	public function a($string) {
		echo $string;
	}
}
 
class Foo2 {
	public function b() {
		$kernel = Kernel::getInstance();
		$kernel->foo->a('test');
	}
}
 
$foo2 = new Foo2;
$foo2->b();
 
?>
[PHP] pobierz, plaintext 

Wystarczy zadeklarować instancję w konstruktorze jajka i działa

Problemem tego kodu jest fakt, że możesz utworzyć dowolną liczbę instancji każdej ze zdefiniowanych klas. Klasa Kernel ma publiczny konstruktor, a pozostałe nie definiują, czyli posiadają domyślny (który też jest publiczny...). Poza tym - publiczne składowe też nie sprzyjają zachowaniu spójności obiektu. Z dowolnego miejsca możesz nadpisać wartość atrybutu i zrobi się bałagan.

Wiedziałem, że to nie jest idealne, dlatego czekam na propozycję jakiejś mądrzejszej osoby.

W PHP5 obiekty zawsze sa przekazywane przez referencje

eee NuLL ; ]
Odpowiadałem na pytanie:


Chodziło mi o lokalną referencję, czy to w obiekcie, czy w funkcji.

Pozdrawiam.

Kompozycja jest zwykle najlepszym wyjściem...

Osobiście nie spotkałem się z sytuacją, w której musiałbym korzystać z Singletonu/Rejestru do pobrania instancji obiektu. Wszystko trafia albo do konstruktora, albo jako argument metody, albo leci w jakimś kontenerze (np. w czasie wykonywania łańcucha akcji). Singleton ogólnie nie sprzyja testom jednostkowym, gdyż wprowadza zależność, której nie zlikwidujesz raczej bez modyfikacji kodu.

Rozumiem, że chodzi o coś takiego?

[PHP] pobierz, plaintext 
<?php
 
class Foo {
	private $db;
 
	public function __construct($db) {
		$this->db = $db;
	}
 
	public function a() {
		$this->db->query('zapytanie');
	}
}
 
$db = new DB();
$foo = new Foo($db);
 
?>
[PHP] pobierz, plaintext 

Ps Dziękuje @Ludvik za zainteresowanie!

Nie ma problemu Dokładnie o to chodzi Do tego możesz wymusić kontrolę typów, dzięki czemu zachowasz spójność obiektu. Zauważ, że przy testach jednostkowych klasy Foo likwidujesz skutki działania obiektu klasy DB. Po podstawieniu mocka test zostanie zaliczony, nawet gdy klasa DB nie przechodzi swoich testów. Singleton wymuszałby użycie konkretnej klasy, która niekoniecznie działa prawidłowo (o ile została już zaimplementowana).

[PHP] pobierz, plaintext 
<?php 
(object)$this->foo2 = $foo2; 
?>
[PHP] pobierz, plaintext 

o to chodzi?

Chodzi o:

[PHP] pobierz, plaintext 
<?php
 
class Foo {
	private $db;
 
	// Kontrola typów w konstruktorze
	public function __construct(DB $db) {
		$this->db = $db;
	}
 
	public function a() {
		// Jeżeli DB implementuje metodę query(string)
		// to wiemy, że możemy ją zawsze tu wywołać.
		$this->db->query('zapytanie');
	}
}
 
$db = new DB();
$foo = new Foo($db);
 
?>
[PHP] pobierz, plaintext