[mvc] Pare pytan, lancuchy akcji etc. Opcje

[Helios]

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

[NoiseMc]

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