[MVC] Moj poczatek / implementacja Opcje

[Helios]

Witam
Od niedawna zaczalem interesowac sie MVC. Ale w dalszym ciagu nie wiem czy dobrze rozumuje istote MVC. Dlatego postanowilem napisac prosty (powtarzam, prosty, system) zeby sprawdzic czy ide w dobrym kierunku. Prosze o komentarze, jesli chodzi o sam schemat i istote wzorca MVC.

Na poczatek przedstawie strukture katalogow:

./ - katalog glowny
actions/ - katalog przechowujacy pliki .ini do wybranej akcji mozemy podac co ma zostac wykonane nastepnie
configs/ - katalog z plikami konfiguracyjnymi - narazie nie uzyty
controllers/ - katalog z kontrolerami - narazie tylko kontroler glowny
models/ - katalog z plikami modeli
views/ - katalog z plikami widokow
index.php - plik glowny ktory uruchamia kontroler

Zaczynamy od poczatku - kod kontrolera:

[PHP] pobierz, plaintext 
<?php
 
class Ctrl{
 
	private $Request=false; // obiekt
	private $action; // nazwa akcji - nieobowiazkowa
	private $view; // nazwa widoku
	protected $model; // nazwa modelu - nieobowiazkowa
 
	protected $objView; // obiekt widoku
	protected $objModel; // obiekt modelu
 
	// konstruktor
	public function __construct($request){ 
 
		// ustawianie pol klasy
		$this->view=$request->getParam('view');
		$this->view=($this->view) ? $this->view : 'main'; // ustawianie standardowego widoku
		// jesli zostal podany model ustawia jesli nie jest taki sam jak nazwa widoku
		$this->model=($request->getParam('model')) ? $request->getParam('model') : $this->view;
		// model z sufixem Mod
		$this->model.='Mod';
		// pobiera parametr akcji o ile zostal podany
		$this->action=$request->getParam('action');
 
		// jesli zostala podana akcja laduje model i wykonuje odpowiednia czynnosc
		if($this->action){
			require_once('./models/'.$this->model.'.model.php');
			$this->objModel=new $this->model($request);
			$action=$this->action;
			$this->objModel->$action();
			do{
				// sprawdza czy plik konfiguracyjny akcji uwzglednia wykonanie kolejnej akcji
				$tmpCfg=parse_ini_file('./actions/'.$this->action.'.ini');
				$action=$tmpCfg['next'];
				// jesli tak to ja wykonuje itd.
				$this->objModel->$action();
			}while(!$tmpCfg['next']);
		}
		// laduje obiekt widoku
		$this->execute();
	}
 
	public function execute($view=false){
		// sprawdza czy obiekt modelu juz istnieje czyli czy podana byla akcja
		if(!is_object($this->objModel)){
			require_once('./models/'.$this->model.'.model.php');
			$this->objModel=new $this->model($request);
		}
		// laduje obiekt widoku
		require_once('./views/'.$this->view.'.view.php');
		$this->objView=new $this->view($this->objModel);
	}
 
}
 
?>
[PHP] pobierz, plaintext 

Zakladamy, ze uzytkownik wpisal: index.php?view=Users&action=addUser&userName=root
kontroler laduje wiec model UsersMod:

[PHP] pobierz, plaintext 
<?php
 
class UsersMod {
 
		public $addedUser;
		public $request;
		public $count;
 
		function __construct($request){
			$this->request=$request;
		}
 
		function addUser(){
			$this->addedUser=$this->request->getParam('userName');
			return true;
		}
 
		function countUsers(){
			return $this->count++;
		}
 
		function getAddedUser(){
			return $this->addedUser;
		}
 
		function getUsersNum(){
			return $this->count;
		}
 
}
 
?>
[PHP] pobierz, plaintext 

i wykonuje w nim akcje zmianu danych, a poniewaz akcja addUser() posiada plik konfiguracjny z wywolaniem jako kolejnej akcji countUsers wywoluje kolejna akcje. No i pozostaje wyswietlenie widoku Users:

[PHP] pobierz, plaintext 
<?php
 
class Users {
 
	function __construct($objModel){
		print 'Dodalem uzytkownika: '.$objModel->getAddedUser().' i teraz mam zarejestrowanych uzytkownikow: '.$objModel->getUsersNum();
	}
 
}
 
?>
[PHP] pobierz, plaintext 

Co sadzicie o tym? Czy dobrze rozumuje istote MVC? Zapraszam do dyskusji...

[thornag]

