Kontroler, akcje ale trochę inaczej. Opcje

[Joachim Peters]

Witam,

Ostatnio trochę czytałem na temat MVC i tak naprawdę tylko niektóre rzeczy z tego mi się podobają, nie wiem może jeszcze nie dostrzegam za wielu plusów, ale chce zastosować u siebie coś podobnego. Napisałem sobie klasę router i request, pierwsza pobiera dane z adresu, druga z formularzy, cookies, sesji i obsługuje dane z routera, klasę FrontController, które uruchamia moduł i tworzy instancję i klasę Action, uruchamia akcję (wywołuje metodę klasy).
Kodowo wygląda to mniej więcej tak:

[PHP] pobierz, plaintext 
<?php
 
class FrontController {
	 public $module;
	 public $action;
	 public $objectAction;
	 
	public $request;
	 
	public function __construct(HttpRequest $request) {
		$this->request = $request;
 
		$this->module = strtolower($this->request->get('get', 0));
		$this->action = strtolower($this->request->get('get', 1));
 
		$this->runModule();
	}
 
	public function runModule() {
		if(file_exists('./modules/user/'.$this->module.'.php')) {
			include_once('./modules/user/'.basename($this->module.'.php'));
 
			$module = ucfirst($this->module);
 
			$this->objectAction = new $module();
		} else {
			throw new Exception('Nie można znaleźć plików modułu');
		}
	}	
}
 
class Action {
	 private $controller;
	 
	 public function __construct(FrontController $controller) {
		  $this->controller = $controller;
 
		$this->runAction();
	}
 
	public function runAction() {		
		if(!is_null($this->controller->objectAction)) {
			 $action = $this->controller->action;
			 
			$this->controller->objectAction->$action();
		} else {
			throw new Exception('Brak akcji');
		}
	}
}
 
$controller = new FrontController($request);
$action	= new Action($controller);
 
?>
[PHP] pobierz, plaintext 

Jak powiecie czy to rozwiązanie jest optymalne, warto to dalej kodować, czy lepiej zabrać się za kodowanie 'standardowe' - z wprowadzaniem argumentów do funkcji itd.?

[Joachim Peters]

Mam teraz tak:

[PHP] pobierz, plaintext 
<?php
 
class FrontController {
	 private $moduleName;
	 private $actionName;
	 
	 private $oModule;
	 
	private $request;
	 
	public function __construct(HttpRequest $request) {
		$this->request = $request;
 
		$this->moduleName = strtolower($this->request->get('get', 0, 'htmlspecialchars'));
		$this->actionName = strtolower($this->request->get('get', 1, 'htmlspecialchars'));
	}
 
	public function run() {
		$this->runModule();
	}
 
	private function runModule() {
		if(file_exists('./modules/user/'.$this->moduleName.'.php')) {
			include_once('./modules/user/'.basename($this->moduleName.'.php'));
 
			$moduleName = ucfirst($this->moduleName);
 
			if(class_exists($moduleName)) {
				$this->oModule = new $moduleName();
				$this->oModule->runAction($this->actionName);
			} else {
				throw new Exception('Moduł <i>'.$this->moduleName.'</i> nie istnieje.');
			}
		} else {
			throw new Exception('Nie można znaleźć pliku modułu <i>'.$this->moduleName.'</i>.');
		}
	}	
}
 
interface Module {
	public function runAction($action);
}
 
abstract class AbstractModule implements Module {
	public function runAction($action) {
		if(method_exists($this, $action)) {
			$this->$action();
		} else {
			throw new Exception('Akcja <i>'.$action.'</i> nie istnieje.');
		}
	}
}
 
$controller = new FrontController($request);
$controller->run();
?>
[PHP] pobierz, plaintext 

Pytanie i problemy:
1. Chciałbym jeszcze zrobić, żeby w module było można używać funkcji i danych z klasy request bez potrzeby tworzenia instancji w każdej z klas gdzie ma być to użyte.
2. Nie bardzo rozumiem, jaki sens jest w tworzeniu typu wyjątku takiego jaki podałeś?
3. Co teraz z szablonami? Chce użyć zwykłej składni PHP w HTML-u w plikach .php i czy robić do wczytywania tego jakąś osobną klasę czy w metodach modułów dawać po prostu include?
4. Czy jestem taki toporny czy mi się tylko wydaję? (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Ten post edytował Joachim Peters 31.08.2007, 15:55:27