Klasa do obsługi mySQL. Opcje

[boosik01]

Witam! Od 5 dni uczę się OOP. Pewnie jak każda osoba która uczy się OOP naukę rozpoczyna przy pisaniu klasy do obsługi mysql.
No więc i tak ja też uczyniłem. Chciałbym byście spojrzeli na moją klasę i postarali się jakoś mi pomóc (co dodać, poprawić etc.)
Chcę napisać jeszcze klasę do zapisywania logów z błędów, i tu pojawia się pytanie czy w OOP zapisywanie do pliku jest normalnie za pomocą funkcji
fwrite() czy też jest jakaś inna funkcja bardziej nadająca się do tego typu operacji.

PS. Pod klasą dałem utworzenie obiektów, bo nie chciało mi się robić już tego w osobnych plikach.

[PHP] pobierz, plaintext 
<?php
/**
 *  Prosta klasa obsługi bazy danych MySQL
 *  Author: boosik01
 *  (c) 2011
 */
error_reporting(E_ALL);
class mySQL
{
private $sConfig ;
 
	public function __construct($sConfig = '')
	{
		if(!$sConfig)
		{
			$this->host = $sConfig = 'localhost' ;
			$this->user = $sConfig = 'root' ;
			$this->pass = $sConfig = '' ;
			$this->db = $sConfig = 'test' ;
 
				if(!@$this->connect = mysql_connect($this->host, $this->user, $this->pass))
				throw new Exception ('Nie można połączyć się z bazą danych! ' . mysql_error() );
				if(!@mysql_select_db($this->db)) 
				throw new Exception ('Błąd wyboru bazy danych! ' . mysql_error());
		}
		else
		{
		$sConfig == NULL ;
		}
	}
 
	public function __destruct()
	{
		mysql_close($this->connect) ;
	}
 
	public function mysql_fetch_array($mysql_query)
        {
        $this->mysql_query = $mysql_query ;
        if(!@$this->mysql_query = mysql_fetch_array(mysql_query($this->mysql_query)))
		throw new Exception ('Wystąpił błąd w wyświetlaniu / lub UPTADE danych. ' . mysql_error());
		return $this->mysql_query ;
 
         }
 
	public function query($query)
	{
		$this->query = $query ;
		if(!@$this->query = mysql_query($this->query)) 
		throw new Exception ('Wystąpił błąd w konstrukcji zapytania. ' . mysql_error()) ;
		return $this->query ;
	}
}	
 
try
{
$new = new mySQL() ;
$dane = $new->query("INSERT INTO users SET user='boooooo'") ;
echo $dane['user'] ;
} 
catch (Exception $error)
{
echo 'Wystąpił błąd. Administracja została już o tym poinformowana. Przepraszamy za kłopoty.' ;
$sError = '<b>Komunikat</b>: '.$error->getMessage().'<br /><b>Plik</b>: '.$error->getFile().'<br /><b>Wiersz</b>: '.$error->getLine();
}
 
[PHP] pobierz, plaintext 

Ten post edytował boosik01 30.05.2011, 18:39:46

[wookieb]

Ok.
Wywalić i użyc PDO

[melkorm]

Nie zaczynaj od pisania, zacznij od czytania / przeglądania gotowych już komponentów.

Tematów na temat `[Ocena]Moja super klasa do obsługi baz danych` było mnóstwo i w kilku niektórzy znaleźli trochę czasu i napisali wiele porad jak taka klasa w ogóle powinna wyglądać, a najlepiej przejrzyj sobie źródła już istniejących modułów do obsługi baz danych, jak np. PDO o którym już wspomniał wookieb.

P.S. Kod to nie ZOO, więc małpą mówimy NIE, szczerze aż dziw bierze że to jeszcze istnieje i jest używane _^_.

Ten post edytował melkorm 30.05.2011, 19:19:42

[boosik01]

