Singleton, spróbowałem zrobić i niestety nie działa :( Opcje

[Aztech]

To zacznijmy od początku.
Mam sobie taką klasę bazową (zostawiam najważniejsze elementy)

[PHP] pobierz, plaintext 
<?php
 
/**
 * @author Tomasz Suchanek
 * @copyright Tomasz Suchanek 
 * @final 
 */
 
require_once('config.inc.php');
require_once('$RBX_CORE/includes/errors.inc.php');
require_once('$RBX_CORE/includes/constant.inc.php');
 
/**
 * Klasa bazowa dla wszystkich klas
 *
 */
abstract class BaseClass {
 
  /**
   * Połaczenie z bazą danych
   *
   * @var array Przechowuje identyfiaktor połączenia z bazą danych.
   * @access private
   */
  private $db;
 
  /**
   * Wynik zapytania
   *
   * @var array Wynik zapytania skierowanego do bazy
   * @access private   
   */
  private $result;
 
  /**
   * Zapytanie do bazy
   *
   * @var string Zapytanie kierowane do bazy
   * @access private
   */  
  private $sql;
 
  /**
   * Konstruktor
   * @access public
   */	  
  public function __construct(){
 
	global $RBX_DB;
 
	# połącz z bazą typu MySQL
	$this->db = new PDO('mysql:host='.$RBX_DB['host'].';dbname='.$RBX_DB['name'], $RBX_DB['user'], $RBX_DB['pass']);
 
	#polskie litery
	$this->setSql('SET NAMES latin2;
	SET collation_connection = latin2_general_ci;');
	$this->doSQL();
 
	# raportuj wszystkie błędy z PDO
	$this->db->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
 
	# raportuj wszysrkie błedy z php
	error_reporting(E_ALL);
 
  }
 
  /**
   * Ustawia wartości domyślne dla pól prywatnych klasy
   * 
   * @abstract 
   *
   */
  abstract public function setDefault();
 
  /**
   * Zwróc obiekt połączenia z bazą
   *
   * @return array Tablica z identyfiaktorem połączenia z bazą danych
   * @access public
   */	 
  public function getDB(){
	return $this->db;
  }
 
  /**
   * Zwraca rezultat zapytania
   *
   * @return array Tablica zawierająca rezultat zapytania
   * @access public
   */
  public function getResult(){
	return $this->result;
  }
 
  /**
   * Ustaw zapytanie sql
   *
   * @param string $sql Zapytanie SQL
   * @access public   
   */
  public function setSql($sql) {
	$this->sql = $sql;
  }
 
  /**
   * Zwróć treść zapytania
   *   
   * @return string $sql Zapytanie SQL
   */
  public function getSql() {
	return $this->sql;
  }
 
  /**
   * Zwróć wyjątek jeśli rezultat zapytania jest pusty
   * 
   * @return true jeśli wynik zapytania jest poprawny
   * @access public
   */
  public function checkResult() {
	if ($this->result) {
	  return true;
	}
	else {
	  return false;
	}
  }
 
  /**
 * Wywołanie wszytkich metod związanych z wykonaniem zapytanie w jednej funkcji
   * treść zapytania zostaje zapamiętana
   *
   * @param string $sql Zapytanie SQL
   * @access public
   * @return boolean
   */  
  public function doSQL($sql=''){
 
	# gdy niepusty to zapisz zapytanie SQL
	if (!empty($sql)) $this->setSql($sql);
 
	#wykonaj zapytanie i zapisz do wyników
	$stmt = $this->db->prepare($this->getSql());
	$stmt->execute();
	$this->result = $stmt->fetchAll(PDO::FETCH_ASSOC);
	if (isset($this->result[0])) {
		 $this->result = $this->result[0];
	 } 
	$stmt->closeCursor();	   
	return $this->checkResult();
  }
}
 
?>
[PHP] pobierz, plaintext 

i chciałem skorzystać z wzorca Singleton (osatnio czytałem i postanowiłem przerobić to tak, żeby działało)
Więc zrobiłem tak:
$db jest statyczne

[PHP] pobierz, plaintext 
<?php
private static $db = false;
?>
[PHP] pobierz, plaintext 

getDB zmieniłem żeby działało jak getIsntance

[PHP] pobierz, plaintext 
<?php
public function getDB(){
	global $RBX_DB;
	if (self::$db == false) {
	  self::$db = new PDO('mysql:host='.$RBX_DB['host'].';dbname='.$RBX_DB['name'], $RBX_DB['user'], $RBX_DB['pass']);
	  #polskie litery
	  $this->doSQL("SET NAMES latin2; SET collation_connection = latin2_general_ci;");
	  # raportuj wszystkie błędy z PDO
	  $this->db->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
	}
	return self::$db;
  }
?>
[PHP] pobierz, plaintext 

a kontruktor teraz wygląda tak

[PHP] pobierz, plaintext 
<?php
public function __construct(){
 
	global $RBX_DB;
 
	$this->getDB();
	# raportuj wszysrkie błedy z php
	error_reporting(E_ALL);
  }
?>
[PHP] pobierz, plaintext 

Pozostałość bez zmian.
Mam inne klasy, które dziedziczą po BaseClass i które wywołują w swoim kontruktorze kontruktor rodzica oraz korzystają z $this->doSQL(); Niestety wywołuje too błąd:

Kod

atal error: Cannot access empty property in C:\wamp\www\robotECS\core\classes\baseclass.php on line 141

(to ta linia 143 z pierwszego listingu)

Dlaczego właściwość jest pusta? Pewnie coś robię źle, jeśli tak to co? Czy wywołanie $this->db->jakaśFunkcja() w funkcjach dziedziczacych po baseClass jest porpawne jeśli nie to jak to robić?

[orson]

witam ...

Zamień $this->db na self:$db - to jest static ... ale też nie wiem czy będzie działać (musisz chyba przepisać to do zmiennej bo nie pamiętam czy można napisać self:b->metoda)

pozdrawiam

[Aztech]

Zmieniłem wszędzie w klasie bazowej na self::$db i póki co wszystko działa.
Z innej beczki.
Mam klasę User dziedziczącą po BaseClass i funkcję która wywołuje mi coś takiego:

[PHP] pobierz, plaintext 
<?php
function getUserInfoById($user_id) {
  $user = new User($user_id);
  $user->readDataFromSqlById();
  $pdo = $user->getDB();
  #ilość postów napisanych przez użytkownika
  $stmt=$pdo->prepare('SELECT count(*) as posts FROM rbx_posts WHERE author_id='.$user_id);
  $stmt->execute();
  /* i dalej leci jakiś tam kod ...*/
}
?>
[PHP] pobierz, plaintext 

konstruktor usera wywołuje kostruktor bazowy: parent::__construct()
Jeśli dobrze rozumiem to:
- dzięki temu co napisałem (mam nadieję, że to dobry Singleton jest ) jeśli była stworzona już instancja $db to nie powstanie mi nowa, tylko będę korzystał z już istniejącej
- $pdo = $user->getDB(); zwróci mi do zmiennej $pdo tego static $db dzięki czemu nie postanie żadne nowe połączenie tylko będę korzystał z już istniejącego

Pytanie za 100pkt. Jak sprawdzić czy działa to tak jak napisałem tzn czy mimo wszystko jakimś cudem nie jest tworzone nowe połączenie do bazy?

[orson]

witam

$user = new User($user_id); daje nowy obiekt ... musisz odwoływać się przez getDB ($user = BaseClass::getDB)

by sprawdzić ile razy tworzy się pdo wewnątrz getDB w ifie daj sobie printa albo jakieś logowanie do pliku

a tak wogóle to powinno być tak: konstruktor powinien być private i do obiektu powieneś się odwoływać tylko przez ::getDB z klas potomnych oraz z zewnątrz ... tworzenie pdo przenieś do konstrukra ... i w getDB tylko zwracanie zapisanej instancji ...

pozdrawiam

[Aztech]

Pobawiłem się trochę moim kodem i okazało się, że chyba nie jest to do końca Singleton albo ja coś spaprałem.
Oto co zrobiłem
BaseClass.php

[PHP] pobierz, plaintext 
<?php
/**
   * Połaczenie z bazą danych
   *
   * @var array Przechowuje identyfiaktor połączenia z bazą danych.
   * @access private
   */
  private static $db = false;
 
  /* ... */
 
  /**
   * Konstruktor
   *
   * @access public
   */	  
  public function __construct(){
 
	global $RBX_DB;
 
	if (self::$db == false) {
	  self::$db = new PDO('mysql:host='.$RBX_DB['host'].';dbname='.$RBX_DB['name'], $RBX_DB['user'], $RBX_DB['pass']);
	  # raportuj wszystkie błędy z PDO
	  self::$db->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
	  #polskie litery
	  #wykonaj zapytanie i zapisz do wyników
	  $stmt = self::$db->prepare('SET NAMES latin2; SET collation_connection = latin2_general_ci;');
	  $stmt->execute();
	  echo '<br>Tworzę połączenie z bazą danych';
	}
 
	# raportuj wszysrkie błedy z php
	error_reporting(E_ALL);
  }
 
  /* ... */
 
  /**
   * Zwróc obiekt połączenia z bazą
   *
   * @return array Tablica z identyfiaktorem połączenia z bazą danych
   * @access public
   */	 
  public static function getDB(){
	return self::$db;
  }
 
  /* ... */
 
  /**
* Wywołanie wszytkich metod związanych z wykonaniem zapytanie w jednej funkcji
   * treść zapytania zostaje zapamiętana
   *
   * @param string $sql Zapytanie SQL
   * @access public
   * @return boolean
   */  
  public function doSQL($sql=''){
 
	# gdy niepusty to zapisz zapytanie SQL
	if (!empty($sql)) $this->setSql($sql);
 
	#wykonaj zapytanie i zapisz do wyników
	$stmt = self::$db->prepare($this->getSql());
	$stmt->execute();
	$this->result = $stmt->fetchAll(PDO::FETCH_ASSOC);
	if (isset($this->result[0])) {
		 $this->result = $this->result[0];
	 } 
	$stmt->closeCursor();	
	return $this->checkResult();
  }
?>
[PHP] pobierz, plaintext 

Później podczas logowania wywoływana jest klasa LoginUSer, która dziedziczy po User a ta po BaseClass, loguję się do aplikacji, czyli jest tworzona instancja obiektu user, wywołuje się kontruktor BaseClass.php, można się zatem odwołać do połączenia z bazą poprzez BaseClass:$db i tutaj się myliłem, bo taki kod nie chce mi zadziałąć.

[PHP] pobierz, plaintext 
<?php
/**
 * Zwraca listę forów wyświetlanych wg ustalonej kolejności (wg app_order a nie wg id)
 *
 * @return array tablica asocjacyjna
 */
function getForumList()
{
  $pdo = BaseClass::getDB();
  $stmt=$pdo->prepare("SELECT * FROM rbx_forums AS rf"
  ." LEFT JOIN rbx_permissions AS rp ON rf.forum_id=rp.forum_id"
  ." WHERE rp.subforum_id=0 AND rp.topic_id=0"
  ." ORDER BY rf.app_order ASC"); 
  $stmt->execute();
  return $stmt->fetchAll(PDO::FETCH_ASSOC);
}
?>
[PHP] pobierz, plaintext 

Czy ma jakiekolwiek znaczenie to, że instancja klasy user jest wywoływana w pliku index.php a getForumList() w rbx.php?

Ten post edytował Aztech 12.07.2006, 18:56:06