[PHP] singleton a trzy bazy danych Opcje

[patryczakowy]

Witam mam mały problem mam skrypt który korzysta z trzech baz danych założenie było takie żeby podczas wykonywania skryptu były maksymalnie trzy połączenia z bazą dlatego chciałem zimplementować wzorzec singleton dl a każdej z baz jednak problem jest taki że podczas wywołania klasy z połączeniem do drugiej bazy połączenie z pierwszą zostaje przerwane po niżej przykładowy kody

[PHP] pobierz, plaintext 
class db_portal extends db{
 
 
private function __construct() {
	$cfg['db']['host']='localhost';
	$cfg['db']['user']='xxxxxx';
	$cfg['db']['password']='xxxxx';
	$cfg['db']['name']='xxxxxx';
	$this->conn_id = mysql_connect($cfg['db']['host'],$cfg['db']['user'],$cfg['db']['password']);
	mysql_select_db($cfg['db']['name'],$this->conn_id);
	mysql_query("SET CHARACTER SET utf8");
	mysql_query("SET collation_connection = utf8_unicode_ci");
		if(! is_resource($this->conn_id)) {
         	return false;
      		}
}
static public function connect_db () {
	 static $objDB;
	 	if(!isset($objDB))
		$objDB = new db_portal;
 
	return $objDB;
	}
}
class db_portal2 extends db{
 
 
private function __construct() {
	$cfg['db']['host']='localhost';
	$cfg['db']['user']='xxxxxx';
	$cfg['db']['password']='xxxxx';
	$cfg['db']['name']='xxxxxx';
	$this->conn_id = mysql_connect($cfg['db']['host'],$cfg['db']['user'],$cfg['db']['password']);
	mysql_select_db($cfg['db']['name'],$this->conn_id);
	mysql_query("SET CHARACTER SET utf8");
	mysql_query("SET collation_connection = utf8_unicode_ci");
		if(! is_resource($this->conn_id)) {
         	return false;
      		}
}
static public function connect_db () {
	 static $objDB;
	 	if(!isset($objDB))
		$objDB = new db_portal2;
 
	return $objDB;
	}
}
$DB=db_portal::connect_db();
//zapytania z bazy 1
$DB2=db_portal2::connect_db();
// zapytania z bazy 2
$DB->query($SQL);
//problem objekt $DB istnieje ale niema już połączenia z bazą przez co zapytanie się nie wykonuje 
//nie pomaga nawet dodanie linii wcześniej $DB=db_portal::connect_db();
[PHP] pobierz, plaintext 

Klasa db to standardowa klasa mająca metody typu query fetch itp

Ma ktoś pomysł jak można to rozwiązać(IMG:style_emoticons/default/questionmark.gif) ?

Z góry dzięki za sugestie.

Ten post edytował patryczakowy 22.01.2011, 08:07:27

[Crozin]

1. mysql_query powinno mieć określone z którego połączenia korzysta (patrz drugi argument funkcji).
2. Przestań korzystać z Singletona, bo na stronach WWW jest naprawdę baaaaardzo niewiele sytuacji, gdzie jego użycie miałoby sens, a korzystanie z tego wzorca poza jedną zaletą (powodem, dla którego stworzono go - czyli zapewnieniem istnienia co najwyżej jeden instancji danej klasy) ciągnie za sobą masę problemów.
3. Zapewne korzystasz z Singletona by uzyskać globalny dostęp do bazy danych (to jest złe użycie oczywiście), ale mógłbyś wykorzystać coś takiego:

[PHP] pobierz, plaintext 
// Zakładam, że klasa DB to mniej-więcej coś takiego:
 
class DB {
  protected $connection;
 
  public function __construct($host, ...) {
    $this->connection = mysql_connect(...);
  }
 
  public function query($sql) {
    $result = mysql_query($sql, $this->connection);
  }
 
  // etc.
}
 
// Tak więc utwórzmy sobie najpierw jakiegoś globalnego (złe, ale skoro tak już musisz mieć) menadżera połączeń:
class DBManager {
  protected static $connections = array();
 
  public static function get($name) {
    return self::$connections[$name];
  }
 
  public static function set($name, DB $connection) {
    self::$connections[$name] = $connection;
  }
}
 
// W którymś tam miejscu skryptu inicjalizujesz połączenia:
 
$defaultConnection = new DB('...', '...');
$extraConnection = new DB('....');
$yetAnotherConnection = new DB(...);
 
// I dodajesz je do menadżera:
 
DBManager::set('default', $defaultConnection);
DBManager::set('extra', $extraConnection);
DBManager::set('yetAnothert', $yetAnotherConnection);
 
// Teraz w dowolnej części aplikacji możesz zrobić coś w stylu:
 
$db = DBManager::get('extra');
$db->query('...');
 
$db = DBManager::get('default');
$db->fetchArray('....');
[PHP] pobierz, plaintext 

Tzn?

To znaczy, że to co podałeś nie jest Singletonem. To papka bezsensownego kodu (konstruktor zwracający coś? wtf?!)