PDO jest klasą samą w subie tak? Więc pisanie klasy do obsługi bazy danych odpada więc to jest + bo mogę się już zabrać od razu za rejestrację etc...

Do tego dobra obsługa wyjątków, brak ataków sql Injection no, no ciekawe (IMG:style_emoticons/default/biggrin.gif)

Ok napisałem klasę do łatwej obsługi PDO (IMG:style_emoticons/default/wink.gif)

[PHP] pobierz, plaintext 
<?php
error_reporting(E_ALL);
 
class datebase
{
private $sSql ;
private $pdo ;
	public function __construct($sSql = NULL)
	{
		$this->host = $sSql = 'localhost' ; // Host bazy danych. 
		$this->user = $sSql = 'root' ;     // Użytkownik bazy danych. 
		$this->password = $sSql = '' ;    // Hasło do bazy danych.
		$this->db = $sSql = 'test' ;     // Wybór bazy.
		$this->pdo = $pdo = NULL ;
			try
			{
			$this->pdo = new PDO('mysql:host='.$this->host.';dbname='.$this->db.'', $this->user, $this->password);
			$this->pdo -> setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
			}
			catch(PDOException $e)
			{
			echo '<b>Kominikat:</b> Połączenie nie mogło zostać utworzone. <br> <b>Info:</b>  ' . $e->getMessage() . ' <br> <b>Wiersz:</b> ' . $e->getLine(). ' <br> <b>Plik: </b> ' .$e->getFile() ;
			}
 
	}
 
	public function __destruct()
	{
		$sSql = NULL ;
	}
 
	public function select($select)
	{
		try
		{
		$this->select = $select ;
		$this->select = $this->pdo -> query($this->select);
		return $this->select->fetch();
 
		$this->select -> closeCursor();
		}
		catch(PDOException $e)
		{
		echo '<b>Kominikat:</b> Bład z pobieraniem danych! <br> <b>Info:</b>  ' . $e->getMessage() . ' <br> <b>Wiersz:</b> ' . $e->getLine(). ' <br> <b>Plik: </b> ' .$e->getFile() ;
		}
	}
 
	public function update($update)
	{
		try
		{
		$this->update = $update ;
		return $this-> pdo -> exec($this->update);
 
		$this->update -> closeCursor();
		}
		catch(PDOException $e)
		{
		echo '<b>Kominikat:</b> Bład z aktualizowaniem rekordu! <br> <b>Info:</b>  ' . $e->getMessage() . ' <br> <b>Wiersz:</b> ' . $e->getLine(). ' <br> <b>Plik: </b> ' .$e->getFile() ;
		}
	}
 
}
 
$new = new datebase() ;
 
$new->update("UPDATE `users` SET `user` = 'asdasss11sssssssd'"); 
 
[PHP] pobierz, plaintext 

Wszystko działa jak należy, chodź pewnie ta klasa i tak jest daleka od ideału i jeśli wgl ta klasa trzyma jakieś standardy (IMG:style_emoticons/default/tongue.gif)

[wookieb]

Rozszerz PDO po prostu.
http://pl2.php.net/manual/en/keyword.extends.php
I naucz się formatować kod.
Tutaj nie ma co oceniać bo wszystko jest Nie tak.
Poza tym sprawdź czy ona naprawdę działa bo tu błąd na błędzie.

[boosik01]

Kod jest przecież sformatowany ;p Przynajmniej u mnie w notepad++.

Ok PDO rozszerzyłem.

PS. Wszystko to tzn. co? Prosze o jakieś wskazówki czy coś (IMG:style_emoticons/default/smile.gif)

Mało w sieci jest klas obsługi PDO, zaś patrząc na inne to też mam podobnie...

[konole]

Kod jest przecież sformatowany ;p Przynajmniej u mnie w notepad++.

Ok PDO rozszerzyłem.

PS. Wszystko to tzn. co? Prosze o jakieś wskazówki czy coś (IMG:style_emoticons/default/smile.gif)

