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

[Noidea]

1. Nic nie stoi na przeszkodzie, żeby UserManager zarządzał wieloma użytkownikami. Nie musisz tworzyć nowego obiektu dla każdego usera. Jeśli natomiast chcesz tworzyć UserManagera dla każdego użytkownika, to po co ci klasa User. Równie dobrze zamiast:

[PHP] pobierz, plaintext 
$userId = 1234;
 
$user = new User($userId); // tworze obiekt User o podanym id, może być nie istniejące
$userManager = new UserManager($user);
 
// Mógłbyś pisać:
$userManager = new UserManager( $userId );
[PHP] pobierz, plaintext 

2. W związku z pkt. 1 nie możesz już przekazywać obiektu User w konstruktorze UserManagera. Zamiast tego będziesz musiał poinformować metodę getData() o jakiego użytkownika ci chodzi.
3. Nie wiem czym się u ciebie różni user od userData, ale moim zdaniem jest to zbędne rozgraniczenie

Po oczyszczeniu kodu ze zbędnych instrukcji, zostanie ci pewnie coś w rodzaju:

[PHP] pobierz, plaintext 
$userManager = new UserManager($userId);
$userData = $userManager->getData();
 
$userManager->changePassword('abc123');
$userManager->save();
$userDataNew = $userManager->getData();
[PHP] pobierz, plaintext 

Jak widzisz jedynymi informacjami jakie przekazujesz tutaj w parametrach jest ID użytkownika i hasło. Jak się pewnie domyślasz, obok DI to to nawet nie stało (IMG:style_emoticons/default/smile.gif)

Dla porównania podaję zalążek kodu wykorzystania (a przynajmniej tego, jak ja to widzę) klasy UserManager, korzystającej z PDO oraz logującej błędy:

[PHP] pobierz, plaintext 
<<?php
 
interface ILogger
{
    function Log( $message );
}
 
class TxtLogger implements ILogger
{
    public function __construct( $filePath )
    { /*...*/}
 
    public function Log( $message )
    { /* Zapisuje wiadomość do pliku ... */ }
}
 
class NullLogger implements ILogger
{
    public function Log( $message )
    {
        return; // ignoruje wiadomość (nie zapisuje jej nigdzie)
    }
}
 
class User { /* kontener na dane użytkownika */ }
 
class UserManager
{
    private $db;
    private $logger;
 
    public function __construct( PDO $db, ILogger $logger )
    {
        $this->db = $db;
        $this->logger = $logger;
    }
 
    public function getDataById( $userId )
    {
        // Pobierz dane na temat użytkownika z bazy danych.
        // Jeśli znaleziono użytkownika o podanym ID, utwórz, wypełnij i zwróć obiekt klasy User
        // W przeciwnym wypadku zapisz informację o błędzie przy użyciu loggera i rzuć wyjątek
    }
}
 
$mysqlDb = new PDO( "blablabla" );
$userManager = new UserManager( $mysqlDb, new NullLogger() ); // Utworzenie managera użtykowników, korzystającego z bazy MySQL i nie logującego błędów
 
?>
[PHP] pobierz, plaintext