user class + dziedziczenie, Ocena klasy Opcje

[PawelC]

Hej piszę sobie klasę users, która jest odpowiedzialna za logowanie i dodanie użytkownika, później dojdzie kasowanie i edycja konta. Moje pytanie brzmi czy sposób w który piszę jest dobry:

[PHP] pobierz, plaintext 
<?
class user extends sqli
  {
 
    public function userLogin()
      {    
 
       $this->log=trim($_POST['log']);
       $this->pass=$pass=trim($_POST['pass']);
       $result=$this->mysqli->query("select log,pass from users where log='$this->log' and pass='$this->pass'");
        if(mysqli_num_rows($result)==0)
            {
                echo "brak takiego usera";
             }
             else
             {
                $_SESSION['zalogowany']="$this->log";
                header("Location: test.php");
              }
 
 
            }
            
            
   public function addUser()
            {
 
             $this->log=trim($_POST['log']);
             $this->pass=$pass=trim($_POST['pass']);
             $result=$this->mysqli->query("insert into users values('$this->log','$this->pass')");
             if($result===true)
              {
                  echo "Użytkownik został dodany poprawnie.";
              }
               else
               {
                  echo "Wystąpił błąd podczas dodawania nowego użytkownika!";
              }
 
            }
 
 
 
        }
$user=new user('localhost','root','','katalog');
?>
[PHP] pobierz, plaintext 

I dodawanie użytkownika i samo logowanie działa poprawnie, a mi chodzi o Waszą ocenę czy to jest dobrze napisane. Oczywiście zdaję sobie sprawę, że można to napisać zawsze lepiej.

Ten post edytował ExPlOiT 2.05.2009, 22:57:27

[PawelC]

Sorki że wracam do tego tematu, ale wyczarowałem pewny kod, a dokładnie wykorzystanie klasy sqli w klasie user i chciałem się dowiedzieć czy dobrze to zrobiłem. Oto kod:

[PHP] pobierz, plaintext 
<?php
 
 
        class sqli
            {
                
                
                
                //Konstruktor
            public function __construct($host,$login,$haslo,$baza)
            {
                $this->mysqli=new mysqli($host,$login,$haslo,$baza);
            }
            
            
            
            
            
            public function getRow($query)
            {
                return $this->mysqli->query($this->query=$query)->fetch_object();
                
            }
                    
            
            
            
            // Pobranie danych z bazy
            public function getRows($query)
            {
                $dane=array();
                $result=$this->mysqli->query($this->query=$query);
                while($wynik=$result->fetch_object()){
                    $data[]=$wynik;
                    
                }
                return $data;
                
                /* $rows=$sqli->getRows("select * from users");
                foreach($rows as $row)
                {
                echo $row->log.' | '.$row->pass;
                }
                */
            }
            
            
            
            //Dodawania wartości do bazy danych
            public function insert($query)
            {
                $result=$this->mysqli->query($this->query=$query);
                if($result===true)
                {
                    echo "Dane zostały dodane do tabeli!";
                }
                else {
                    echo "Wystąpił błąd podczas dodawania danych do tabeli!";
                }
            }
            
            
            
            // Aktualizacja danych w bazie
            public function update($query)
            {
                $result=$this->mysqli->query($this->query=$query);
                if($result===true)
                {
                    echo "Dane zostały zaaktualizowane!";
                }
                else {
                    echo "Wystąpił błąd podczas aktualizowania danych!";
                }
            }
            
            
            
            
            // Kasowania z bazy danych
            public function delete($query)
            {
                $result=$this->mysqli->query($this->query=$query);
                if($result===true)
                {
                    echo "Dane zostały usunięte!";
                }
                else {
                    echo "Wystąpił błąd podczas usuwania danych!";
                }
            }
            
            
            // Koniec klasy
        }
        
        
        class User{
              private $db;
 
              public function __construct(sqli $db){
             $this->db = $db;
              }
 
 
              public function addUser($login,$haslo){
             $this->db->insert("insert into user values('$login','$haslo')");
              }
        }
?>
[PHP] pobierz, plaintext 

A tak go używam:

[PHP] pobierz, plaintext 
<?php
$db = new sqli('localhost','root','','spam');
 $user = new User($db);
 $user->addUser('adminek','adminek2');
?>
[PHP] pobierz, plaintext 

Oczywiście kod działa jak należy, tylko chodzi mi o to czy jest to poprawnie napisane. Krótko mówiąc rozumuję iż, w taki sposób jak zrobiłem powyżej, uzyskuje dostęp do klasy sqli oraz jej metod, dobrze myślę czy może się mylę? Wnioskuje to po tym że uzyskałem dostęp do metody update, która jest w klasie sqli.

Ten post edytował ExPlOiT 6.06.2009, 00:11:09