[PHP][MYSQL] Co sądzicie o tym kodzie? Opcje

[marekc12]

Witam!

Piszę CMSa, napisałem klasę, która będzie takim jakby 'jądrem' tego mojego CMSa (IMG:http://forum.php.pl/style_emoticons/default/biggrin.gif) Czyli po prostu klasa do obsługi bazy danych...

Kod działa, chcę tylko wiedzieć, czy w dobry sposób to rozwiązałem:) To moja pierwsza tak rozbudowana klasa :-) hehe.

Dzieki z gory za wszelkie sugestie, mam nadzieje ze ktoś bardziej doświadczony oceni kod. Pozdrawiam!

[PHP] pobierz, plaintext 
<?php
 class BazaDanych
 {
     private static $sqlSerwer = 'localhost';
     private static $sqlUser = 'uzytkownik';
     private static $sqlPass = 'haslo';
     private static $sqlName = 'nazwabazy';
     private static $sqlConn;
     private static $idZapytania;
     protected $tabela, $dane, $warunek, $flagi;
  
     public static function polacz() //metoda łącząca się z bazą danych
     {
         self::$sqlConn = mysql_connect( self::$sqlSerwer, self::$sqlUser, self::$sqlPass );
         if( !self::$sqlConn ) throw new Exception(1);
      
         if(!mysql_select_db( self::$sqlName, self::$sqlConn ))
             throw new Exception(2); 
     }
  
     public static function rozlacz() //metoda która rozłącza nas z bazą danych
     {
         if(!mysql_close( self::$sqlConn ))
             throw new Exception(3); 
     }
  
     public static function filtrMysql($zmienna) //metoda, dzięki której możemy filtrowć dane 
     {
         if(get_magic_quotes_gpc())
             $zmienna = stripslashes($zmienna);
         $zmienna = mysql_real_escape_string($zmienna);
      
         return($zmienna);
     }
 
     private static function zapytanie($tekstZapytania) //metoda wywołująca zapytanie do bazy
     {
         if(!self::$idZapytania = mysql_query( $tekstZapytania, self::$sqlConn ))
             throw new Exception(4);
         return;
     }
  
 
// poniższe metody tworzą tekst zapytania do bazy danych i wysyłają je do metody 'zapytanie'
 
// Do tego celu wykorzystują zmienne: $this->tabela, $this->dane, $this->warunek, $this->flagi - te zmienne są zmiennymi obiektów innych klas(klas-dzieci) -zobacz drugi kod, na samym dole (class Uzytkownicy extends BazaDanych)  :)
 
     protected function dodaj() //metoda dodająca nowe rekordy do bazy danych
     {
         foreach($this->dane as $klucz => $wartosc)
             $temp[] = '`'.$klucz.'`';
      
         self::zapytanie( 'INSERT INTO ' . $this->tabela . ' (' . implode(&#092;", \", $tekstZapytania) . ') VALUES (' . implode(\", \", $this->dane) . ')' );
         return;
     }
  
     protected function usun() //metoda usuwająca rekordy z bazy danych
     {
         self::zapytanie( 'DELETE FROM ' . $this->tabela . ' WHERE ' . $this->warunek );
         return;
     }
  
     protected function edytuj() //metoda edytująca rekordy w bazy danych
     {
         foreach($this->dane as $klucz => $wartosc)
             $tekstZapytania[] = '`'.$klucz.'` = ' . $wartosc;
      
         self::zapytanie( $tekstZapytania = 'UPDATE ' . $this->tabela . ' SET ' . implode(&#092;", \", $temp) . ' WHERE ' . $this->warunek );
         return;
     }
  
     protected function odczytaj() //metoda odczytująca rekordy z bazy danych, zwraca odczytane dane w postaci dwuwymiarowej tabeli
     {
         foreach($this->dane as $klucz => $wartosc)
             $temp[] = '`'.$wartosc.'`';
      
         self::zapytanie( 'SELECT ' . implode(&#092;", \", $temp) . ' FROM ' . $this->tabela . ' ' . $this->flagi );
      
         while($wiersz = mysql_fetch_row(self::$idZapytania))
             $wynik[] = $wiersz;
                  
         return($wynik);
     }
 }
?>
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
 class Uzytkownicy extends BazaDanych
 {
     function uzytkownicy()
     {
         $this->tabela = '`users`';
     }
 
     public function usun($warunek)
     {
         $this->warunek = $warunek;
         parent::usun();
     }
  
     public function edytuj($warunek, $nazwa, $uprawnienia, $haslo, $imie)
     {
         $this->warunek = $warunek;
         $this->dane = array('nazwa' => $nazwa, 'uprawnienia' => $uprawnienia, 'haslo' => $haslo, 'imie' => $imie);
         parent::edytuj();
     }
  
     public function odczytaj()
     {
         $this->dane = array('nazwa', 'uprawnienia', 'haslo', 'imie');
         return parent::odczytaj();
     }
 }
?>
[PHP] pobierz, plaintext 

Tak wywołuje kod:

[PHP] pobierz, plaintext 
<?php
BazaDanych::polacz();
 $uzytkownicy = new Uzytkownicy;
 $a=$uzytkownicy -> odczytaj();
 BazaDanych::rozlacz();
?>
[PHP] pobierz, plaintext 

Ten post edytował marekc12 15.11.2008, 23:38:33

[ayeo]

Witam!

Rozwiązałeś to bardzo niedobrze (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) Czemu właściwości (z danymi do połączenia) są statyczne? Czemu z bazą nie łączy się konstruktor? Ten ostatni kod to całkowity bezsens! Poza tym wygodniej korzystać z SQLa niż z metody pobierającej 5 argumentów. Klasa użytkownicy powinna mieć właściwości odpowiadające kolumnom w bazie:

[PHP] pobierz, plaintext 
<?php
$user = new Uzytkownicy;
$user->name = "Adam";
$user->surname = "Kowalski";
$user->select();
echo $user->id;
?>
[PHP] pobierz, plaintext 

Metoda select sama sobie powinna zbudować warunek na podstawie $this->name i $this->surname i załadować resztę danych do właściwości. Oczywiście zakładając, że obiekt Uzytkownicy reprezentuje jeden wiersz z tabeli. Jednak jeżeli jest (wyników) więcej to powinniśmy dostać kolekcję obiektów Uzytkownicy (Uzytkownik). Zresztą to już jest DAO, a Ty chciałeś napisać klasę bazy danych. Więc do metody select(); przekazuj po prostu SQL.

Podsumowując, Twój kod jest raczej do wyrzucenia (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

Pozdrawiam!

[Mize]

Uzytkownik dziedziczy po bazie danych ? (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)
Jeśli chcesz mieć dostęp do bazy danych to zainteresuj się czym jest context lub registry.
Hmm, nie widze tu sensu stosowania statycznych metod.

ayeo, polaczenie nie musi powstawać w konstruktorze, to już raczej dowolność zależna od programisty.

[marekc12]

Ok:) Dzięki za odpowiedzi:> Dokształce się jeszcze (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) )