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

[Crozin]

1) To nie jest poprawna implementacja singletona. Nadal istnieje możliwość stworzenia wielu obiektów - poprzez klonowanie.
2) Taka drobna rada. Dla Uzytkownik::oInstance ustaw domyślnie null, nie false. Widzę, że użyłeś tego śmiesznego sposobu zapisywania nazw zmiennych (gdzie pierwszy znak określa typ zmiennej - o: object) (właśnie, jak to się nazywało? :]) więc wypada być konsekwentnym i nie przypisywać do niej false (typ: boolean)