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

[Crozin]

No to wlasnie albo korzystasz z FW w 100% i z gotowych klas albo wszystko piszesz sam robic z kodu takie spaghetti gdzie mieszasz swoj kod z cudzym to raczej wyjdzie hujow* i tobie tez nie bedzie smakowalo taka moja skromna uwaga

A co jest złego w korzystaniu z cudzych bibliotek? Jeżeli jest to w ramach treningu to może, ale jeżeli nie to po co ma tracić czas?

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.

Hmmm... a co w PDO tak dokładnie miałoby mu utrudnić pracę?


Możesz spróbować zaprojektować sobie swój model nieco inaczej. Zrób wszystko od drugiej strony - nie "jakich klas potrzebuję" tylko "jakiej funkcjonalności potrzebuję".
Przykładowo masz prostą strukturę danych, gdzie każdy użytkownik jest przypisany do jakiejś grupy (relacja 1-n)

[PHP] pobierz, plaintext 
<?php
 
//zwraca nam jeden obiekt, klasy Group
$group = GroupPeer::retrieveByName('user');
 
//tworzymy sobie nowy obiekt, reprezentujący użytkownika
$user = new User();
$user->setUsername('blah blah')
     ->setFirstName('blag blah')
     ->setLastName('blah blag')
     //tak wiem, że można by przekazać poprostu $group :)
     ->setGroupId($group->getId());
 
$user->save();
?>
[PHP] pobierz, plaintext 

To jest akurat pokazane na przykładzie Propela. Do tego dochodzi jeszcze walidacja danych - ale to już "przy formularzach".