Witajcie,

stworzyłem klasę singleton służącą do obsługi bazy danych (aplikacja będzie korzystać tylko z jednej bazy danych). Konstruktor klasy przyjmuje argumenty takie jak adres serwera, login, hasło do DB.

Mam drugą klasę, która potrzebuje korzystać z bazy danych. Mam pytanie jak rozwiązać kwestię tworzenia obiektu klasy singleton, bo przecież nie tworzy się tutaj obiektu klasy "bezpośrednio", lecz za pomocą innej funkcji. Aktualnie zrobiłem to tak, że w klasie obsługującej bazę danych, w funkcji tworzącej/zwracającej obiekt tejże klasy, dodałem argumenty te same co w konstruktorze:

[PHP] pobierz, plaintext 
private function __construct($server, $username, $password, $database)
{
	// tutaj jakieś operacje
}
public static function singleton($server = NULL, $username = NULL, $password = NULL, $database = NULL)
{
	if (!isset(self::$instance)) {
		$className = __CLASS__;
		self::$instance = new $className($server, $username, $password, $database);
	}
	return self::$instance;
}
[PHP] pobierz, plaintext 

Gdzieś tam np. w index.php tworzę sobie obiekt klasy obsługującej bazę danych:

[PHP] pobierz, plaintext 
$DB = BazaDanych::singleton($server, $username, $password, $database);
[PHP] pobierz, plaintext 

W tym wypadku w drugiej klasie, wykorzystującej bazę danych, mam referencję do wcześniej utworzonego obiektu klasy obsługującej bazę danych:

[PHP] pobierz, plaintext 
 
private $DB;
 
public function __construct(){
	$this->DB = BazaDanych::singleton()
}
[PHP] pobierz, plaintext 

Wszystko działa, ale nie jestem pewien czy dobrze to zorganizowałem. Nie wygląda to sensownie...

Osobiście uważam, że nie jest Ci potrzebny tutaj singleton. Obsługę baz danych naprawdę da się zrealizować bez jego implementacji, a to, że teraz korzystasz z jednej bazy to też nie jest dobry argument, bo w przyszłości może być potrzeba zainicjowania kolejnego połączenia lub będziesz miał stworzyć nową aplikację, która wykorzystuje n baz danych. I wtedy Twój obecny kod będzie nie przydatny.
To tyle w kwestii moralizowania:P A co do odpowiedzi na twoje pytanie, to można to zrobić w ten sposób:

[PHP] pobierz, plaintext 
class DbConnection
{
 private static $_instance = null;
 private $_connectionData; //dane niezbędne do połączenia, to jak je przedstawisz (tablica lub rozbicie na osobne atrybuty) jest nieistotne.
 private $_connection = null; //połączenie z bazą danych
 
 public static function getInstance()
 {
   if (self::$_instance === null)
      self::$_instance = new self;
 
   return self::$_instance;
 }
 public function isConnected() { return $this->_connection !== null; }
 public function connect($server, $username, $password, $database) { /*łączenie się z bazą*/ }
 private function __construct();
 private function __clone();
}
[PHP] pobierz, plaintext 

Dzięki temu oddzielasz pobieranie instancji obiektu od inicjowania połączenia. Do metody connect() możesz dodać kod, który rzuca wyjątkiem, jeżeli jest już połączenie.

A przypisywanie instancji singletona do atrybutu klasy (Twoja druga klasa), jest naprawdę zbędne skoro już implementujesz singleton. W takim wypadku DbConnection zamieniłbym na normalną klasę i użyłbym DI do przekazania go do tej klasy.

Być może singleton nie jest tutaj potrzebny i rzeczywiście taka implementacja singletona nie przyda się w sytuacji, gdy będziemy chcieli uzyskać dostęp do kilku baz danych. Warto jednak wiedzieć, że można zaimplementować wzorzec singleton tak, żeby obsługiwał nam te nasze n połączeń z bazą danych. Oczywiście ma to pewne ograniczenia, bo musimy określić maksymalną liczbę połączeń. W każdym bądź razie chciałem zwrócić uwagę, że klasa singleton nie zawsze musi służyć do tworzenia i obsługi dokładnie jednej instancji obiektu tej klasy, choć przeważnie do tego służy.

A jak inaczej w tym przypadku, uzyskać dostęp do obiektu klasy DB który jest singletonem (pomijając przekazywanie jako argument)?