[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...

[Prph]

Witam,

Ostatnio napisalem (mam nadzieje) framework i z przyjemnoscia podziele sie moimi uwagami.

1. NAJWAZNIEJSZA ZASADA
Jezeli piszesz framework dla siebie - pisz go tak, aby byl wygodny dla Ciebie.

2. MVC
MVC mowi, ze masz miec model, widok i kontroler. Model nie wyswietla daych, widok nie grzebie w bazie. Kontroler umozliwia im jakas tam wspolprace.


Nie podoba mi sie struktura katalogow. Tzn ja zmianilbym ja na nastepujaca:

actions/ - kontrolery akcji
models/ - modele
views/ - widoki
configurations/ - konfiguracje wszystkiego co zbudujesz na owym frameworku (np. akcji lub bibliotek, ktore bys mogl pozniej dodac.)

konfiguracje mozesz zapisywac w plikach o przykladowych nazwach: DodajUzytkownikaAction.ini, UsunNewsaAction.ini, MojaBibliotekaJPEGLibrary.ini.

Teraz nieco do kontrolera. Ktos powiedzial, ze jest malo elastyny. No w sumie trudno go zrobic innego, jak kolega dopiero zaczyna.
Ja przyczepie sie do fragmentu nastepujacego:

[PHP] pobierz, plaintext 
<?php
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();
?>
[PHP] pobierz, plaintext 

Dlaczego ladujesz model? To nie nalezy do kontrolera. Ja wiem, ze moze byc tak wygodnie. Ale sa akcje w ktorych z niego nie skorzystasz. Przyklad?

Akcja ktora wyswietla kominikat, jezeli nie znaleziono strony. Np masz akcje PokazZdjecia. Ktos wpisze adres:
jakasStrona.pl/akcja=PokazFotografie.

Jezeli kontroler nie znajdzie PokazFotografie to milo jest, jak odpalisz akcje np. NieZnaleziono. Ona wyswietli jedynie komunikat, zwroci Error 404 lub cos innego. Model nie jest tu najczesciej potrzebny.

Zrob tak:

1. Sprawdz o jaka akcje uzytkownik prosi. np: strona.pl/akcja=PokazZdjecie.
2. Jezeli jest pusta (np. podano strona.pl/), to ustaw domyslna (np. StronaGlowna).
3. Jezeli nie jest pusta, pobiez ja i sprawdz czy istnieje.
4. Jezeli nie, przypisz akcji akcje NieZnaleziono. Sprawdz czy istnieje (tak dla spokoju) i jak nie ma akcji (usunales ja, albu dysk padl (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) ), to zglos jakis blad.

5. Jezeli akcja istanieje - uruchom ja.

Teraz akcja jezeli chce, prosi po model, ustawia sobie widok. Pobiera dane z modelu, przekazuje je widokowi. Widok moze wyswietlic, albo zwrocic kontrolerowi (w kontrolerze piszesz $oWidok = $oAkcja->uruchom()(IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) . Wtedy kontroler ja wyswietla albo robi co mu sie podoba (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

Zeby nie bylo malo, zalaczam fragmenty mojego kodu:

[PHP] pobierz, plaintext 
<?php
class Error403Action extends Action
{
 
  public function execute()
  {
	return Controller::getInstance()->getView('Error403'); // kontroleg glowny tworzy mi obiekt widoku o tej nazwie.
  }
}
?>
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
 
class Error403View extends ExtendedHtmlView
{
  public function __construct()
  {
	parent::__construct(); // wywoluje konstruktor klasy bazowej. Raczej jest mi to potrzebne.
 
	$this->setTemplate('Error403'); // ustawiam sobie szablon. Uzywam zwyklych szablonow php.
  }
}
 
?>
[PHP] pobierz, plaintext 

Dam jeszcze przyklad jak to jest z modelem:

[PHP] pobierz, plaintext 
<?php
class ShowPageAction extends Action
{
  public function execute()
  {
	$oController = Controller::getInstance();
 
	$sPageName = $oController->getContext()->getRequest()->getParameter('page', HttpRequest::GET);
 
	if(is_null($sPageName))
	  $sPageName = 'Index';
 
	$oPageModel = $oController->getModel('Page');
 
	if(!($aPage = $oPageModel->getPageByName($sPageName)))
	{
	  $oController->forward(_ACTION_ERROR_404);
	  return null;
	}
 
	$oView = $oController->getView('ShowPage');
 
	if(!empty($aPage['title']))
	  $oView->setAttribute('pageTitle', $aPage['title']);
 
	$oView->setAttribute('pageContent', $aPage['content']);
 
	return $oView;
  }
 
  public function isSecure()
  {
	return false;
  }
}
?>
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
class PageModel
{
  public function getPageByName($sPageName)
  {
	$sPageName = basename($sPageName);
 
	$sPageFileName = _DIR_APPLICATION . 'pages/' . $sPageName . '.html';
 
	if(is_readable($sPageFileName))
	{
	  $aPage['title'] = null;
	  $aPage['content'] = file_get_contents($sPageFileName);
 
	  return $aPage;
	}
	else
	  return false;
  }
}
?>
[PHP] pobierz, plaintext 

Mama nadzieje, ze troche pomoglem.

Adrian.