Pierwsze kroki z MVC Opcje

[Vielta]

Hej!
W celu lepszego zrozumienia idei MVC postanowiłem troszkę się pobawić i korzystając z wielu dostępnych mi źródeł napisać coś na wzór pseudo-MVC-frameworka. Mam jednak parę pytań, z czego najważniejsze to czy to się w ogóle trzyma kupy?

Autoloader wygląda następująco:

[PHP] pobierz, plaintext 
<?php		
	function __autoload($className) {
		if(file_exists(ROOT.DIRECTORY_SEPARATOR.'library'.DIRECTORY_SEPARATOR.strtolower($className).'.class.php')) {
			require_once ROOT.DIRECTORY_SEPARATOR.'library'.DIRECTORY_SEPARATOR.strtolower($className).'.class.php';
		} elseif(file_exists(ROOT.DIRECTORY_SEPARATOR.'application'.DIRECTORY_SEPARATOR.'controllers'.DIRECTORY_SEPARATOR.strtolower($className).'.php')) {
			require_once ROOT.DIRECTORY_SEPARATOR.'application'.DIRECTORY_SEPARATOR.'controllers'.DIRECTORY_SEPARATOR.strtolower($className).'.php';
		} elseif(file_exists(ROOT.DIRECTORY_SEPARATOR.'application'.DIRECTORY_SEPARATOR.'models'.DIRECTORY_SEPARATOR.strtolower($className).'.php')) {
			require_once ROOT.DIRECTORY_SEPARATOR.'application'.DIRECTORY_SEPARATOR.'models'.DIRECTORY_SEPARATOR.strtolower($className).'.php';
		} elseif(file_exists(ROOT.DIRECTORY_SEPARATOR.'library'.DIRECTORY_SEPARATOR.'helpers'.DIRECTORY_SEPARATOR.strtolower($className).'.class.php')) {
			require_once ROOT.DIRECTORY_SEPARATOR.'library'.DIRECTORY_SEPARATOR.'helpers'.DIRECTORY_SEPARATOR.strtolower($className).'.class.php';
		} else {
			// nie ma takiego pliku, jakis error handler?
		}
	}
[PHP] pobierz, plaintext 

Mam taki pseudo "front Controller", który ma za zadanie odpalenie odpowiedniego kontrolera na podstawie $_SERVER['request'] i wygląda on tak:

[PHP] pobierz, plaintext 
<?php
	class FrontController {
		private $_queryString;
		private $_controller;
		private $_action;
		private $_dispatch;
		public function callHook($url) {
			$urlArray = explode('/', $url);
 
			$controller = $urlArray[0]?$urlArray[0]:'main';
			array_shift($urlArray);
			$action = $urlArray[0]?$urlArray[0]:'index';
			array_shift($urlArray);
			$queryString = $urlArray;
 
			$controllerName = $controller;
			$controller = ucfirst($controller);
			$model = $controller;
			$controller .= 'Controller';
 
			$this->_queryString = $queryString;
			$this->_controller = $controller;
			$this->_action = $action;
			if(class_exists($controller)) {
				$this->_dispatch = new $controller($model, $controllerName, $action);
			} else {
				$this->_action = 'error404';
				$this->_controller = 'error';
				$this->_queryString = $_SERVER;
				$this->_dispatch = new ErrorController('error', 'error', 'error404');
			}
		}
 
		public function addHelper($class) {
			if($class instanceof TemplateHelper) {
				$this->_dispatch->addTemplateHelper($class);
			} else {
				$this->_dispatch->addModule($class);
			}
		}
 
		public function run() {
			if((int)method_exists($this->_controller, 'init')) {
				$this->_dispatch->init();
			}
			$this->_dispatch->{$this->_action}($this->_queryString);
			$this->_dispatch->render();
		}
	}
[PHP] pobierz, plaintext 

Klasa Controller, po której dziedziczy każdy kontroler wygląda natomiast tak

[PHP] pobierz, plaintext 
	class Controller {
		protected $_model;
		protected $_controller;
		protected $_action;
		protected $_template;
		protected $_modules;
 
		public function __construct($model, $controller, $action) {
			$this->_controller = $controller;
			$this->_model = $model;
			$this->_action = $action;
			$this->_template = new Template($this->_controller, $this->_action);
		}
 
		public function addModule($module) {
			$this->_modules[get_class($module)] = $module;
		}
 
		public function addTemplateHelper($helper) {
			$this->_template->addTemplateHelper($helper);
		}
 
		public function set($name, $value) {
			$this->_template->set($name, $value);
		}
 
		public function render() {
			$this->_template->render();
		}
	}
[PHP] pobierz, plaintext 

A klasa Template, która odpowiada za wczytanie i wyrenderowanie odpowiedniej templatki (jest dość primitywna)

