Wzorzec Singletona w MySQL z PEAR, Problem połączenia 3 rzeczy Opcje

[Marys91]

Witam,
chciałbym wykorzystać wzorzec Singletona do połączenia się z bazą danych (MySQL) i do tego dołożyłem sobie PEAR' a, ale coś mi nie idzie. To co mam to są takie pliki:

classa.Databases.php

[PHP] pobierz, plaintext 
<?php
require_once('pear/MDB2.php');
require_once('config.php');
 
 
class Datebase{	
	private $conn;
 
	private function __construct($zmienna=null){
		global $mysql_dsn;
 
		if($zmienna==null){
			$zmienna = $mysql_dsn;
		}
 
		$this->conn=MDB2::connect($zmienna);
	}
 
	static public function instance(){
		static $objMDB2;
 
		if (!isset($objMDB2)){
			$objMDB2 = new Datebase();
		}
 
		return $objMDB2;
	}
 
	function select($sql){
		$result = $this->conn->query($sql);
 
		return $result;
	}
 
	function fetchAll($sql){
		$result = $this->conn->fetchAll($sql);
 
		return $result;
	}
 
}
?>
[PHP] pobierz, plaintext 

Wiem, że brakuje obsługi błędów.

test.php

[PHP] pobierz, plaintext 
<?php
	require_once('class.Database.php');
 
	$ble = Datebase::instance();
	$elb = $ble->fetchAll('SELECT * FROM moajtabela');
 
	echo $elb;
?>
[PHP] pobierz, plaintext 

I teraz tak. Po pierwsze czy dobrze myślę o wzorze Singletona (dobrze jest napisana klasa)? Po drugie czemu mi wywala błąd(Fatal error: Call to undefined function: MDB2_Driver_mysqli::numCol(). in pear/MDB2.php on line 1950)? To wygląda na błąd PEAR, ale jak robię normalnie bez mojej klasy to jest dobrze. Po trzecie, widziałem gdzieś jak koleś właśnie tworzył takie metody jak u mnie (fetchAll), ale czy przed tym nie powinno występować query()?

Nie wiem co tu jeszcze może być źle

[Noidea]

PEAR MDB2 ma już wbudowany singleton http://pear.php.net/manual/en/package.data...tro-connect.php więc twoja klasa jest trochę bez sensu.

Twój singleton napisany jest w sumie dobrze. Nie zabezpiecza jedynie przed:

[PHP] pobierz, plaintext 
<?php
 
$ble = Datebase::instance();
$ble2 = clone $ble;
 
?>
[PHP] pobierz, plaintext 

Dodaj do swojej klasy pustą funkcję:
private function __clone() {}


No i źle używasz fetchAll(). W tej bibliotece MDB2_Driver_Common::query() zwraca obiekt klasy MDB2_Result_Common, na którym dopiero można wykonać fetchAll().

[Marys91]

Czyli teraz trochę bez sensu jakby tworzyć tą klasę, tylko bezpośrednio korzystać MDB2?

Przykładowo

[PHP] pobierz, plaintext 
<?php
require('MDB2');
 
$db = MDB2->singleton();
 
$db->query('');
 
...
?>
[PHP] pobierz, plaintext 

czy jak teraz tego singletona wykorzystać w tym momencie?

[olechafm]

to zależy, co potrzebujesz zrobić, opisane są 3 funkcje i każda z nich zwraca instancję, factory() nie nawiązuje połączenia, connect() nawiązuje gdy już masz np. zainicjowany ten obiekt, a singleton w tym wypadku robi to tak jak Ty chcesz to wykorzystać

[Marys91]

To może inaczej. Czytałem właśnie artykuł o abstrakcyjnej klasie obsługującej bazę, m.in. wzorzec singletona. Mi głównie chodzi o to, aby nie łączyć się z bazą za każdym razem.

[olechafm]

no to factory() nie łączy się z bazą, ale zwraca instancję obiektu

Will instantiate a new MDB2_Driver_Common instance, but will not connect to the database until required. This will delay making the actual connection. This is called lazy connecting. Using this makes sense if it is possible that due to caching inside the application no connection will ever need to be established.