[klasa] User.class.php, Klasa uzytkownicy bardzo prosta Opcje

[fannet]

Napisałem takie "coś" i mam nadzieję że jest to klasopodobne. Proszę o opinie, dobrze myślę czy też nie. Osobiście mam wrażenie że zamknełem kod strukturalny w klasie i nic więcej. Aczkolwiek szukając w internecie ciężko jest trafić na coś zrozumiałego. W większości spotkykam się z informacją co to jest klasa i jak ją stworzyć, dziedziczenie czy parę słów o hermetyzacji.. ;(

Proszę Was również o możliwe przykłady klas, nie zbyt skomplikowanych lecz ukazujących myślenie obiektowe, jeżeli takowa znacie (IMG:style_emoticons/default/wink.gif)

[PHP] pobierz, plaintext 
<?php
 
class User
{
 
   private $userid;
   private $userName;
   private $userEmail;
 
   // Konstruktor pobiera dane uzytkownika, jezeli użytkonik nie jest zalogowany zglasza blad
   public function __construct()
   {
      if(!static::IsUserOnline())
      {
         throw new Exception("BRAK DOSTEPU DO TEJ STRONY");
      }
 
      $this->GetUserData($_SESSION['user']);
   }
 
   // Pobiera dane użytkowników
   public function GetUserData($id)
   {
      if(isset($id) && is_numeric($id))
      {
         $result = DatabaseManager::selectBySQL("SELECT userName FROM users WHERE userName = $id LIMIT 1");
         foreach ($result as $r)
         {
            $this->userName = $r['userName'];
            $this->userEmail = $r['userEmail'];
         }
      }
   }
 
   // Loguje użytkonwika do serwisu
   public function UserLogin($name, $pass)
   {
      $name = mysql_escape_string($name);
      $pass = mysql_escape_string($pass);
      $result = DatabaseManager::selectBySQL("SELECT userid FROM users WHERE userName = '$name' && userPass = md5('$pass') LIMIT 1");
 
      if($result)
         return $_SESSION['user'] = $result[0]['userid'];
      else
         return false;
   }
 
   // Wylogowanie użytkonika z bazy danych
   public function UserLogout()
   {
      $_SESSION['user'] = false;
      session_destroy();
   }
 
   // Tworzy nowego użtykonika
   static function SetNewUser($user, $email, $pass, $pass2)
   {
      // tworzymy polaczenie z baza danych
      $db = DatabaseManager::getConnection();
 
      // Sprawdzamy czy użytkownik istnieje
      $checkUsers = $db->query("SELECT nick FROM users WHERE nick = $user");
      if(count($checkUsers) >= 1)
         return $msg = "Podana nazwa uzytkownika juz istnieje";
 
      // Sprawdzamy czy podane hasła są jednakowe
      if($pass == $pass2)
         return $msg = "Podane hasła nie są jednakowe";
 
      // Sprawdzamy poprawnosc adresu email
      if(!filter_var($email, FILTER_VALIDATE_EMAIL))
         return $msg = "Adres email jest nie poprawny";
 
      // Wysyłamy użytkownika do bazy danych
      $query = $db->query("INSERT INTO users (`id`,`nick`,`email`,`pass`) VALUES (NULL, '$user', '$email', 'md5($pass)');");
      return $msg = false;
   }
 
   // Funkcja statyczna sprawdzajaca czy uzytkonik jest zalogowany
   static function IsUserOnline()
   {
      if(isset($_SESSION['user']) && $_SESSION['user'] && is_numeric($_SESSION['user']))
         return true;
      else
         return false;
   }
}
 
?>
[PHP] pobierz, plaintext 

Pozdrawiam, Luke (IMG:style_emoticons/default/wink.gif)

[Fifi209]

Konstruktor nie "zgłasza błąd" a rzuca nic nie znaczący wyjątek, nawet nie wiadomo z jakiej klasy pochodzi ani co jest przyczyną jego wystąpienia. W dodatku tutaj nie powinien być rzucany wyjątek.

Wszędzie gdzie masz zapytania SQL używasz "DatabaseManager" - co też ma złą nazwę, bo nijak nie służy do zarządzania bazami jak na moje oko, ale chciałem poruszyć inny temat, mianowicie filtrowania, powinno to się odbywać w w/w klasie a nie w "User".

[PHP] pobierz, plaintext 
 if($result)
         return $_SESSION['user'] = $result[0]['userid'];
      else
         return false;
   }
[PHP] pobierz, plaintext 

Powinno zwracać true/false, nic więcej użytkownika nie interesuje, jeżeli chcesz wyciągnąć ID zalogowanego to napisz metodę do tego.

SetNewUser -> CreateUser, Register etc.
Też powinna zwracać true/false i zapisać gdzieś błędy, które mógłbyś odczytać poprzez kolejną metodę.

Resztę napisał kolega przede mną (IMG:style_emoticons/default/wink.gif)