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

a tak btw. to lepiej zainteresowac sie gotowym rozwiazaniem, polecam creole

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

[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ą

[SongoQ]

A gdzie wyjatki? Dla mnie troche dziwnym pomyslem jest obecnie przy programownaniu OOP dawac die, czy zwracac false zamiast zwracac wyjatku.

[Joachim Peters]

Wyjątki...czyli jak to powinno wyglądać?

[PHP] pobierz, plaintext 
<?php
$this->dbc = @mysql_connect($db_host, $db_user, $db_pass);
if(!$this->dbc) {
throw new Exception();
}
mysql_select_db($db_name);
?>
[PHP] pobierz, plaintext 

i przykład

[PHP] pobierz, plaintext 
<?php
try {
$db = new DB('root', 'user', 'pass', 'localhost');
} catch(Exception $e) {
die('blad');
}
?>
[PHP] pobierz, plaintext 

tak?

[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.

[Joachim Peters]

Czym mam zastąpić to?

[PHP] pobierz, plaintext 
<?php
TwojException
?>
[PHP] pobierz, plaintext 

[SongoQ]

Czym chcesz przewaznie nadaje sie nazwe klasy ktora wyrzuca wyjatek clasa A => AException ale to zalezy od Ciebie jak sobie nazwiesz.

[Joachim Peters]

Mam teraz tak

[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();
	 }
 
}
 
 
try
{
$db = new DB('root', '', '', 'localhost');
 
$sql = "SELECT * FROM table";
$result = $db->query($sql);
 
while($row = $db->fetch($result)) 
{
echo $row['cos'];
}
 
$db->show_query();
$db->close();
 
} catch( DBException $e )
{
die('blad');
}
 
?>
[PHP] pobierz, plaintext 

i wywala mi
Fatal error: Class 'DBException' not found in index.php on line 52
co robie zle?

Ten post edytował d@ro 10.08.2006, 07:55:14

[mike]

A masz zdaefiniowaną klasę DBException ?

Jak nie masz to napisz sobie. Pamiętaj że musi ona dziedziczyć po klasie z Exception z biblioteki SPL.

P.S.
Exceptions :: Extending Exceptions

[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ę.

[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