Rozszerzanie klasy PDO - propozycja, Singleton, zrywanie połączenia Opcje

[elnino.pl]

Co myślicie o poniższym kodzie:

[PHP] pobierz, plaintext 
<?php
class db extends PDO
{
	public function singleton()
	{
		static $instance;
		if(!isset($instance))
			$instance = new db;
		return $instance;
	}
 
	function __construct()
	{
		require_once('../config.php5');
		parent::__construct($dbd['type'] . ':host=' . $dbd['server'] . ';dbname=' . $dbd['db'], $dbd['user'], $dbd['pass']);
		parent::query("SET CHARACTER SET utf8"); 
		define('P', $dbd['prefix']);
	}
}
?>
[PHP] pobierz, plaintext 

Niby wszystko działa tak jak chcę, ale w jaki sposób mam zabijać uchwyt do bazy? Czyste pdo niszczyłem poprzez $obiektPDO = null - A w tej chwili nie bardzo wiem jak to zrobić. Próbowałem wywołać destruktor PDO w destruktorze mojej klasy, ale okazało się, że PDO nie posiada destruktora. Nie chcę, żeby mi "zwisały" otwarte połączenia z bazą. Pamiętam, że przy standardowych połączeniach z bazą miałem z tym problem. Czasem zapominałem kończyć połączenia.

[bigZbig]

Po pierwsze ta klasa nie jest Singletonem ponieważ mozna utworzyć wiecej niż jedną jej instancję.

Po drugie brak deklaracji wlasciwosci statycznych

Po trzecie gdyby to byl singleton to nie mialbys problemu ze "zwisającymi" otwartymi polaczeniami z baza bo byloby tylko jedno polaczenie.

Po czwarte - destrukcja $obiektDB = null; i po sprawie

[elnino.pl]

Nie jest singletonem, bo php odmawia oznaczenia konstruktora jako prywatny.

[bigZbig]

A czytales temat Singleton i dziedziczenie?

[elnino.pl]

Czytałem. Z tego, co zrozumiałem w wersji 5.1.4 nie da się tego obejść.

[bigZbig]

Czytaj do konca. Poza tym nie zmienia to faktu, ze Twoja klasa zawiera bledy formalne. Poczytaj na temat implementacji wzorca singleton.

Pokaz mi jak uzywasz swojej klasy.

[elnino.pl]

Chodzi Ci o wersję cvs?

[PHP] pobierz, plaintext 
<?php
 
$dbr = db::singleton();
 
?>
[PHP] pobierz, plaintext 

W sumie czytałem o singletonach tutaj: http://webcity.pl/webcity/artykuly.php/t/62

O jakie błędy formalne Ci chodzi?

[bigZbig]

[PHP] pobierz, plaintext 
<?php
$dbr = db::singleton();
?>
[PHP] pobierz, plaintext 

Spojrz na swoja klase i na sposob uzycia? Czy wszysko Ci tu pasuje? Poza tym przeczytaj jeszcze raz moj pierwszy post w tym temacie.

-- edit --
Zobacz Singleton

Ten post edytował bigZbig 21.07.2006, 12:28:06

[NuLL]

