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]

Trochę nie tędy droga. Zacznijmy od front kontrolera, który ma właściwości publiczne - powinny być prywatne.

Usunąłeś słowo kluczowe abstract z definicji klasy AbstractModule, która miała być abstrakcyjna. Nie możesz utworzyć instancji tej klasy, bo ona nic nie robi... Posiada tylko metodę runAction, która na podstawie nazwy akcji wywołuje odpowiednią metodę. Te metody musisz już napisać sam w klasie dziedziczącej AbstractModule.

W przypadku tej konstrukcji, nie widzę sensu w podawaniu do konstruktora klasy modułu instancji front controllera. Na nic on nie jest potrzebny w tej klasie...

Metoda runAction() w AbstractModule jest źle napisana - po co wyciągać obiekt modułu z front controllera? Przecież front controllera wywołuje tę metodę dla odpowiedniego modułu, dlatego runAction() operuje na składowych instancji, dla której została wywołana. Dlatego powinno być:

[PHP] pobierz, plaintext 
<?php
public function runAction($action) {
	if(method_exists($this, $action)) {
		$this->$action();
	} else {
		throw new Exception('Akcja <i>'.$action.'</i> nie istnieje.');
	}
}
?>
[PHP] pobierz, plaintext 

Warto wyrzucić do tego jakiś lepszy wyjątek, typu NoActionException. Komunikat nic Ci tak na prawdę nie daje w kodzie. Po typie wyjątku możesz już manipulować przechwytywaniem...

Warto też do akcji jakoś przekazać obiekt żądania.