Mało w sieci jest klas obsługi PDO, zaś patrząc na inne to też mam podobnie...

A po co ci klasa do obsługi PDO? PDO jest napisane w OOP przecież.

[boosik01]

Pnieważ caly czas w każdym pliku musiałbym dołączać try{}catch{}, a druga sprawa by pobrać jakieś dane trzeba maznąć tyle kodu:

[PHP] pobierz, plaintext 
<?php
 
   try
   {
      $pdo = new PDO('mysql:host=localhost;dbname=produkty', 'root', 'root');
      $pdo -> setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
 
      $stmt = $pdo -> query('SELECT id, nazwa, opis FROM produkty');
      echo '<ul>';
      foreach($stmt as $row)
      {
          echo '<li>'.$row['nazwa'].': '.$row['opis'].'</li>';
      }
      $stmt -> closeCursor();
      echo '</ul>';
   }
   catch(PDOException $e)
   {
      echo 'Połączenie nie mogło zostać utworzone: ' . $e->getMessage();
   }
?>
[PHP] pobierz, plaintext 

Czy nie lepiej napisać sobie sobie rozszerzenie klasy PDO która by to załatwiała w jednej linijce no ewentualnie w dwóch ;p ?

Ten post edytował boosik01 31.05.2011, 08:12:01

[Crozin]

Pnieważ caly czas w każdym pliku musiałbym dołączać try{}catch{}

Jeżeli nie masz potrzeby by obsługiwać błędy możesz całą stronę objąć jednym blokiem try..catch.

a druga sprawa by pobrać jakieś dane trzeba maznąć tyle kodu

Nie wiem czy zauważyłeś, ale do pobrania danych wykorzystałeś... jedną linię kodu.

Czy nie lepiej napisać sobie sobie rozszerzenie klasy PDO która by to załatwiała w jednej linijce no ewentualnie w dwóch ;p ?

Tak patrzę na ten Twój wynalazek sprzed kilku postów... nie, nie lepiej. Bo pozbawiłeś się tam jakiejkolwiek kontroli czy możliwości ingerencji w pobieranie danych.

Jeżeli już chcesz udostępnić sobie jakąś warstwę abstrakcji ponad PDO (które rzeczywiście nie ma jakiegoś idealnego interfejsu) skorzystaj lepiej z jakiegoś gotowego rozwiązania, np. Doctrine DBAL, bo na pisanie swoich jeszcze nie jesteś gotowy.

[gac3k]

Ja się jeszcze przyłączę do dyskusji, bo niepotrzebnie starasz się zrobić z psa, hipopotama. Akurat w kwestii obsługi baz danych jest tyle gotowych rozwiązań że niepotrzebne jest pisanie nowych, a tym bardziej kiepskich jak ta wyżej (IMG:style_emoticons/default/smile.gif)

Masz właśnie PDO, ale masz też np. PEAR DB, ADO DB już o wszelakiej maści ORM-ach, bez których w tym momencie nie wyobrażam sobie pracy, nie wspomnę.

Jak już koniecznie chcesz mieć własną klasę to tak jak wspominali koledzy wyżej rozszerzaj to PDO, czy jakąkolwiek inną klasę, ale są one już w sobie tak bogate że ciężko mi sobie wyobrazić jakieś specjalne ficzery które można by dodać.

[sarxos]

Uwierz mi, że nie ma sensu pisania własnej klasy do obsługi bazy danych. Zgodzę się z przedmówcami i tak samo doradzę - użyj PDO! Jeśli dziedziczenie nie pasuje ponieważ chcesz stworzyć tylko DAO do abstrakcji danych, to zrób sobie nową klasę, najlepiej aby implementowała interfejs, w którym wylistujesz wszystkie wymagane przez Ciebie metody. Pod spodem jednak tez najbardziej opłaca się użyć PDO lub MDB2 z repozytorium PEAR jeśli masz PHP 5.0 bez PECL-owego modułu dla PDO ;]