Oceńcie czy kod jest ok Opcje

[marekc12]

[PHP] pobierz, plaintext 
<?php
class bazaDanych
    {
        private $serwer, $nazwa, $uzytkownik, $haslo, $idPolaczenia, $wynikZapytania;
        
        function __construct( $serwer, $nazwa, $uzytkownik, $haslo )
        {
            $this->serwer = $serwer;
            $this->nazwa = $nazwa;
            $this->uzytkownik = $uzytkownik;
            $this->haslo = $haslo;
            
            $this->idPolaczenia = mysql_connect( $this->serwer, $this->uzytkownik, $this->haslo );
            if( !$this->idPolaczenia )
                $this->blad();
 
            $dbSelected = mysql_select_db( $this->nazwa, $this->idPolaczenia );
            if( !$dbSelected )
                $this->blad();
        }
 
        function __destruct() {
            mysql_close( $this->idPolaczenia );
        }
        
        function modyfikujBaze( $trescZapytania )
        {
            $this->wynikZapytania = mysql_query( $trescZapytania, $this->idPolaczenia )
            or $this->blad( $trescZapytania );
        }
        
        function czytajBaze( $trescZapytania )
        {
            $this->modyfikujBaze( $trescZapytania );
            
            while ($row = mysql_fetch_array( $this->wynikZapytania ))
                $table[] = $row;
            
            return $table;
        }
 
        private function blad( $trescZapytania ) {
            die( mysql_errno( $this->idPolaczenia ) . ': ' . mysql_error( $this->idPolaczenia ) );
        }
    }
 
    class uzytkownik
    {
        private $bazaDanych, $id, $uprawnienia, $login, $haslo, $imie;
        
        function __construct( $bazaDanych, $id, $uprawnienia, $login, $haslo, $imie )
        {
            $this->bazaDanych = $bazaDanych;
            $this->id = $id;
            $this->uprawnienia = $uprawnienia;
            $this->login = $login;
            $this->haslo = $haslo;
            $this->imie = $imie;
        }
        
        function dodaj() {
            $this->bazaDanych -> modyfikujBaze( "INSERT INTO users (id, rights, login, pass, name) VALUES ('', '$this->uprawnienia', '$this->login', '$this->haslo', '$this->imie')\" );
        }
        
        function usun() {
            $this->bazaDanych -> modyfikujBaze( "DELETE FROM users WHERE id=$this->id&#092;" );
        }
 
        function edytuj() {
            $this->bazaDanych -> modyfikujBaze( "UPDATE users SET rights='$this->uprawnienia', login='$this->login', pass='$this->haslo', login='$name->imie' WHERE id=$this->id\" );
        }
        
        function wczytajPozycje() {
            return $this->bazaDanych -> czytajBaze("SELECT * FROM users WHERE id=$this->id&#092;" );
        }
    }
?>
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
$mySql = new bazaDanych( 'localhost', 'lala', 'asdas', 'asdkl' );
$osoba = new uzytkownik( $mySql, null, 1, 'login123', 'mojehaslo', 'marek' );
$osoba -> dodaj();
?>
[PHP] pobierz, plaintext 

Czy ten kod jest dobry? Uczę się obiektowości i zastanawiam się czy dobrze to napisałem. Baze danych zrobiłem jedną klasą, użytkownika drugą, to chyba jest dobrze. Tylko nie wiem czy dobrze to rozwiązałem, że do klasy użytkownik przekazuję obiekt bazy danych. Proszę, napiszcie mi co myślicie o tym kodzie, czy mogę pisać dalej, czy może to jest w ogóle bez sensu (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) . Nie chodzi mi o idealny sposób na zapis tego kodu(każdy z Was pisze troszeczkę inaczej), tylko o to czy mogę to rozwiązać tak jak to zrobiłem, czy mój sposób nie kłóci się z zasadami programowania obektowego.

pozdrawiam!

Ten post edytował erix 12.06.2009, 14:13:14

Powód edycji: [erix] przeniosłem

[Spawnm]

zły dział...

klasa bazaDanych jest bez sensu , na siłe dałeś mysqla w klase i koniec ...
napisz singletona czy coś... bo obecna klasa to zwykły mysql tylko wolniejszy i ma zmienione nazwy.

uzytkownik

zbyt wiele wymagasz danych do constructa , klasa zbyt wymusza dane i wiele nie daje ...