[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ć?

Z góry dzięki za sugestie.

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

[Smertius]

Wg mnie powinieneś łączenie z bazą danych przenieść do konstruktora bazy DB, a w klasach które po niej dziedziczą wywoływać konstruktor rodzica przez parent::__construct(), nie ma sensu tego pchać do każdej klasy w dodatku jeśli będziesz chciał zmienić dane do BD to bedziesz to zmieniał w N klasach? Trochę to niewygodne.

Co do Singeltona, to błędnie go implementujesz. Rzuć okiem na kod:

[PHP] pobierz, plaintext 
class klasa {
 
    private static $instance;
 
    public function __construct() {
        if (!(self::$instance instanceof klasa)) {
            self::$instance = $this;
            return self::$instance;
        } else {
            return self::$instance;
        }
    }
}
[PHP] pobierz, plaintext 

Tak jak powyżej, jeśli po pierwszym utworzeniu obiektu "klasa", spróbujesz go utworzyć jeszcze raz (new Klasa) to zostanie zwrócony już istniejąca instancja "klasa"

IMHO mamy PDO, mamy ORM a ludzie dalej korzystają z niewygodych, przestarzałych funkcji typu mysql_query itp. ...

[Mephistofeles]

Użyj np. Doctrine DBAL (czy nawet całego ORMa), załatwi sprawę kilku połączeń. Albo po prostu PDO. Mysql_ było dobre może 5 lat temu, nie pamiętam kiedy wszedł PHP 5 z PDO.

[Crozin]

@Smertius: Proponuje Ci sprawdzać swój kod przed podaniem, bo kompletne bzdury wypisujesz...

[Smertius]

Tzn?

[patryczakowy]

(..) nie ma sensu tego pchać do każdej klasy w dodatku jeśli będziesz chciał zmienić dane do BD to bedziesz to zmieniał w N klasach? Trochę to niewygodne.

Tak w N klasach bo mam N baz danych

Co do Singeltona, to błędnie go implementujesz. Rzuć okiem na kod:

Co do implementacji wzorca singleton to zaczerpnełem go z tej książki

Tak jak powyżej, jeśli po pierwszym utworzeniu obiektu "klasa", spróbujesz go utworzyć jeszcze raz (new Klasa) to zostanie zwrócony już istniejąca instancja "klasa"

IMHO mamy PDO, mamy ORM a ludzie dalej korzystają z niewygodych, przestarzałych funkcji typu mysql_query itp. ...

i tak samo działa moja klasa objekt jest ale nie mam już połączenia z bazą danych gdyż wcześniej zainicjowałem połączenie z drugą bazą danych

Użyj np. Doctrine DBAL (czy nawet całego ORMa), załatwi sprawę kilku połączeń. Albo po prostu PDO. Mysql_ było dobre może 5 lat temu, nie pamiętam kiedy wszedł PHP 5 z PDO.

Wygodne nie wygodne jak się ma masę kodu do przerobienia to się kombinuje tak żeby wszystkiego nie trzeba było pisać od początku, a było w miarę optymalne jeśli zainicjuje trzy połączenia do trzech różnych baz danych przy pomocy PDO to te trzy połączenia będą istnieć cały czas równolegle ? i co w takim razie ze wszystkimi zwykłymi zapytaniami mysql_query które już istnieją bo w obecnym momencie do póki nie inicjowałem drugiej klasy to po wywołaniu

[PHP] pobierz, plaintext 
$DB=db_portal::connect_db();
[PHP] pobierz, plaintext 

jeśli w kodzie gdzieś dalej istniało coś takiego

[PHP] pobierz, plaintext 
$a= mysql_query("SELECT * FROM TABELA");
 
[PHP] pobierz, plaintext 

to wszystko działało bez konieczności ingerencji w już napisany kod

[Fifi209]

A do manuala nie łaska zajrzeć?

resource mysql_connect ([ string $serwer = ini_get("mysql.default_host") [, string $nazwa_użytkownika = ini_get("mysql.default_user") [, string $hasło = ini_get("mysql.default_password") [, bool $nowe_połączenie = false [, int $flagi_klienta = 0 ]]]]] )

[patryczakowy]

A do manuala nie łaska zajrzeć?

sorki ale może nie zawsze umiem wyciągnąć z manula to co mi potrzebne, niezabardzo wiem z jakimi parametrami mam wywołać to mysql_connect aby działało tak jak ja chcę mysql_connect , nawet jeślim dam tak

[PHP] pobierz, plaintext 
$this->conn_id = mysql_connect($cfg['db']['host'],$cfg['db']['user'],$cfg['db']['password'],true);
[PHP] pobierz, plaintext 

to nie działa, czemu zresztą się nie dziwię gdyż przy drugim wywołaniu objekt istnieje więc konstruktor nie jest wywoływany, z resztą takie było założenie żeby z bazą łączyć się tylko raz

chyba coś przekombinowałem powoli zaczyna dochodzić do mnie że bez przepisania wszystkiego pożądanego efektu nie osiągnę niemniej jednak czekam na jakieś koło ratunkowe

[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?!)

[patryczakowy]

To co podałeś rozwiązało prawie cały problem i wielkie dzięki, jednak tak jak pisałem wcześniej niestety wiąże się to z przepisaniem wszystkich wystąpień mysql_query, a chciałem tego uniknąć jak widać nie da rady, ale może to i lepiej przy okazji przejże wszystkie zapytania do bazy

[Crozin]

Jeżeli i tak jesteś zmuszony przepisać zapytania to skorzystaj z PDO, które mimo iż nie idealne jest znacznie lepszym interfejsem niż funkcje mysql_*().