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

[Sajrox]

Wydaje mi się że poniższe rozwiązanie nie jest zbyt dobre, mianowicie:

[PHP] pobierz, plaintext 
$mysqlDb = new PDO( "blablabla" );
$userManager = new UserManager( $mysqlDb, new NullLogger());
[PHP] pobierz, plaintext 

Zawsze gdy chcemy utworzyć instancję klasy 'UserManager' musimy tworzyć obiekt PDO. Klasa 'UserManager' i tak jest przystosowana do pracy z PDO, więc można po prostu obiekt tworzyć już wewnatrz 'UserManager' albo wykorzystać dziedziczenie np. po klasie DbManager

Oto kolejna partia kodu kóra jest wygodna i chyba jako tako obiektowa. User jest tylko kontenerem, Manager zarządza tym kontenerem (gdy User ma id to znaczy że tyczy się nowego usera, gdy nie ma to jest to nowy kontener dla nowego usera)

[PHP] pobierz, plaintext 
// tworzymy obiekt User
$user = new User();
$user->id = 1;
$user->name = 'Bob';
 
// tworzymy obiekt managera ktory zarządza użytkownikiem
$userManager = new UserManager(
  $user, // domyslnie jest NULL
  new DbSqlManager(), // mozna ustawić domyslonie w klasie 
  new NullLogger() // mozna ustawić domyslonie w klasie 
);
$userData = $userManager->getData(); // zwracamy obiekt User wszystkie inne dane uzytkownika o id = 1
 
$userData->name = 'Steve';
 
$userManager->setData($userData); // zmiana danych
$userManager->save(); // zapisuje usera korzystająć z jego id
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
// Inny przykład
$user = new User();
//$user->id = 1; // bez id wieć teraz obiekt User jest nowym użytkownikiem
$user->name = 'Bob';
 
// manager nie może robic edycje i usuwanie gdyż obiekt User nie ma id więc jest nowym oniektem a nie istniejącym dla danego usera
$userManager = new UserManager(
  $user,
  new DbSqlManager(),
  new NullLogger()
); 
 
$user = $userManager->getUser(); // Zwraca błąd
$userManager->saveNew(); // Doda nowego o imieniu Bob
[PHP] pobierz, plaintext 

Ten post edytował Sajrox 7.09.2011, 10:18:19