[PHP] pobierz, plaintext 
<?php
	class Template {
		protected $_variables = array();
		protected $_controller;
		protected $_action;
		protected $_helpers;
 
		public function __construct($controller, $action) {
			$this->_controller = $controller;
			$this->_action = $action;
		}
 
		public function __set($name, $value) {
			$this->set($name, $value);
		}
 
		public function set($name, $value) {
			$this->_variables[$name] = $value;
		}
 
		public function addTemplateHelper($helper) {
			$this->_helpers[get_class($helper)] = $helper;
		}
 
		public function getHelper($helper) {
			return $this->_helpers[$helper];
		}
 
		public function load($helper) {
			$template = new Template('helpers', $helper);
			$template->render();
		}
 
		public function render($doNotRenderHeader = false) {
			extract($this->_variables);
 
			if($doNotRenderHeader == false) {
				if(file_exists(ROOT.DIRECTORY_SEPARATOR.'application'.DIRECTORY_SEPARATOR.'views'.DIRECTORY_SEPARATOR.$this->_controller.DIRECTORY_SEPARATOR.'header.phtml')) {
					require_once ROOT.DIRECTORY_SEPARATOR.'application'.DIRECTORY_SEPARATOR.'views'.DIRECTORY_SEPARATOR.$this->_controller.DIRECTORY_SEPARATOR.'header.phtml';
				} elseif(file_exists(ROOT.DIRECTORY_SEPARATOR.'application'.DIRECTORY_SEPARATOR.'views'.DIRECTORY_SEPARATOR.'header.phtml')) {
					require_once ROOT.DIRECTORY_SEPARATOR.'application'.DIRECTORY_SEPARATOR.'views'.DIRECTORY_SEPARATOR.'header.phtml';
				}
			}
			require_once ROOT.DIRECTORY_SEPARATOR.'application'.DIRECTORY_SEPARATOR.'views'.DIRECTORY_SEPARATOR.$this->_controller.DIRECTORY_SEPARATOR.$this->_action.'.phtml';
 
			if($doNotRenderHeader) {
				if(file_exists(ROOT.DIRECTORY_SEPARATOR.'application'.DIRECTORY_SEPARATOR.'views'.DIRECTORY_SEPARATOR.$this->_controller.DIRECTORY_SEPARATOR.'footer.phtml')) {
					require_once ROOT.DIRECTORY_SEPARATOR.'application'.DIRECTORY_SEPARATOR.'views'.DIRECTORY_SEPARATOR.$this->_controller.DIRECTORY_SEPARATOR.'footer.phtml';
				} elseif(file_exists(ROOT.DIRECTORY_SEPARATOR.'application'.DIRECTORY_SEPARATOR.'views'.DIRECTORY_SEPARATOR.'footer.phtml')) {
					require_once ROOT.DIRECTORY_SEPARATOR.'application'.DIRECTORY_SEPARATOR.'views'.DIRECTORY_SEPARATOR.'footer.phtml';
				}
			}
		}
	}
[PHP] pobierz, plaintext 

Jeśli komukolwiek udało się przebrnąć przez te linijki kodu mam 3 pytania:
a) Czy udało mi się uchwycić ideę OOP
(IMG:style_emoticons/default/cool.gif) Czy udało mi się uchwycić ideę MVC
c) Co mogę poprawić, żeby kod był bardziej OOP/MVC.

Przepraszam za chaotyczność (?) tego postu ale mam grypę i logiczne myślenie jest ciężkawe :<.

Pozdrawiam i z góry dzięki,
Vielta

[Quantum]

Trochę dziwnie na moje oko rozwiązałeś sprawę front controllera, ja u siebie mam zadania routingu/walidacji rozbite na dwie klasy - router oraz dispatcher. Dla tej pierwszej przekazuje w argumencie query string i po wykonaniu dostaje od niej ostateczne dane. Tworzenie obiektu i wywoływanie akcji zrzucam właśnie na klasę front controller. W nim tworzę obiekt dispatcher-a i on sprawdza czy żądany kontroler/akcja istnieje (oraz czy jest poprawny - przydaje się tutaj ReflectionClass) jeżeli tak załącza odpowiedni plik i po utworzeniu obiektu kontrolera odpalamy akcję. Co do autoloader-a to ja to rozwiązałem w ten sposób, że nazwy klas zapisywane są w stylu "underscore", bardziej po zendowsku (IMG:style_emoticons/default/biggrin.gif) Przykładowo z ciągu af_Database_Interface_QueryBuilder usuwam prefix "af" i podkreślniki zamieniam na ukośniki, konwertuje na małe znaki i dostaję ścieżkę "/database/interface/querybuilder". W taki sposób autoloader elegancko odnajduje sobie poprawną ścieżkę. Oczywiście nie musisz robić tego w sposób jaki podałem. Luźne sugestie (IMG:style_emoticons/default/winksmiley.jpg)

pozdrawiam

edit:

TYPE HINT zdecydowanie możesz użyć w metodach gdzie wymogiem jest przekazanie obiektu. Np w twoim addTemplateHelper.

Przecież w PHP nie da się go nigdzie indziej użyć (no... tam jeszcze tablice, ale to śmiechu warte jest :])

urok tego języka (IMG:style_emoticons/default/biggrin.gif) nie ma to jak dodać obsługę dla obiektów/tablic i zapomnieć o reszcie

Ten post edytował Quantum 8.09.2010, 18:40:18