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

[Pr0100]

Odnośnie PDO to jeżeli jesteś początkującym programistą (jak wywnioskowałem z twoich postów) to moim zdaniem nie warto utrudniać sobie pracy.

Odnośnie klas to proponuje 2 klasy:
- klasa do obsługi tabeli z użytkownikami w MySqlu
- klasa do obsługi pojedynczego użytkownika podczas pracy z widokiem/kontrolerem (->getNick, ->getMail, ->getProfil (link do profilu), ->checkPass($string) itp)

Mówisz że pisząc 100 liniowy projekt nie opłaca sie używać FW? Idąc tym tropem to czy opłaca się używać klas?

Moim zdaniem pisząc 100 liniowy projekt szkoda czasu na zastanawianie się nad tym problemem (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Ten post edytował Pr0100 7.06.2009, 01:00:29