Zaprojektowanie klasy użytkownika - jak to wygląda od węwnątrz? Opcje

[-kubek3898-]

Witam, zaczynam dopiero zabawę z programowaniem OOP w PHP i zastanawia mnie jak ma wyglądać klasa użytkownika (logowanie, rejestracja itp.). To co napisałem do tej pory wydaje mi się zagmatwane i napisane mało profesjonalnie, a chciałbym by klasa była "ładnie" napisana. Największy problem mam z wyobrażeniem sobie w jaki sposób ma działać np. logowanie. Czy przesłane argumenty do tej metody mam porównywać wysyłając zapytanie do bazy danych, czy może wystarczą dane składowe tej klasy, które przypisuję w momencie rejestracji. Również zastanawia mnie jak i gdzie ma działać sesja? Wydaje mi się, że klasa równie dobrze działała by bez tych danych składowych takich jak login, czy adres E - Mail, jednak wtedy to straciło by sens. Może mnie ktoś naprowadzić jak taki kod powinien wyglądać?

To co dotychczas nabazgrałem:

[PHP] pobierz, plaintext 
<?php
 
require_once 'config.php';
require_once 'class.mysql.php';
 
class User
{
	private $id;
	private $username;
	private $password;
	private $email;
 
	private $db;
 
	public function __construct($db)
	{
		$this->db = $db;
 
		ob_start();
		session_start();
	}
 
	public function __destruct()
	{
		$this->db = null;
 
		ob_end_flush();
		session_destroy();
	}
 
	public function register($username, $password, $email)
	{
		if(!filter_var($email, FILTER_VALIDATE_EMAIL))
		{
			throw new Exception('Email is not valid!', 1);
		}
		else if(strlen($password) < 4 || strlen($password) > 20)
		{
			throw new Exception('Password is not valid', 2);
		}
		else if(strlen($username) < 4 || strlen($username) > 20)
		{
			throw new Exception('Username is not valid!', 3);
		}
 
		// Checking for exists user
		$checkExistsUser = 'SELECT `id` FROM `' . USERS_TABLE_NAME . '` WHERE `username`=:username;';
		$sth = $this->db->prepare($checkExistsUser);
		$sth->bindValue(':username', $username, PDO::PARAM_STR);
		$sth->execute();
 
		if($sth->rowCount() > 0)
		{
			throw new Exception('Someone have got account with this username!', 5);
		}
		else
		{
			$password = hash('sha256', $password);
 
			// Insert new user to database
			$insertNewUser = 'INSERT INTO `' . USERS_TABLE_NAME . '` 
			(`id`, `username`, `password`, `email`) VALUES
			(NULL, :username, :password, :email);';
			$sth = $this->db->prepare($insertNewUser);
			$sth->bindValue(':username', $username, PDO::PARAM_STR);
			$sth->bindValue(':password', $password, PDO::PARAM_STR);
			$sth->bindValue(':email', $email, PDO::PARAM_STR);
			$checkSuccess = $sth->execute();
 
			if($checkSuccess > 0)
			{
				throw new Exception('Register successfully!', 6);
				$this->id = $_SESSION['id'] = $this->db->lastInsertId();
				$this->username = $_SESSION['username'] =  $username;
				$this->password = $password;
				$this->email = $_SESSION['email'] = $email;
			}
			else
			{
				throw new Exception('Register failed!', 7);
			}
		}
	}
 
       // Najbardziej interesuje mnie jak wyglądać ma ta funkcja...
	public function login($username, $password)
	{
		if(strlen($password) < 4 || strlen($password) > 20)
		{
			throw new Exception('Password is not valid', 2);
		}
		else if(strlen($username) < 4 || strlen($username) > 20)
		{
			throw new Exception('Username is not valid!', 3);
		}
 
		if($this->username == $username && $this->password == $password)
		{
			throw new Exception('Login successfully!', 8);
		}
		else
		{
			throw new Exception('Login failed!', 9);
		}
	}
}
 
?>
[PHP] pobierz, plaintext 

I jeszcze jedno pytanie. Czy rzucanie wyjątków mogę wykorzystwać również do sytuacji wcale nie wyjątkowych? Takie coś działa, lecz chciałem się dowiedzieć czy nie jest to mało profesjonalne.

[amii]

Tak szczerze mówiąc to ten kod z regułami wzorcowego OOP ma nie wiele wspólnego. To na co w powinnieś zwrócić uwagę:
1. Każda klasa wykonuje ściśle określone zadanie
2. MVC czyli trójwarstowy podział na model, widok i kontroler
3. Zainterewsuj się czymś takim jak service container i dependency injection - ogólnie to fajnie w symfony2 jest zrobione
http://symfony.com/doc/current/book/service_container.html
http://www.youtube.com/watch?v=DcNtg4_i-2w

To tak tyle na początek (IMG:style_emoticons/default/smile.gif)