Abstrakcyjna klasa obslugi bazy danych Opcje

[Beynar]

Napisalem abstrakycjna klase do obslugi bazy mysql
Co o niej sadzicie?
Chce na niej bazowac w duzym projekcie i nie chcialbym wtopic
Jak robic cachowanie? Nie mam pomyslu...

[PHP] pobierz, plaintext 
<?php
 
 class db{
 
	 public $_link;		// connection id
	 public $_database;		// selected db
	 public $_query;		// current query
	 public $_resource;		// data resource
	 public $_data;		// data
	 public $_rows;		// number of processed rows
	 public $_queries=0;		// number of queris
	 public $_lastId;		// last primary key
	 
	 
	 public function connect($host, $login, $pass, $_database){
	 if($_link=mysql_connect($host, $login, $pass))
		 $this -> _link = $_link;
	 else{
		 $this -> error();
		 return false;
		 }
	 if($_database=mysql_select_db($_database))
		 $this -> _database = $_database;
	 else{
		 $this -> error();
		 return false;
		 }
	 return true;
	 }
 
		 
	 public function query($_query){
	 if(!$this -> _link || !$this -> _database){
		 $this -> error("can not do _query - no _database selected");
		 return false;
		 }
	 $this -> _query = $_query;
	 if($this -> _resource)
		 @mysql_free_result($this -> _resource);
	 if($_resource=mysql_query($_query, $this -> _link))
		 $this -> _resource = $_resource;
	 else{
		 $this -> error();
		 return false;
		 }
	 $this -> _queries++;
	 $this -> _lastId = mysql_insert_id($this -> _link);
	 return $this -> _resource;
	 }
 
	 public function getData(){
	 if(!$this -> _link || !$this -> _database || !$this -> _resource){
		 $this -> error("can not get data from resource - no database selected or there is no data recosurce");
		 return false;
		 }
	 while($_data=mysql_fetch_array($this -> _resource, MYSQL_ASSOC))
		 $this -> _data[]=$_data;
	 if(!is_array($_data))
		 return $this -> _data;
	 else
		 $this -> error();
	 }
		 
	 public function getOnce(){
	 if(!$this -> _link || !$this -> _database || !$this -> _resource){
		 $this -> error("can not get data from resource - no database selected or there is no data recosurce");
		 return false;
		 }
	 if($_data=mysql_fetch_array($this -> _resource, MYSQL_ASSOC)){
		 $this -> _data[0]=$_data;
		 return true;
		 }
	 else
		 return false;
	 }
 
	 public function numRows(){
		 if(!$this -> _link || !$this -> _database || !$this -> _resource){
		 $this -> error("can not get data from _resource because no _database selected or there is n
o data recosurce");
		 return false;
		 }
	 $_rows=mysql_num_rows($this -> _resource);
	 $this -> _rows = $_rows;
	 return $_rows;
	 }
 
	 
	 public function flush(){
	 if($this -> _resource)
		 @mysql_free_result($this -> _resource);
	 unset($this -> _query);
	 unset($this -> _resource);
	 unset($this -> _data);
	 unset($this -> _rows);
	 $this -> _queries=0;
	 unset($this -> _lastId);
	 return true;
	 }
 
	 private function error($error=false){
	 if($error)
		 echo "<p style=\"font-weight:bold;color:red;\">".$error."</p>";
	 echo "<p style=\"font-weight:bold;color:red\">".mysql_errno()." = ".mysql_error()."</p>";
	 if($this -> _query!="")
		 echo "<p style=\"font-weight:bold;color:red\">".$this -> _query."</p>";
	 }
	 
	 public function __destruct(){
	 $this -> flush();
	 if($this -> _link && $this -> _db){
		 if(!mysql_close($this -> _link)){
		 $this -> _error();
		 return false;
		 }
		 else
		 return true;
		 }
	 else 
		 return true;
	 }
 }
 ?>
[PHP] pobierz, plaintext 

Ten post edytował Beynar 11.12.2007, 22:59:41

[Beynar]

@ayeo ->
"Właściwości powinny być private, używaj metod dostępowych gdzie trzeba." - OK, przerobie to ale co to daje?

"Łączenie z baza to zadanie dla konstruktora (zamykanie polaczenia dla destruktora)." - tak zrobilem na poczatku ale problemy zaczely sie pojawiac gdy od-serializowywuje. Jak wtedy moge podac argumenty dla konstruktora do polaczenia?

"Sama klasa powinna być Singletonem (albo częścią rejestru)." - Czytalem o singletoenie itd... ale dalej nie wiem jak moge to zaimplementowac w prkatyce?

"Żadna metoda klasy nie powinna (mam tu na myśli error()) wysyłać nic bezpośrednio na wyjście (echo)." zgadzam się, takie rozwiaznie jest tylko na czas rozwijania aplikacji, ulatwi nam debugowanie. Pozniej ta linijke zamienie na zapis do logow.

"Sama warstwa abstrakcji powinna korzystać ze sterownika bazy danych (osobna klasa) i na podstawie parametrów wybierać odpowiedni (np MYSQL)" - zatem pomyliłem chyba pojecia- nie jest to warstwa abstrakcyjna w takim razie. Nastawiamy sie tylko na mysql, tyle ze ta klasa bedzie bazowa i uniwersalna, i w razie zmiany bazy (co jest jednak bardzo mało prawdopodobne) bedzie mniej problemow.

"Nazywanie publicznych właściwości od _ jest mylące, ale o w sumie Twoja sprawa." - prefix _ ma zapewnic unikalnosc, gdyz zamierzam w wyzszych warstwach aplikacji korzystajacaych z bazy dziedziczyac ta funkcje i nie chce miec problemow jesli ktos nada potomnej klasie wlasciwosc o tej samej nazwie

"Poza tym i tak dalej w kodzie używasz czystego SQLa." - zatem robic takie funkcje jak insertData($table, $data), deleteRecord($primary_id_name, $id)? I tak nie unikne wpisywania skladni SQL w skryptach bo niektore zapytania beda zbyt zawile zeby to skracac przez wyprowadzenie zmiennych w metodzie.

Dzieki za rady.

Pozdrawiam

Ten post edytował Beynar 12.12.2007, 19:33:33