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

[ayeo]

Właściwości powinny być private, używaj metod dostępowych gdzie trzeba. Łączenie z baza to zadanie dla konstruktora (zamykanie polaczenia dla destruktora). Metoda numRows jest zupelnie niepotrzebna, można użyć count() bezpośrednio w kodzie. Sama klasa powinna być Singletonem (albo częścią rejestru). Żadna metoda klasy nie powinna (mam tu na myśli error()) wysyłać nic bezpośrednio na wyjście (echo). Funkcje bazy takie jak INSERT czy UPDATE powinny być zaimplementowane osobno, a nie razem z SELECT bo nie zwracają przecież tablicy wyników! Sama warstwa abstrakcji powinna korzystać ze sterownika bazy danych (osobna klasa) i na podstawie parametrów wybierać odpowiedni (np MYSQL) - wzorzec STRATEGY, FACTORY...
Nazywanie publicznych właściwości od _ jest mylące, ale o w sumie Twoja sprawa. Poza tym i tak dalej w kodzie używasz czystego SQLa. Odczyteywanie _last_id z właściwości klasy jest błędem bo jeśli inny User doda coś do bazy na innej sesji to otrzymujesz błędny wynik! Nie do zmiany bazy danych musisz tworzyć nową instancje! Brak jakichkolwiek zabezpieczeń...
Już mi się nie chce dalej, a troche jeszcze tego jest (IMG:http://forum.php.pl/style_emoticons/default/biggrin.gif)

Co do cachowania to dość rozległy temat.

PS to chyba nie jest abstrakcyjna klasa (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)

Ten post edytował ayeo 12.12.2007, 00:38:13