Forum PHP.pl

Tak sobie od kilku dni czytam ksiazeczke na temat programowania obiektowego i chce zrozumiec wzorzec MVC. Sklecilem taki kod i chce zebyscie go zweryfikowali czy moje rozumowanie idzie w dobra strone.
Bez obslugi bledow, bez pojedynczych instancji, prosty kod zeby sie przekonac czy dobrze rozumuje itd. Narazie sie bawie...
Zerknijcie okiem i napiszcie cos od siebie. Dzieki z gory za wszelkie uwagi!

[PHP] pobierz, plaintext 
<?php
 
/**
* Interfejs kontrolera - kazdy niech ma metode ustawiania widoku
*/
interface Controller {
	public function setView($viewName);
}
 
/**
* Interfejs widoku - kazdy niech ma metode wyswietlania
*/
interface View {
	public function display();
}
 
/**
* Klasa widoku produktu z zaimplementowanym interfejsem widoku
*/
class ProductView implements View {
 
	private $vars = array();
 
	public function setVar($varName, $varValue = false){
		$this->vars[$varName] = $varValue;
	}
 
	public function display(){
		foreach($this->vars AS $varName => $varValue){
			print "[{$varName}] => {$varValue}\n";
		}
	}
 
}
 
 
/**
* Klasa pojedynczego produktu
*/
class Product {
 
	protected $title;
	protected $price;
 
	public function setTitle($title){
		$this->title = $title;
	}
 
	public function setPrice($price){
		$this->price = $price;
	}
 
	public function getTitle(){
		return $this->title;
	}
 
	public function getPrice(){
		return $this->price;
	}
 
}
 
/**
* Klasa kontrolera produktow korzysta z klasy Product i implementuje interfejs k
ontrolera
* Wywolanie akcji addProduct powoduje ustawienie title, price, widoku ProductVie
w i wyswietlenie danych
*/
class ProductController extends Product implements Controller {
 
	private $viewObj;
 
	public function addProduct($title, $price){
		$this->setTitle($title);
		$this->setPrice($price);
		$this->setView('ProductView');
		$this->viewObj->setVar($this->getTitle(), $this->getPrice());
		$this->execView();
	}
 
	public function setView($viewName){
		$this->viewObj = new $viewName;
	}
 
	public function execView(){
		$this->viewObj->display();
	}
 
}
 
/**
* Tu powinien byc jakis front controller ktory wywola akurat kontroler produktow
 ale do tego jeszcze nie doszedlem
*/
$o = new ProductController;
$o->addProduct('Nazwa 1', '44.99');
 
?>
[PHP] pobierz, plaintext 

Ten post edytował Helios 13.08.2007, 17:54:47

Front controller powinien tylko wywolac akcje kontrolera, a nie decydowac o tym czy kontroler doda rekord czy nie, dopiero w akcji okreslasz co masz zrobic dalej. Kontroler nie powinien byc sprzezony z modelem ani z widokiem, powinienes w kontrolerze stworzyc sobie nowy obiekt Product i na nim operowac, a nie dziedziczyc z tej klasy. W skrocie ja to widzialbym tak:

[PHP] pobierz, plaintext 
<?php
 
/**
* Interfejs widoku - kazdy niech ma metode wyswietlania
*/
interface View {
	public function display();
}
 
/**
* Klasa widoku produktu z zaimplementowanym interfejsem widoku
*/
class ProductView implements View {
 
	private $vars = array();
 
	public function setVar($varName, $varValue = false){
		$this->vars[$varName] = $varValue;
	}
 
	public function display(){
		foreach($this->vars AS $varName => $varValue){
			print "[{$varName}] => {$varValue}\n";
		}
	}
 
}
 
/**
* Klasa pojedynczego produktu
*/
class Product {
 
	protected $title;
	protected $price;
 
	public function setTitle($title){
		$this->title = $title;
	}
 
	public function setPrice($price){
		$this->price = $price;
	}
 
	public function getTitle(){
		return $this->title;
	}
 
	public function getPrice(){
		return $this->price;
	}
 
}
 
/**
 * Model produktu
 */
class ProductModel
{
	public function add (Product $product){
		// Tutaj rozebranie obiektu Product i zapisanie go do bazy
	}
}
 
class ProductController {
 
	private $request;
	private $response;
 
	private $view;
	private $model;
 
	public function __construct ($request, $response){
		$this->request = $request;
		$this->response = $response;
 
		$this->view = new ProductView();
		$this->model = new ProductModel();
	}
 
	public function addproductAction(){
		$product = new Product();
 
		/**
		 * Zakladajac ze mamy obiekt request ktorzy przechowuje wszystkie dane z GET i P
OST
		 */
		$product->setTitle($this->request->getParam('title'));
		$product->setPrice($this->request->getParam('price'));
 
		$this->model->add ($product);
 
		$this->view->setVar($this->request->getParam('title'), $this->request->getParam('price'));
		$this->view->display();
	}
}
 
$title, $price 
 
/**
* Tu powinien byc jakis front controller ktory wywola akurat kontroler produktow
 ale do tego jeszcze nie doszedlem
* Front controller nie powinien wiedziec jakie parametry bedzie posiadal nowy pr
odukt, nie powinien rowniez wiedziec nic o 
* jakichkolwiek produktach dane do produktu mozesz zebrac z GET albo POST
*/
$o = new ProductController ($request, $response);
$o->addproductAction ();
 
?>
[PHP] pobierz, plaintext 

Ten post edytował NoiseMc 10.08.2007, 14:48:45

Chce zrobic sobie jakos ladnie lancuchy akcji i wymyslilem takie oto rozwiazanie.

Stworze klase abstrakcyjna AppQuery, ktora bedzie podstawa dla klas z niej korzystajacych.
Na jej podstawie utworze dwie klasy: AppAction i AppView. Chce, aby kazda wywolana akcja zwracala do kontrolera obiekt klasy AppAction lub AppView ustawiajac w skladowych nazwe akcji do odpalenia i ewentualne argumentu a w AppView nazwe szablonu do zaladowania. Kontroler chce utworzyc w ten sposob, aby dostawal z zewnatrz np. z klasy Router dane, wybieral akcje i wykonywal je dalej o ile zwracane obiekty sa instancja klasy AppAction, jezeli akcja zwroci AppView to wywolam odpowiedni widok. Co myslicie o takim rozwiazaniu? Wydaje mi sie calkiem okej, tylko aktualnie mam problem jak powiazac komunikacje miedzy widokiem a akcjami, gdyz akcje zwracaja mi obiekty AppQuery a chcialbym np. zwrocic rowniez liste wynikow danego zapytania. Macie jakies pomysly? Moze wykorzystac zwracany obiekt AppAction?

Wyglada to mniej wiecej tak:

Router->Front Controller->RunAction(action) tu obsluga lanuchow akcji az do momentu kiedy jakas akcja zwroci AppView.

[PHP] pobierz, plaintext 
<?php
public function run(AppQuery $objAppQuery){
		if($objAppQuery instanceof AppAction){
			$this->runAction($objAppQuery);
		}elseif($objAppQuery instanceof AppView){
			$this->runView($objAppQuery);
		}
	}
 
	public function runAction(AppAction $objAppAction){
		$this->run(call_user_func_array(array('jakismodul', $objAppAction->getName()), $objAppAction->getArgs()));
	}
 
	public function runView(AppView $objAppView){
	}
?>
[PHP] pobierz, plaintext 

Ten post edytował Helios 13.08.2007, 18:08:37

Możesz też poczytać tutaj. Może się przyda ;]

Pozdrawiam.