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

Nie podoba mi sie u Ciebie organizacja MVC. Po pierwsze: strona.pl?model=Cos - zgloszenie moze zawierac jedynie akcje i parametry.

Uruchamiana jest akcja.
Akcja wie jakiego widoku uzyje i jakiego modelu uzyje.

Po drugie: nie kazda akcja potrzebuje modelu. Nawet polowa akcji nie wymaga modelu. Zobacz np. akcje DodajNowosc. Zadne dane nie pobierana sa z bazy (w naszym przykladzie). Akcja uzywa tylko widoku.

Bywa tak, ze akcja uzywa tylko modelu, widoku zas nie. Np: ZapiszUzytkownika.

Ko9lejna sprawa to uzywane kilku modeli w akcji. Zerknij:

[PHP] pobierz, plaintext 
<?php
// Akcja Pokaz zdjecia
 
$album = // pobierz id albumu do wyswietlenia z adresu url;
 
$oAlbym = new Model_Album($album);
$id_autora = $oAlbum->getAuthor();
 
$oAutor = new Model_User($id_autora);
echo 'Album nalezy do ' . $oAutor->getName();
?>
[PHP] pobierz, plaintext 

Nie mozesz wiec automatycznie ladowac widoku i modelu do akcji. Zrob klase bazowa dla akcji np. Action.
Kazda klasa akcji z niej dziedziczy.
Action posiada uzyteczne metody getModel($sModel), getView($sView). Kiedy potrzebujesz modelu/widoku po prostu piszesz w swojej akcji:

[PHP] pobierz, plaintext 
<?php
$oWidok = $this->getView('DodajUzytkownika');
?>
[PHP] pobierz, plaintext 

Co z walidacja?

Zajgorsza czesc aplikacji. Zawsze zajmuje najwiecej czasu i zawsze jest nudna (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) Nie zastanawialem sie jak zrobie to w nowymn frameworku, ale poprzednio mmialem cos w stylu:

[PHP] pobierz, plaintext 
<?php
// akcja DodajUzytkownia;
 
if(formularz nie przeslany) // czyli musimy go wyswietlin
{
  $oView = $this->getView('DodajUzytkownika');
  $oView->display();
  return;
}
 
// form przeslany, sprawdzamy dane:
 
$name = pobierz pole name z formularza;
 
$aErrors = array(); // tablica z komunikatami o bladach.
 
if(empty($name) || strlen($name) > 128)
{
  $aErrors[] = 'Niepoprawna nazwa uzytkownika';
}
 
// tu inne testy.
 
if(count($aErrors)) // czy wystapily jakies blady
{
  $oView = $this->getView('DodajUzytkiownika');
  $oView->set('Errors', $aErrors);
  $oView->display();
  return;
}
 
// tu odpalamy model i dodajemy uzytkiownika.
// na koniec robimy:
 
$this->forward('PokazUzytkiownikow');
?>
[PHP] pobierz, plaintext 

Pozdrawiam, Adrian.