RFC użytkownicy i grupy Opcje

[zajonc]

witam.

chciałbym się dowiedzieć, co sądzicie o takim zaprojektowaniu klas do obsługi użytkowników i grup.
będą używać 3 tabel z bazy danych:
- users: id, name, hash, email // ([nazwa]:[pola])
- membership: id, uid, gid
- groups: id, name

będę wdzięczny za jakiekolwiek sugestie

[PHP] pobierz, plaintext 
class user
{
    public function __construct ( $uid ) { ... }
    public function get_groups () { ... }
    public function get_name () { ... }
    public function set_name ( $name ) { ... }
    public function get_hash () { ... }
    public function set_hash ( $hash ) { ... }
    public function get_email () { ... }
    public function set_email ( $email ) { ... }
}
 
 
#####
 
 
class group
{
    public function __construct ( $gid ) { ... }
    public function get_users () { ... }
    public function count_users () { ... }
    public function user_add ( $uid ) { ... }
    public function user_rm ( $uid ) { ... }
    public function user_exists ( $uid ) { ... }  
    public function get_name () { ... }
    public function set_name ( $name ) { ... }
}
 
 
#####
 
 
class groups
{
    public function add ( $name ) { ... }
    public function rm ( $gid ) { ... }
    public function exists ( $value, $key = 'id' ) { ... } // $key = ( id, name )
    public function get_all () { ... }
    public function count () { ... }
}
 
 
#####
 
 
class users
{
    public function add ( $name ) { ... }
    public function rm ( $gid ) { ... }
    public function exists ( $value, $key = 'id' ) { ... } // $key = ( id, name )
    public function get_all () { ... }
    public function count () { ... }
    public function check ( $login, $pass ) { ... }
    final public function hash ( $hash ) { ... }
}
[PHP] pobierz, plaintext 

Ten post edytował zajonc 7.09.2011, 10:40:31

[bastard13]

Co do kwestii klasy user, to tworzenie jej obiektu powinno wyglądać tak:

[PHP] pobierz, plaintext 
$user = new User();
[PHP] pobierz, plaintext 

Bez id, ponieważ jeżeli podajesz id w konstruktorze oznacza to, że użytkownik nie jest tworzony tylko pobierany z bazy.

Pobieranie użytkownika z bazy powinno wyglądać tak:

[PHP] pobierz, plaintext 
$user = Users::find(3);
[PHP] pobierz, plaintext 

I w tym momencie, jeżeli istnieje użytkownik o id 3, to klasa Users go zwraca, a jeżeli nie istenieje, to zwraca false. Dlatego też nie jest ci potrzebna metoda exists()

Coś o TableAbstract:

[PHP] pobierz, plaintext 
abstract class TableAbstract
{
  public function find($id) {}
  public function delete($id) {}
  public function update($id, array $data) {}
  abstract protected function _getTableName();
}
 
class Users extends TableAbstract
{
   protected function _getTableName() {
      return 'Users';
   }
}
 
 
class Groups extends TableAbstract
{
   protected function _getTableName() {
      return 'Groups';
   }
}
[PHP] pobierz, plaintext 

Mniej więcej o to chodzi. Jak dobrze przemyślisz sprawę to metody update, delete i find różnią się tak naprawdę jedynie nazwą tabeli.