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

[dr_bonzo]

Nie!!!
1. Zrob tak zebys nie mial zadnego die() w bibliotece
2.

[PHP] pobierz, plaintext 
<?php
if(!$this->dbc) {
throw new Exception();
}
?>
[PHP] pobierz, plaintext 

to nie tak sie robi,

[PHP] pobierz, plaintext 
<?php
try
{
$x = new DB(...)
....
$x->query()....
}
catch( TwojException $e )
{
 ....zrob cos gdy baza "nie dziala"
}
?>
[PHP] pobierz, plaintext 

ma wyrzucic wyjatek gdy nie moze sie polaczyc/ wybrac bazy danych.