Forum PHP.pl

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

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. ...

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.

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

Tzn?

Tak w N klasach bo mam N baz danych

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


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

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

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

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

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_*().