@prph korzystajac z Twojej zyczliwosci a przedewszystkim cierpliwosci i dobrych rad postanowilem w koncu cos splodzic. No i wyglada to tak.

Controller

[PHP] pobierz, plaintext 
<?php
class Controller {
 
		protected $sAction;
 
		protected $oRequest;
 
		protected $sDefaultAction = 'Index';
 
		protected $sView;
 
		protected $sDefaultView = 'Index';
 
		protected $aVars = array();
 
		public function __construct() {
			$oRequest = New WebRequest();
		}
 
		static function & getInstance() {
			return $this;
		}
 
		public function forward($sAction) {
			$this->sAction = $sAction;
			$this->Perform();
		}
 
		public function getAction($sAction = null) {
			$this->sAction = empty($sAction) ? $this->sDefaultAction : $sAction;
		}
 
		public function getRequest() {
			return $this->oRequest;
		}
 
		public function getView($sView = null) {
			$this->sView = empty($sView) ? $this->sDefaultView : $sView;
		}
 
		public function setVariable($value, $key = null) {
			$this->aVars[$key] = $value;
		}
		public function Perform() {
			$sActionPath = _PATH_ACTION.'actions.'.$this->sAction.'.php';
			require_once(_PATH_LIB.'class.Action.php');
 
			if (!file_exists($sActionPath)) {
 
				$this->forward("ActionDoesntExists");
 
			} else {
 
				require_once($sActionPath);
			}
			$oAction = New $this->sAction();
			$oAction->Execute();
			return;
		}
 
		  public function Display() {
			  $sTemplatePath = _PATH_TEMPLATE.'views.'.$this->sView.'.php';
			  require_once(_PATH_LIB.'class.View.php');
 
			  if (!file_exists($sTemplatePath)) {
				  $this->forward("ViewDoesntExists");
			  } else {
				  require_once($sTemplatePath);
			  }
			  $oView = New $this->sView();
			  $oView->Assign(& $this->aVars);
			  $oView->Parse();
		  }
 
	}
?>
[PHP] pobierz, plaintext 

Przykladowy widok

[PHP] pobierz, plaintext 
<?php
abstract class View {
 
		protected $oSmarty;
 
		public function __construct() {
			$this->oSmarty = New Smarty();
		}
 
		public function Assign(& $aVars) { 
			foreach ($aVars as $key => $value) {
					$this->oSmarty->Assign($key, $value);
				}	
		}
 
		abstract function Parse() {}
	}
 
 
	class ActionDoesntExists extends View {
 
		public function Parse() {
			$oSmarty->setTemplate('path/template');
			$oSmarty->Display();
		}
	}
 
?>
[PHP] pobierz, plaintext 

I akcja

[PHP] pobierz, plaintext 
<?php
 
	abstract class Action {
 
		protected $oController;
 
		public function __construct() {
			$this->oController = Controller::getInstance();
		}
 
		abstract function Execute() { }
	}
 
 
 
	class ActionDoesntExists extends Action {
 
		public function Execute() {
			$oController->getView(get_class($this));
		}
	}
?>
[PHP] pobierz, plaintext 

EDIT:
I wywolanie

[PHP] pobierz, plaintext 
<?php
 
 
	$Application = New Controller();
	$Application->getAction($Application->getRequest()->getParam('action', Request::GET));
	$Application->Perform();
	$Application->Display();
 
?>
[PHP] pobierz, plaintext 

EDIT END

Zaczalem wczoraj tak na dziko troche, fakt faktem tak jak pisales zdaje sobie sprawe z tego ze aplikacja moze sie zapetlic wiec musze dodac jeszcze jakies wyjatki i obsluge tych bledow, no chyba ze zalozenie ze akcja ActionDoesntExists i ViewDoesntExists i odpowiadajace jej widoki sa zawsze (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) Ale jak to mowia lepiej dmuchac na zimne (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Ogolnie Prph, czy zrealizowalem ogolne zalozenia MVC tym kodem ?

Jesli tak to oczywiscie zdaje sobie sprawe z tego ze to pierwszy framework i w miare budowania na nim aplikacji bede wpadal na nowe i (mam nadzieje) lepsze pomysly. Zalezy mi tylko na jakims komentarzu i stwierdzeniu wprost czy moj start jest zgodny czy nie zgodny z zalozeniami (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

P.S jeszcze nie testowalem, dzisiaj napisze klase Request i Abstrakcje dla DB i jakis model. (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Ten post edytował thornag 25.08.2006, 12:28:58