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

[bim2]

Mhh, dobra to może teraz cos takieog pokaże. Fajnie działa:
index.php

[PHP] pobierz, plaintext 
<?
define('PATH_ACTION', 'actions/');
define('PATH_MODEL', 'models/');
define('PATH_VIEW', 'views/');
define('PATH_CLASS', 'class/');
define('PATH_LIB', 'libs/');
$conn = mysql_connect('localhost', 'root', '');
mysql_select_db('nfs_new', $conn);
class Controler {
public $objAction; //objekt akcji
public $action; //akcja
private $defaultAction = 'index'; //domyślna akcja
public $objMod;
public $model;
public $objViewC;
public $objView;
 
public $router;
public $oldAction;
public $getView;
public $view;
 
	function __construct($router)
	{
		$this->router = $router;
		$this->action = $router->getString('action');
		$this->view = $this->router->getString('view');
	}
	function & getInstance()
	{
		return $this;
	}
	function getAction()
	{
		return $this->action = ($this->action) ? $this->action : $this->defaultAction;
	}
	function setAction($action='')
	{
		$this->action = $action;
		$this->perfom(true);
	}
	function setView($view='')
	{
		$this->view = ($view) ? $view : 'defaults';
		$this->perfom();
	}
	function getView()
	{
		return ($this->view) ? $this->view : 'defaults';
	}
	function perfom($doesntAction=false)
	{
		$action_path = PATH_ACTION.$this->getAction().'.action.php';
 
		if(!file_exists($action_path))
		{	
			if(!$doesntAction)
			{	
				$this->oldAction = $this->getAction();
				$this->setAction('ActionDoesntExists');
				return;
			}
			else
			{
				echo 'Nie można znaleĽć '.$this->action;
				return;
			}
		}
		else
		{	
			if(!is_object($this->objAction))
			{
			require_once $action_path;
			$act = $this->getAction();
			$this->objAction = New $act($this);
			}
			$gview = $this->getView();
			$this->objAction->$gview($this);
 
		}
	}
	function display()
	{
		if(!is_object($this->objViewC))
		{
		require_once PATH_LIB.'view.lib.php';
		$this->objViewC = new View();
		}
		return $this->objViewC;			
	}
 
	function model()
	{
		$this->model = $this->getAction().'Mod';
		$path = PATH_MODEL.$this->getAction().'.model.php';
		if(!is_object($this->objMod))
		{
			if(!file_exists($path))
			{
				$this->setAction('ModelDoesntExists');
				return;
			}
			else
			{
				require_once $path;
			}
 
		$this->objMod = new $this->model($this);
		}
 
		return $this->objMod;			
 
	}
 
		function view()
	{
		$this->view = $this->getAction().'View';
		$path = PATH_VIEW.$this->getAction().'.view.php';
		if(!is_object($this->objView))
		{
			if(!file_exists($path))
			{
				$this->setAction('ViewDoesntExists');
				exit;
			}
			else
			{
				require_once $path;
			}
 
		$this->objView = new $this->view($this);
		}
 
		return $this->objView;			
 
	}
 
}
class Router
{
	function getString($name_param, $typ='get')
	{
		return  (string) $_GET[$name_param];
	}
}
$router = new Router;
 
$ctrl = new Controler($router);
$ctrl->perfom();
?>
[PHP] pobierz, plaintext 

libs/view.lib.php

[PHP] pobierz, plaintext 
<?
class View {
private $Assign;
function assign($assign)
{
	foreach ($assign as $key => $value) 
	{
		echo $key.'= '.$value.'<hr>';
 
	} 
}
function fetch($plik)
{
	return $this->Assign;
}
}
?>
[PHP] pobierz, plaintext 

models/index.model.php

[PHP] pobierz, plaintext 
<?
class IndexMod {
public $query;
public $ctrl;
	function __construct($ctrl)
	{	
		$this->ctrl = $ctrl;
	}
	function defaults()
	{
		$q=mysql_query("SELECT * FROM new_news");
		while($r=mysql_fetch_assoc($q))
		{	
		$news[$r['news_id']]['tytul']=$r['news_tytul'];
		$news[$r['news_id']]['text']=$r['news_tekst'];
 
		}
		$this->query = $news;
	/*$view = $cont->display();
	$view->assign(array('lol' => 'lolek'));
	echo $view->fetch('jakis_plik');*/
	}
	function news()
	{
		if($this->ctrl->router->getString('id'))
		{
		$q=mysql_query("SELECT * FROM new_news WHERE news_id='".$this->ctrl->router->getString('id')."'");
		$r=mysql_fetch_assoc($q);
		$news['tytul']=$r['news_tytul'];
		$news['text']=$r['news_tekst'];
		$this->query = $news;
		}
		else
		{
			$this->ctrl->setView();
		}
	/*$view = $cont->display();
	$view->assign(array('lol' => 'lolek'));
	echo $view->fetch('jakis_plik');*/
	}
}
?>
[PHP] pobierz, plaintext 

views/index.view.php

[PHP] pobierz, plaintext 
<?
class IndexView {
public $query;
	function __construct($ctrl)
	{	
 
	}
	function defaults($mod)
	{
	$views = $mod->ctrl->display();
	foreach($mod->query AS $val)
	{
		$views->assign($val);
	}
 
		echo $views->fetch('rrf');
	/*$view = $cont->display();
	$view->assign(array('lol' => 'lolek'));
	echo $view->fetch('jakis_plik');*/
	}
	function news_one($mod)
	{
		$views = $mod->ctrl->display();
		$views->assign($mod->query);
 
		echo $views->fetch('rrf');
	}
}
?>
[PHP] pobierz, plaintext 

no i ostanie
actions/index.action.php

[PHP] pobierz, plaintext 
<?
class Index {
	function __construct($ctrl)
	{	
 
	}
	function defaults($ctrl)
	{
		$mod = $ctrl->model($ctrl);
		$mod->defaults();
		$widok = $ctrl->view();
		$widok->defaults($mod);
 
	}
	function news($ctrl)
	{
		$mod = $ctrl->model($ctrl);
		$mod->news();
		$widok = $ctrl->view();
		$widok->news_one($mod);
 
	}
}
?>
[PHP] pobierz, plaintext 

Czy jest juz na lepszej drodze? Powiem tlyo że to działa ?view=news&id=1 lub jak podasz tylko view=news to przejdzie do defaults() (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) Jestem na dobrej drodze? Prosze powiedzieć prosto z mostu. "TAK" lub "NIE" (IMG:http://forum.php.pl/style_emoticons/default/biggrin.gif)