Klasa do obsługi bazy danych...poprawcie Opcje

[Joachim Peters]

Od niedawna dopiero zaczełem interesować się php5 i dzisiaj napisałem sobie taką klase do obsługi bazy danych:

[PHP] pobierz, plaintext 
<?php
class DB 
{
	 private $dbc;
	 public $sql_queries = 0;
 
	 public function __construct($db_user, $db_name, $db_pass, $db_host)
	 {
	 $this->dbc = @mysql_connect($db_host, $db_user, $db_pass) OR die ('Nie mogłem połączyć się z MySQL-em: ' . mysql_error() );
	 @mysql_select_db($db_name) OR die ('Nie udało mi się wybrać bazy danych: ' . mysql_error() );
	 }
 
	 public function query($sql) 
	 {
	 $this->sql_queries++; 
	 return @mysql_query($sql);
	 }
 
	 public function fetch($result) 
	 {
	 return mysql_fetch_array($result);
	 }
 
	 public function show_query() 
	 {
	 print "<br />Zapytań SQL: <b>{$this->sql_queries}</b>";
	 }
 
}
?>
[PHP] pobierz, plaintext 

i przykład:

[PHP] pobierz, plaintext 
<?php
$db = new DB('root', 'user', 'pass', 'localhost');
 
$sql = "SELECT * FROM table";
$result = $db->query($sql);
 
while($row = $db->fetch($result)) 
{
echo $row['cos'];
}
 
$db->show_query();
?>
[PHP] pobierz, plaintext 

tak jak pisałem wyżej, dopiero zaczynam z php5 więć moge robić błędy...proszę o pisanie co jest w tej klasie żle napisane i uwagi co moge dodać.

Pozdrawiam

[athabus]

d@ro - jakiś czas temu też próbowałem napisać coś do obsługi danych. Wydaje mi się, że powinieneś zadbać a kilka rzeczy:
- możliwość iterowania wyników zapytania
- możliwość informowania jakie zapytania zostały wykonane itp.

Moim zdaniem warto również zadbać o możliwość cachowania wyników - ja akurat tworzyłem rozwiązanie dla PostgreSql i były wyniki zachęcające - co do Mysql'a zdania były podzielone czy to daje jakiś wzrost efektywności - mi wydaję się, że tak, ale nie testowałem.

Tutaj moje rozwiązanie

Nie jest to nic wielkiego i trzeba na pewno jeszcze pare rzeczy dodać, ale na początek jest ok. Oczywiście takie rzeczy warto pisać dla wprawy, bo zazwyczaj gotowe rozwiązania są dużo lepsze - mądrzy ludzie nad nimi pracują (IMG:http://forum.php.pl/style_emoticons/default/biggrin.gif)