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 taki kod:

[PHP] pobierz, plaintext 
<?php
 
class FrontController {
	 public $moduleName;
	 public $actionName;
	 
	 public $oModule;
	 
	public $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);
}
 
class AbstractModule implements Module {
	  public $controller;
	 
	 public function __construct(FrontController $controller) {
		$this->controller = $controller;
	}
 
	public function runAction($action) {
		if(method_exists($this->controller->oModule, $action)) {
			$this->controller->oModule->$action();
		} else {
			throw new Exception('Akcja <i>'.$action.'</i> nie istnieje.');
		}
	}
}
 
	$controller		= new FrontController($request);
	$module			= new AbstractModule($controller);
// tutaj klasa modułu z 'extends AbstractModule
$controller->run();
?>
[PHP] pobierz, plaintext 

i pokazuje błąd Fatal error: Argument 1 passed to AbstractModule::__construct() must be an object of class FrontController...on line 47