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

[Ludvik]

Poza tym wypadało by wyrzucić wyjątek zanim zechcesz go przechwycić. Zapoznaj się z mechanizmem wyjątków.

Póki co, to ta klasa komplikuje Ci tylko całą sytuację. Nie implementuje całego interfejsu mysql. Skończy się to tym, że będziesz na przemian używał tej klasy i funkcji mysql_*. Jedyną rzeczą, którą wykonuje ta klasa inaczej niż rozszerzenie to zliczanie zapytań.

Fetch nie należy do interfejsu połączenia z bazą. Dane pobierasz z wyniku zwróconego przez bazę.