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

[marekc12]

ok, dzięki za rady (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) Co do nazewnictwa to wiem, że może jest trochę śmieszne, ale myślę, że lepiej jest się odnaleźć w kodzie, jeżli używa wymyślonych przez samego siebie nazw.

Poprawiłem(jeszcze mialem wrzucic wyjątki o których pisał em1X) klasę do bazy i obsługi użytkowników, zapewne nie jest idealna, ale wrzucam ją tu, może akurat komuś się przyda (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) ) Pozdrawiam!

[PHP] pobierz, plaintext 
<?php
class bazaDanych
{
 //singleton
 protected static $instance=null;
 protected function __construct() {}
 public static function getInstance($class=__CLASS__) {
     return empty(self::$instance) ? self::$instance=new $class : self::$instance; }
 ///////////
 
 public $serwer, $nazwa, $uzytkownik, $haslo;
 private $idPolaczenia, $iloscZapytan=0;
 
 function polacz()
 {
     if( !( $this->idPolaczenia = mysql_connect( $this->serwer, $this->uzytkownik, $this->haslo ) ) )
         return false;
 
     if( !( mysql_select_db( $this->nazwa, $this->idPolaczenia ) ) )
         return false;
  
     return true;
 }
 
 function rozlacz() {
     mysql_close( $this->idPolaczenia );
 }
 
 function modyfikujBaze( $trescZapytania )
 {
     $this->iloscZapytan++;
  
  //   echo $trescZapytania;
  
     if( $wynikZapytania = mysql_query( $trescZapytania, $this->idPolaczenia ) )
        return $wynikZapytania;
         else return false;
           
 }
 
 function czytajBaze( $trescZapytania )
 {
     if( !($wynikZapytania = $this->modyfikujBaze( $trescZapytania )) )
         return false;
 
     while ($row = mysql_fetch_array( $wynikZapytania ))
           $table[] = $row;
  
     @mysql_free_result( $wynikZapytania );
  
     return $table;
 }
}
 
class Uzytkownik
{
 //singleton
 protected static $instance=null;
 protected function __construct() {}
 public static function getInstance($class=__CLASS__) {
     return empty(self::$instance) ? self::$instance=new $class : self::$instance; }
 ///////////
 
 public $imie, $nazwisko, $login;
 private $id, $haslo;
 
 function wybierzPozycje( $id )
 {
     $id = filtrMysql( $id );
 
     if( !(is_numeric($id)) )
         return false;
 
     if( $wynik = bazaDanych::getInstance()->czytajBaze( "SELECT * FROM uzytkownicy WHERE id=$id" ) )
     {
         $this->id = $id;
      
         $this->login -> $wynik[1][2];
         $this->haslo -> $wynik[1][3];
         $this->imie -> $wynik[1][4];
         $this->nazwisko -> $wynik[1][5];
 
         return true;
     }
     else
         return false;
 }
 
 function ustawHaslo( $haslo )
 {
     if( !(dlugosc($haslo,8,20)) )
         return false;
     $haslo = filtrMysql( $haslo );
  
     $this->haslo = sha1( $haslo );
     return true;
 }
 
 function dodaj()
 {
     if( empty( $id ) & $this->sprawdzDane() )
         return bazaDanych::getInstance()->modyfikujBaze( "INSERT INTO uzytkownicy (id, login, haslo, imie, nazwisko) VALUES ('', '$this->login', '$this->haslo', '$this->imie', '$this->nazwisko')" );
     return false;
 }
 
 function usun()
 {
     if ( !empty( $id ) )
         return bazaDanych::getInstance()->modyfikujBaze( "DELETE FROM uzytkownicy WHERE id=$this->id" );
     else
         return false;
 }
 
 function edytuj()
 {
     if( !empty( $id ) & $this->sprawdzDane() )
         return bazaDanych::getInstance()->modyfikujBaze( "UPDATE uzytkownicy SET login='$this->login', haslo='$this->haslo', imie='$name->imie', nazwisko='$nazwisko->nazwisko' WHERE id=$this->id" );
     return false;
 }
 
 function wczytajWszystkich()
 {
     return bazaDanych::getInstance()->czytajBaze( "SELECT * FROM uzytkownicy" );
 }
 
 function sprawdzDane()
 {
     $this->login = filtrMysql( $this->login );
     $this->imie = filtrMysql( $this->imie );
     $this->nazwisko = filtrMysql( $this->nazwisko );
 
     if( dlugosc($this->login,3,15) & !empty($this->haslo) & dlugosc($this->imie,3,15) & dlugosc($this->nazwisko,3,15) );
         return true;
     return false;
 }
}
 
function dlugosc( $zmienna, $min, $max )
{
 if( strlen( $zmienna ) >= $min & strlen( $zmienna ) <= $max )
     return true;
 return false;
}
function filtrMysql( $zmienna )
{
 if (get_magic_quotes_gpc())
     $zmienna = stripslashes($zmienna);
 $zmienna = mysql_real_escape_string($zmienna);
 return($zmienna);
}
?>
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
$sql = bazaDanych::getInstance();
 
$sql->serwer = 'localhost';
$sql->nazwa = 'aaa';
$sql->uzytkownik = 'aaa';
$sql->haslo = 'aaa';
 
$sql->polacz();
 
Uzytkownik::getInstance()->imie="Jan";
Uzytkownik::getInstance()->login="jan12";
Uzytkownik::getInstance()->nazwisko="Kowalski";
Uzytkownik::getInstance()->ustawHaslo("passs");
Uzytkownik::getInstance()->dodaj();
 
bazaDanych::getInstance()->rozlacz();
?>
[PHP] pobierz, plaintext 

Ten post edytował marekc12 27.06.2009, 21:45:12