[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

[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

[by_ikar]

To jest bardziej userManager, czyli menadżer użytkowników, niż klasa użytkownika (pojedynczego) sama w sobie.

[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ą

[brzoza91]

[PHP] pobierz, plaintext 
 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;
   }]
[PHP] pobierz, plaintext 

A czy nie lepiej rozplanować to trochę inaczej ? żeby rozdzielić sprawdzanie poprawności danych od umieszczania ich do bazy danych ?

Ten post edytował brzoza91 25.05.2012, 20:43:33

[coolos]

A czy nie lepiej rozplanować to trochę inaczej ? żeby rozdzielić sprawdzanie poprawności danych od umieszczania ich do bazy danych ?

Stwórz menadżera który to będzie tworzyć model danych np.

Model może wyglądać tak jak poniżej, może być obsłużony np. za pomocą magicznych metod __set() oraz __get(). Zadaniem poniższego modelu jest przechowywanie danych i nic więcej

[PHP] pobierz, plaintext 
class USER_MODEL{
 
private $id;
private $name;
private $pass;
 
}
[PHP] pobierz, plaintext 

następnie stwóż menadżera modelu, który to będzie pobierać dane i umieszczać je w modelu i np. zwracać gdy kontroler o nie poprosi np.

[PHP] pobierz, plaintext 
class USER_MODEL{
 
public function get_user($id){
//tutaj kod odpowiedzialny za laczenie z baza danych wynikiem jest tablica $result
 
$user = new MODEL_USER();
$user->id = $result->id;
$user->name = $result->name;
$user->pass = $result->pass;
return $user;
}
 
}
[PHP] pobierz, plaintext 

Co zrobić gdy np. mamy do pobrania 50 użytkowników(obiektów)? stwórz ich kolekcje o nazwie np. USER_MODEL_COLLECTION.

tak ja to widzę.

[bastard13]

Osobiście mam wrażenie że zamknełem kod strukturalny w klasie i nic więcej

Przynajmniej wiesz, że coś tu jest nie tak, a to już połowa sukcesu:) Wiele osób uważa, że jak użyją magicznego słowa klas, to oznacza, że programują obiektowo. Dobry (a przynajmniej mam taką nadzieję) kurs OOP jest tutaj.

A co do Twojego przykładu to:
1) __construct() - pobiera dane nt. użytkownika i korzysta z sesji
Model to model. Użytkownik posiada dane na swój temat: id, name, email (na marginesie nie potrzebujesz prefixu user w atrybutach skoro klasa to User:P) i służy do obsługi logiki dotyczącej użytkownika, czyli metody takie jak: isAdmin(), isBlocked(), getEmailAdderess() są jak najbardziej na miejscu. UserLogin() i UserLogout() już nie. Metody modelu powinny dostarczać/ustawiać informacje na jego temat, wykonywać operacje na nim. Metody Logout() i Login() nie wykonują operacji na użytkowniku, a na aplikacji.

2) GetUserData() - to dego powinieneś stworzyć klasę np UsersManager z metodą getUser($id) i ona powinna Ci zwrócić gotowego użytkownika. Nie powinieneś mieszać logiki (użytkownik) ze sposobem przetrzymywania danych. Teraz jest to baza danych, ale pewnego dnia może dojdziesz do wniosku, że lepsze będą pliki XML?

3) UserLogin() i UserLogout() - o tym było przy okazji konstruktora:)

4) SetNewUser($user, $email, $pass, $pass2)
Po pierwsze walidacja powinna być wcześniej, chociaż to zależy od podejścia, które stosujesz:) Jednak na tym etapie radzę jednak stosować walidację przed tworzeniem instancji modeli.
To konstruktor powinien dostać parametry $user (a raczej $name?), $email i $pass. Ewentualnie, jeżeli chciałbyś użytkownika zapisać w bazie, to powinieneś w tym celu skorzystać z klasy UsersManager i metody save(User $user)

5) IsUserOnline()
Czyli, czy jest zalogowany. Od tego typu rzeczy powinieneś mieć mechanizm kontroli dostępu. I powinno to być sprawdzone wcześniej. Jeżeli dostęp do określonego miejsca może mieć tylko użytkownik zalogowany, to każdy inny powinien dostać odpowiedni komunikat. Z tym, że za to nie odpowiada użytkownik tylko aplikacja.