elnino - po co stosujesz zmienna statyczna metody skoro w php5 mamy prywatne wlasnosci statyczne (IMG:http://forum.php.pl/style_emoticons/default/questionmark.gif)

[elnino.pl]

Dzięki za pomoc. A jak w takim razie rozwiązać kwestię zabijania obiektu? Czy muszę używać "$dbr = null" ? Nie dałoby się tego prościej zrobić?

-- edit --

Fatal error: Access level to db::__construct() must be public (as in class PDO) in /XXX/class.Db.php5 on line 27

[bigZbig]

O co Ci chodzi z tym destruktorem? Obiekt Twojej klasy to pod wzgledem destrukcji taki sam obiekt jak kazdy inny i tak samo go traktuj.

Pokarz jak wyglada Twoja klasa po zmianach. Przetestuje ją u siebie.

[elnino.pl]

[PHP] pobierz, plaintext 
<?php
class db extends PDO
{
	private static $instance;	
 
	public static function singleton()
	{
		if(!isset(self::$instance))
			self::$instance = new db();
		return self::$instance;
	}
 
	public function __construct()
	{
		require_once('../config.php5');
		parent::__construct($dbd['type'] . ':host=' . $dbd['server'] . ';dbname=' . $dbd['db'], $dbd['user'], $dbd['pass']);
		parent::query("SET CHARACTER SET utf8"); 
		define('P', $dbd['prefix']);
		define('HASH', $dbd['hash']);
	}
}
?>
[PHP] pobierz, plaintext 

Public, bo na private się sypie.

I jeszcze jeden kłopot mam z innym kontruktorem. Klasa implementuje interfejs:

[PHP] pobierz, plaintext 
<?php
interface action
{
	function __construct();
	function __destruct();
 
	static function process();	
}
 
class page_view_action implements action
{
//
	private static $instance;	
 
	public static function process()
	{
		if(!isset(self::$instance))
			self::$instance = new page_view_action();
		return self::$instance;
	}	
 
	private function __construct()
	{
//
	}
 
	public function __destruct()
	{
 
	}
}
?>
[PHP] pobierz, plaintext 

Dla powyższego kodu mam bląd: "Fatal error: Access level to page_view_action::__construct() must be public (as in class Action)"

Jeśli zmienię w interfejsie konstruktor na prywatny dostaję błąd: "Fatal error: Access type for interface method Action::__construct() must be omitted"

[bigZbig]

1. Konstruktor klasy db musi byc publiczny poniewaz konstruktor klasy PDO jest publiczny (zapewne implementuje jakis interfejs, ktory tego wymaga).

2. W interfejsie definiuje sie jedynie metody publiczne. Do tego sluzy interfejs - chodzi o to, ze dany obiekt komunikuje sie ze swiatem zewnetrznym za pomoca metod publicznych. Dlatego nie mozesz w intefejsie zadeklarowac metody prywatnej lub chronionej. Wyrzuc konstruktor i destruktor z interfejsu.

[elnino.pl]

Dzięki. Nadal jednak nie wiem co z tym zabiciem uchwytu do bazy danych. Czy muszę null'ować zmienną?

Ten post edytował elnino.pl 21.07.2006, 13:26:42

[bigZbig]

Zamiast Twojej klasy sprobuj tego (pisane z reki)

[PHP] pobierz, plaintext 
<?php
class db
{
	private static $instance = false;
 
	public static function singleton()
	{
		if((self::$instance === false))			
			self::$instance = db::createPdoInstance();
		return self::$instance;
	}
 
	private function __construct()
	{}
 
	private static function createPdoInstance()
	{
	  require_once('../config.php5');
	  $db = new PDO($dbd['type'] . ':host=' . $dbd['server'] . ';dbname=' . $dbd['db'], $dbd['user'], $dbd['pass']);
	  $db->query("SET CHARACTER SET utf8");
	  define('P', $dbd['prefix']);
	  define('HASH', $dbd['hash']);
	  return $db;
	}
}
$db = db::singleton();
?>
[PHP] pobierz, plaintext 

Co do niszczenia polaczenia. Skoro masz tylko jedno to pozwol mu zwyczajnie wygasnac przy zakonczeniu skryptu.

-- edit --
Klasa db nie moze byc rozszerzeniem klasy PDO

Ten post edytował bigZbig 21.07.2006, 13:35:13

[elnino.pl]

Niestety to samo.

A jeśli chodzi o niszczenie - kiedyś miałem kłopot, bo otwierałem połączeina mysql (standardowo) i na końcu ich nie zamykałem. Administrator serwera powiedział, że te wszystkie połączenia "wiszą" ciągle otwarte (pomimo tego, że nie były pconnect)...

[bigZbig]

Zrobilem poprawki zapomnialem usunac extends PDO

Patrz wyzej

[elnino.pl]

Dzięki, gra i buczy!