klasa user - dobrze? Opcje

[em_pl]

Hej
Staram przestawić się na myślenie OOP. Stworzyłem klasę, która odpowiedzialna będzie za dane zalogowanego użytkownika.

[PHP] pobierz, plaintext 
<?php
class user extends DB{
	protected $db;
	private $fetch_mode = PDO::FETCH_ASSOC;
	private $prefix, $tableName, $permissions, $logged = 0, $id, $login, $name, $function;
	public function __construct($prefix) {
		$this->db			= parent::getInstance();
		$this->prefix		= $prefix;
		$this->tableName	= $prefix.'users';
		if(!empty($_COOKIE['sid_'.$this->prefix])){
			$sid = &$_COOKIE['sid_'.$this->prefix];
			$sql = $this->db->prepare('SELECT * FROM '.$this->tableName.' WHERE sid=:sid');
			$sql->bindValue(':sid', $sid, PDO::PARAM_STR);
			$sql->execute();
			$row = $sql->fetch($this->fetch_mode);
			$sql->closeCursor();
			if($row['id']){
				$this->id		= $row['id'];
				$this->login	= $row['login'];
				$this->name		= $row['name'];
				$this->function	= $row['function'];
				$this->permissions	= new userPermissions($this->prefix, $this->id);
				$this->logged	 = 1;
			}
		}
	}
 
	public function login($login, $password){...}
	public function logout(){...}
	public function getName(){
		return $this->name;
	}
	public function getFunction(){
		return $this->function;
	}
	public function getLogged(){
		return $this->logged;
	}
 
}
?>
[PHP] pobierz, plaintext 

Obiekt tej klasy będzie zawierał informacje o użytkowniku (jeśli jest on zalogowany). Dodatkowo zmienna $permission będzie obiektem klasy userPermission:

[PHP] pobierz, plaintext 
<?php
class userPermissions extends DB{
	protected $db;
	private $userId, $users, $portalContent, $categories, $tableName;
	private $fetch_mode = PDO::FETCH_ASSOC;
	public function __construct($prefix, $userId) {
		$this->db			= parent::getInstance();
		$this->tableName	= $prefix.'user_functions';
		$sql = $this->db->prepare('SELECT * FROM '.$this->tableName.' WHERE user_id=:id');
		$sql->bindValue(':id', $userId, PDO::PARAM_INT);
		$sql->execute();
		$row = $sql->fetch($this->fetch_mode);
		$this->editUsers		 = $row['edit_users'];
		$this->editCategories	 = $row['edit_categories'];
		$this->editPortalContent= $row['edit_portal_content'];
	}
	public function getUsers(){
		return $this->editUsers;
	}
	public function getPortalContent(){
		return $this->editPortalContent;
	}
	public function getCategories(){
		return $this->editCategories;
	}
}
?>
[PHP] pobierz, plaintext 

Jeżeli chcę sprawdzić czy user jest zalogowany $user->getLogged();
jeżeli chce sprawdzić czy ma prawa do edycji kategorii $user->permissions->getCategories();

I teraz mam 2 pytania.
1. Czy taka logika, jaką stworzyłem jest w ogóle racjonalna, czyli czy dobrze myślę jeżeli nie, to gdzie robię błędy?
2. PDO ma opcję zwrócenia wyniku jako obiekt. Czy da się automatycznie zapisać atrybuty zwróconego zapytania (czyli pola tabeli) do atrybutów danej klasy?
W chwili obecnej robię to tak:

[PHP] pobierz, plaintext 
<?php
$this->editUsers		 = $row['edit_users'];
		$this->editCategories	 = $row['edit_categories'];
		$this->editPortalContent= $row['edit_portal_content'];
?>
[PHP] pobierz, plaintext 

Dziękuję serdecznie za wszelką pomoc, a także za opinie na temat samej klasy

[sf]

Nie widzę potrzeby, aby klasa użytkownik była zależna od bazy danych, a tym bardziej od konkretnego zapytania SQL. Do klasy Użytkownik powinnień być przekazywany obiekt lub tablica z danymi, a nie, że sama sobie to pobiera.

[em_pl]

Hej
Nie wiem czy dobrze rozumiem... Chodzi o to, aby samym pobieraniem z bazy zajęła się inna klasa (odpowiedzialna za DB tak?) Mógłbyś podać jakiś prosty przykład przesłania danych do obiektu, gdyż w chwili obecnej nie bardzo rozumiem, jak miało by to wyglądać.

Dziękuję z góry

[sf]

wycinek klasy odpowiedzialnej za logowanie

[PHP] pobierz, plaintext 
<?php
		// pobieramy klasę, która przechowuje dane użytkownika
		$oCurrentUser = $this->oContext->getCurrentUser();
		// połączenie z bazą danych
		$oDB = HPDO::getInstance($this->sDSN);
 
		// pobiera dane o użytkowniku poszukaj o ActiveRecord, DAO
		$oU = new DAOUsers($oDB);
		$oU->setWhere('login', $oRequest->login);
		$oU->setWhere('password', md5($oRequest->password));
		$aUser = $oU->selectOne();
 
		// błędny login / hasło, użytkownik nie istnieje, użytkownik nie aktywny
		if(!$aUser || $aUser['status'] != 1) {
			$oResponse->failed = array('login' => gettext('Bad login or password.'));
			return 'failure';
		}
 
		// przypisz dane do użytkownika
		$oCurrentUser->userId = $aUser['userId'];
		$oCurrentUser->login = $aUser['login'];
		$oCurrentUser->name = $aUser['firstName'] . ' ' . $aUser['lastName'];
 
		// dalej pobiera prawa jakie posiada użytkownik w podobny sposób jak użytkownika
		// i zapisuje pod...
		$oCurrentUser->permissions = $aPermissions;
?>
[PHP] pobierz, plaintext 

Wszystko trzymamy w sesji, ewentualnie cookie. Potem w innych miejsach dostęp do danych przez:
$oCurrentUser = $this->oContext->getCurrentUser();

Ten post edytował sf 4.05.2007, 16:39:03

[kicaj]

Zainteresuj sie wzorcem Registry:)