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

[Ludvik]

Powiązania, jak sam zauważyłeś, są złe. Zacznijmy od tego, że Front Controller nie jest akcją - on wywołuje akcje, a to zupełnie inna relacja. Jeżeli chcesz podzielić działanie w stylu: moduł = klasa, akcja = metoda, to bym proponował stworzenie interfejsu modułu:

[PHP] pobierz, plaintext 
<?php
interface Module {
	function runAction($action);
}
?>
[PHP] pobierz, plaintext 

U Ciebie za moduł robił Sub Controller, ale nazwa Module wydaje się logiczniejsza moim zdaniem, szczególnie, że używasz jej w nazwie metody runModule(). Możesz pokusić się o stworzenie klasy abstrakcyjnej, albo zastąpić nią interfejs:

[PHP] pobierz, plaintext 
<?php
abstract class AbstractModule implements Module {
	public function runAction($action) {
		if(method_exists($this->frontController->oAction, $action)) {
			$this->frontController->oAction->$action();
		} else {
			throw new NoActionException();
		}
	}
}
?>
[PHP] pobierz, plaintext 

Front Controller jest ok, tylko usuń to dziedziczenie... Do tego bym odpalał obsługę żądania nie w konstruktorze, ale osobnej metodzie np. run().