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

[pawel_k]

najlepiej zmienna ktora zwraca mysql_query trzymaj w klasie lub zwracaj obiekt na ktorym bedziesz robil fetch() (lub false gdy zle zapytanie), inaczej nie masz pewnosci ze metoda fetch sie wykona bez wywalania bladow...

poza tym brak przenoszalnosci pomiedzy bazami...

dalej to robienie die() gdy nie ma polaczenia z baza...

do tego dorzucilbym jeszcze przejscie na singletona (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

a tak btw. to lepiej zainteresowac sie gotowym rozwiazaniem, polecam creole (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

Ten post edytował pawel_k 9.08.2006, 10:39:51