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

[em1X]

1) Konstruktor służy (w teorii) wyłącznie do przypisywania obiektowi pewnych wartości więc ten mysql_connect tam jest bez sensu, powinien się raczej znaleźć w metodzie connect(). W przypadku rozszerzania klasy miałbyś problem, poza tym ograniczasz sobie/innym pole manewru.

2) Pola w klasie mysql powinny być protected nie private. W przypadku rozszerzania klasy nie byłyby dostępne.

3) te przekazywanie uchwytu do mysqla dla użytkownika jest bez sensu, lepiej skorzystać z singletona bądź wzorca factory z jakiejś globalnej klasy

4) klasy nie powinny wyświetlać żadnych błędów, ponieważ narzucasz je z góry mimo, że mogą być niepotrzebne. Zamiast tego wyrzucaj wyjątek, który ktoś może obsłużyć lub nie - ma wybór.

Ten post edytował em1X 25.06.2009, 15:51:42