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]

Dodałem wzorzec projektowy singleton i coś mi nie działa, mama taki kod:

[PHP] pobierz, plaintext 
<?
 
 
        class Sqli
            {
                
                private static $instance = false;
                
                //Konstruktor
            public function __construct()
            {
                self::$instance=new mysqli('localhost','root','','katalog');
            }
            
            
            static function instance()
            {
            if( !self::$instance )
            {
            self::$instance = new self;
            }                              
        
            return self::$instance;
            }
            
            
            public function getRow($query)
            {
                return Sqli::instance()->query($query)->fetch_object();
                
            }
                    
            
            
            
            // Pobranie danych z bazy
            public function getRows($query)
            {
                $dane=array();
                $result=Sqli::instance()->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=Sqli::instance()->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=Sqli::instance()->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=Sqli::instance()->query($query);
                if($result===true)
                {
                    echo "Dane zostały usunięte!";
                }
                else {
                    echo "Wystąpił błąd podczas usuwania danych!";
                }
            }
            
            
            // Koniec klasy
        }
        
 
$sqli = Sqli::instance();
 
 
 
?>
[PHP] pobierz, plaintext 

Robię np:

[PHP] pobierz, plaintext 
<?php
require 'mysqli.class.php';
$sqli->delete("delete from users where log='admin'");
?>
[PHP] pobierz, plaintext 

I dostaje błąd:

[PHP] pobierz, plaintext 
<?php
[b]Fatal error[/b]:  Call to undefined method Sqli::query() in [b]C:wampwwwmysqlimysqli.class.php[/b] on line [b]91[/b]
?>
[PHP] pobierz, plaintext 

Co najdziwniejsze dla mnie w klasie user, działa wszystko bez problemu jakiegokolwiek, a w tej klasie do obsługi bazy danych nie działa (IMG:http://forum.php.pl/style_emoticons/default/sad.gif)