prawidlowe OOP , struktura klasy, Jak wedlug koncepcji OOP napisac dobra klase Opcje

[rahul]

Witam
Temat pewnie wam znany i stary jak swiat. Pisze sobie swoj cms starajac sie zrobic to obiektowo. Dotychczas przejrzalem i przeczytalem wiele artykulow dotyczacych OOP i robie sie coraz glupszy bo z tego co rozumie to zastosowan jest multum, tylko ktore to najlepsze. mam taka klase

[PHP] pobierz, plaintext 
include_once('db/db.php');
class User
 { 
	private $user_id;
	private $first_name; 
	private $last_name; 
	private $login;
	private $password;  
	private $email; 
	private $ip; 
	private $user_role; 
	private $registration_date; 
	private $db;
 
	function __construct() 
	{
		$this->db = Database::getInstance(); 
	}
 
	function setLogin($login) 
	{ 
		$this->login = $login;
	}
 
	function setPassword($password) 
	{
		$this->password = $password; 
	}
 
	function setFirstName($firstName) 
	{
		$this->firstName = $firstName; 
	}
 
	function setLastName($lastName) 
	{
		$this->lastName = $lastName;
	}
 
	function setEmail($email)
	{
		$this->email = $email;
	}
 
	function setUserRole($role)
	{
		$this->role = $role;
	}
 
	function setRegistrationDate($registration_date) 
	{
		$this->registration_date = $registration_date;
	}
 
 
	function login() 
	{
		$action = $this->db->prepare("SELECT * from users WHERE login LIKE '$this->login' 
									  AND password LIKE '$this->password'"); 
		$action->execute(); 
		$result = $action->fetchAll();
		if(count($result)== 1)
		{
			if($result[0]['Role'] == 0)
			{
				$_SESSION['user_id'] = $result[0]['user_id']; 
				return true;
			}else if($result[0]['Role'] == 1)
			{
				$_SESSION['user_id'] = $result[0]['user_id'];
				$_SESSION['admin'] = 'admin';
				header('location:admin.php');
			}
		}
		return false; 
	}
 
	function register() 
	{ 
		$ip = $_SERVER['REMOTE_ADDR'];
		$action = $this->db->query( "INSERT into users(user_id , Login , Password ,  Name , LastName ,Email , Ip , Role , RegistrationDate)
							    VALUES('' , '$this->login' ,'$this->password' , '$this->firstName' , '$this->lastName' , '$this->email' , '$ip' ,
                                                             '$this->role' , NOW())"); 
		$action->execute();
		return true;
	}
 
	function EditUser() 
	{
		...
	}
 
	function ChangePassword()
	{
		...
	}
} // end of class. 
?>
[PHP] pobierz, plaintext 

Pierwsze pytanie : Czy klasa User powinna miec funkcje takie jak Loguj, Rejestruj, Edytuj, Zmien Haslo i czy te funckje powinny miec juz "hardcoded" zapytania do bazy wewnatrz, patrz funckja loguj();. Wywoluje ja tak:

[PHP] pobierz, plaintext 
 
$user_c = new User(); 
	$user_c->setLogin($_POST['login']);
	$user_c->setPassword(md5($_POST['password']));
	$login = $user_c->login(); 
[PHP] pobierz, plaintext 

Jezeli cos tego pokroju jest ok to spoko. Teraz np dopisalem sobie taka funkcje to tej samej klasy, ktora jak dla mnie moglaby byc w kazdej prawie innej klasie :

[PHP] pobierz, plaintext 
class User { 
 
private $sqlObj; 
private $SqltableRows;
 
function query($query) 
	{
		$sql = $this->db->query($query); 
		$sql->execute(); 
		$this->sqlObj= $sql->fetchAll(PDO::FETCH_OBJ);
		$this->SqltableRows= $sql->fetch(PDO::FETCH_ASSOC);
	}
}
[PHP] pobierz, plaintext 

Dzieki tej funkcji lapie sobie wszystko z bazy i w prosty sposob moge wywolywac wszystkie kolumny :

[PHP] pobierz, plaintext 
$user_c = new User(); 
$user_c->query("SELECT * FROM users");
$users = $user_c->getObj(); 
foreach ($users as $user)
{
echo $user->Name;
echo $user->Password;
}
[PHP] pobierz, plaintext 

Bardzo podoba mi sie mozliwosc poboru rekordow i nazw wierszy tabeli w tak prosty sposob. Teraz do rzeczy :
Funckja ta jest w Users ale generalnie moglaby byc w prawie kazdej innej klasie, np Products, Articles itp. Czy mam utworzyc osobna klase z ta funckja z ktorej jakos beda kozystac wszyskie inne klasy ? Czy ma byc to w klasie bazy danych, czy moze w jakies jeszcze innej ?

Prosze o odpowiedzi i wyrozumialosc (IMG:style_emoticons/default/smile.gif)


Ten post edytował rahul 20.08.2011, 20:50:56

[rahul]

Obiekty różnych typów zajmują się różnymi rzeczami. Zawsze powinieneś starać się doprowadzić do tego by jeden obiekt zajmował się tylko jedną rzeczą. Reprezentowanie danych z bazy danych (obiekt User) i zarządzanie takimi "reprezentantami" (obiekt UserManager) to dwie zupełnie inne sprawy. Oba te obiekty wykonują zadania kompletnie nie związane ze sobą. Dlatego też muszą to być osobne obiekty.

Ok,staram sie to poukladac w glowie. Powiedz mi wtedy czy dobrze to rozumie :
1)klasa User bedzie zawierac funkcje do wyswielania danych uzytkownika takie jak np: pokaz imie, pokaz nazwisko, poka mail. I generalnie tylko takie.
2)Z tego co tak sobie mysle to wszystkie podstawowe informacje o uzytkowniku mozna przeciez wywolas z jednej funckji ("select * form user WHERE id = $id") zwroci mi tablice badz obiekt i sobie bede juz nim operowal zaleznie od tego co chce pokazac, tak ?(IMG:style_emoticons/default/smile.gif) Czy niby mam do imienia, nazwiska, maila pisac po osobnej funckji ? Chyba nieee... , co ? W momencie jak zapytanie zwroci mi obiekt mam mam z niego kozystac tak :

[PHP] pobierz, plaintext 
 
$user_o = $User->getUser($id); 
echo $user_o->Name; 
[PHP] pobierz, plaintext 

badz tak :

[PHP] pobierz, plaintext 
 
$user_o = $User->getUser($id); 
echo $user_o['Name']  
[PHP] pobierz, plaintext 

czy jakos inaczej ?
bo chyba nie tak :

[PHP] pobierz, plaintext 
 
$user_o = $User->getUser($id); 
echo $user->getName(); 
[PHP] pobierz, plaintext 

Strasznie duzo seterow geterow bym wtedy musial pisac. Czy moze tak trzeba? Wydaje mi sie ze funckja UserManager powinna je raczj miec.

3)Teraz druga klasa User Manager ona bedzie miala funckje typu Loguj, Edytuj, Rejestruj i bede z niej kozystal tak jak napisal Noidea w jednej z odpowiedzi, tak?

A co do seterow i geterow czy tylko klasa UserManager bedzie je posiadala (setPass, setName) takowe w klasie User nie sa mi chyba potrzebne, prawda ?
no i niektore funckje beda sie chyba powtarzac jak np getUserById($id).

Ten post edytował rahul 22.08.2011, 16:08:53