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

[Joachim Peters]

Zrobiłem coś na podstawie tego co @Ludvik napisał mi w innym temacie o bazie danych.

[PHP] pobierz, plaintext 
<?php
 
 
class DB 
{
	 public $sql_queries = 0;
 
	 public function __construct($db_user, $db_name, $db_pass, $db_host)
	 {
	 $this->handle = mysql_connect($db_host, $db_user, $db_pass);
	 if (!$this->handle) {
	 throw new Exception();
	 }
	 mysql_select_db($db_name, $this->handle);
	 }
 
	 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>";
	 }
	 
	 public function close()
	 {
	 return mysql_close();
	 }
 
protected $handle;
 
}
 
try {
$db = new DB('root', '', '', 'localhost');
} catch(Exception $e) {
die ('Nie można nawiązać połączenia z bazą danych.');
}
 
$sql = "SELECT * FROM ttt";
$result = $db->query($sql);
 
while($row = $db->fetch($result)) 
{
echo $row['cos'];
}
 
$db->show_query();
 
 
?>
[PHP] pobierz, plaintext 

ale teraz gdy nie ma połączenia z bazą (złe dane) to nic się nie dzieje oprócz wywalenia błędu odnośnie fetch'a.

probowałem też coś takiego

[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);
	 mysql_select_db($db_name);
	 }
 
	 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>";
	 }
	 
	 public function close()
	 {
	 return mysql_close();
	 }
 
}
 
class DBException extends Exception {};
 
try {
$db = new DB('root', '', '', 'localhost');
} catch(DBException $e) {
die ('Nie można nawiązać połączenia z bazą danych.');
}
 
$sql = "SELECT * FROM tabbe";
$result = $db->query($sql);
 
while($row = $db->fetch($result)) 
{
echo $row['title'];
}
 
$db->show_query();
 
 
?>
[PHP] pobierz, plaintext 

ale jest tak samo..proszę o pomoc

Ten post edytował d@ro 10.08.2006, 16:22:26