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 

1) W klasach User i Group masz settery i gettery do każdej wartości, więc jaki jest ich sens? Przypuszczam, że i tak będziesz wysyłał zawsze pełny zestaw danych więc zastanów się nad zastąpieniem tych metod: setData(array $data).
2) W konstruktorze nie powinno być parametru id, bo skąd wiesz jaki id będzie miał rekord? Przypuszczam, że kolumna jest auto inkrementowana. Id powinien być ustawiany dopiero po zapisie User/Group w bazie.
3) Groups i Users powinny rozszerzać klase abstrakcyjną np. TableAbstract, ponieważ w innym wypadku będziesz miał sporo kodu, który będzie duplikował się.
4) Metody add() i remove() powinny przyjmować jako parametr obiekt klasy User/Group, ponieważ to właśnie je chcesz zapisać/usunąć w/z bazy.
5) Czy rzeczywiście jest ci potrzebna metoda exist() ? Zastąpiłbym ją raczej find(), która zwróci albo obiekt (czyli rekord jest) albo false (brak rekordu).
6) Metody check() i hash() z klasy Users powinny wylecieć. check() jest metodą autoryzacji, więc powinna się znaleźć w tego typu klasie, natomiast hash() przypuszczam, że wykonuje jakieś operacje na stringu, więc na to konto również powinna być odpowiednia klasa.

ad. 2. chciałem korzystać np. z user mniej więcej tak:

[PHP] pobierz, plaintext 
$user = new user (3);
echo $user->get_name();
[PHP] pobierz, plaintext 

istnieje usera sprawdziałbym users::exists()
taki miałem plan.

mógłbyś coś wiedzieć o TableAbstract ? prototypy mile widziane

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.

w klasie user nie operuje na danych w bazie, tylko seterami/geterami pracuje na zmiennych, tak?
dopiero users::update () dłubie w bazie.

[PHP] pobierz, plaintext 
class user
{
	private $id;
	private $name;
	private $pass;
 
	function __construct ( $uid, $name, $pass )
	{
		$this->id = $uid;
		# name, pass
	}
 
	function get_name () { /* .. */ }
	function get_id () { /* .. */ }	
	function get_pass () { /* .. */ }
	function set_pass ( $pass ) { /* .. */ }
}
[PHP] pobierz, plaintext 

to już chyba lepiej wygląda. jeszcze chciałbym zaimplementować grupy i pola dodatkowe (np. imię, nazwisko). to też w klasie user?
jeśli tak, go kiedy powinienem pobierać dane z bazy?

A co do metody update w TableAbstract, to trochę się rozpędziłem:)
To klasa User i Group powinna posiadać metodę save(), która przy nowych obiektach dodaje je do bazy, a przy tych wyciągniętych z bazy - zapisuje zmiany.

users::find($uid) zwraca użytkownika ze wszystkimi wypełnionymi polami?
tj. id, name, pass, groups, custom_fields.

czy też już user::get_groups dba o aktualne dane?

[PHP] pobierz, plaintext 
<?php
 
 
 
 
 
	class user
	{
 
		private $id;
		private $name;
		private $pass;
		private $customs = array ();
		private $groups = array ();
 
 
 
		function __construct ( $uid, $name, $pass, array $groups, array $customs )
		{
			$this->id = (int) $uid;
			# name, pass, customs, groups
		}
 
 
 
		function get_id ()
		{
			return $this->id;
		}
 
 
 
		function get_name ()
		{
			return $this->name;
		}
 
 
 
		function get_pass ()
		{
			return $this->pass;
		}
 
 
 
		function set_pass ( $pass )
		{
			$this->pass = $pass;
		}
 
 
 
	# grupy
 
		function get_groups ()
		{
			return $this->groups;
		}
 
 
 
		function add_group ( $gid )
		{
			$this->groups[] = $gid;
		}
 
 
 
		function rm_group ( $gid )
		{
			unset ( $this->groups[ array_search ( $gid, $this->groups ) ] );
		}
 
 
 
		function is_in_group ( $gid )
		{
			return in_array ( $gid, $this->groups );
		}
 
 
 
	# pola dodatkowe
 
		function get_customs ()
		{
			return $this->customs;
		}
 
 
 
		function add_custom ( $id, $value )
		{
			$this->customs[ $id ][] = $value;
		}
 
 
 
		function rm_custom ( $id, $value )
		{
			unset ( $this->customs[ $id ][ array_search ( $value, $this->customs[ $id ] ) ] );
		}
 
 
 
	}
 
 
 
 
 
 
 
	class users
	{
 
 
 
 
 
		function __construct ()
		{
			$this->db = kernel::get ( 'db' );
		}
 
 
 
 
 
		function update ( user $user )
		{
			# nanoszenie zmian do tabel: users, memberships, users_customs
		}
 
 
 
 
 
		function save ( user $user )
		{
			$uid = $this
				-> db
				-> prepare
				(
					"INSERT INTO `users`
					(
						`id`,
						`name`,
						`pass`
					)
					VALUES
					(
						NULL ,
						':name',
						':hash'
					)" )
				-> bind ( 'name', $user->name )
				-> bind ( 'hash', auth::hash ( $user->pass ) )
				-> exec ()
				-> insert_id ();
 
			# zapis customs i groups do bazy
		}
 
 
 
		function find ( $uid )
		{
			$r = # pobieranie z bazy ( $name, $pass )
 
			if ( empty ( $r[0] ) )
			{
				return false;
			}
 
			#pobieranie $customs i $groups z bazy
 
			extract ( $r[0] );
			return new user ( $uid, $name, $pass, $groups, $customs );
		}
 
 
 
 
 
	}
 
 
 
 
 
 
 
[PHP] pobierz, plaintext