Mam problem. Otóż nie mogę wyważyć w jaki sposób odpalać te kontrolery we frameworku, mvc. Pierwszy sposób zawodzi, gdy znajdzie się w Managerze metoda z walidacją, ponieważ muszę użyć instrukcji if by sprawdzić, czy została wywołana i jeśli tak wywołać również walidację. Natomiast druga zawodzi w momencie, gdy ktoś chce złamać zabezpieczenia, bowiem metodę widać w adresie strony. Proszę o rady. Przejrzałem frameworki, ale nie znalazłem odpowiedzi.


Adres:

[PHP] pobierz, plaintext 
index.php?module=user&action=shownotactive&other=1_2
[PHP] pobierz, plaintext 

1) metoda pierwsza

[PHP] pobierz, plaintext 
class UserController{
 
  public function __construct($action, Array $other){
 
    $usermanager = new UserManager;
    $usermanager->$action($other);  
 
  }  
 
}
[PHP] pobierz, plaintext 

2) metoda druga

[PHP] pobierz, plaintext 
 
class UserController{
 
  private $index;
 
  public function __construct($action){
 
    $usermanager = new UserManager;
    $this->index->$action;  
 
  }  
 
  public function set(){
 
    if(isset($this->index)){
 
      $usermanager->showPanel();
 
    }
 
    if($this->index == 'shownotactive'){
 
      $usermanager->shownotactive();
 
    }
 
    if($this->index == 'addnewuser'){
 
      $usermanager->addnewuser();
 
    }
 
  }  
[PHP] pobierz, plaintext 

Tym, jaka akcja dla danego kontrolera powinna zostać wykonana zajmuje się front controller.
Jeżeli nie ma żądanego kontrolera lub brakuje akcji dla danego kontrolera, to powinien (front controller) w tym miejscu zwrócić wyjątek.
Coś w tym stylu, załóżmy, że metoda uruchamiająca odpowiednią akcję nazywa się run:

[PHP] pobierz, plaintext 
public function run()
{
  $request = $this->getRequest(); //metoda request powinna zwracać obiekt z danymi dotyczącymi requestu
  $controllerName = $request->getControllerName(); //zwraca nazwę kontrolera lub nazwę domyślnie ustawioną, gdy url jest bez parametrów
  $actionName = $request->getActionName(); //analogicznie jak wyżej
  try {
    $controller = new $controllerName; //autoloader powinien wyrzucić wyjątkiem, jeżeli nie znajdzie kontrolera
    $controller->setRequest($request); //request powinien zostać przekazany do kontrolera
  }
  catch(Autoloader_Exception $e)
  {
      throw new Controller_Exception('Kontroler ' . $controllerName .' nie istnieje.'); //wyrzucone, gdy autoloader nie znajdzie klasy
  }
 
  if(!method_exists($controller, $actionName))
     throw new Controller_Exception('Akcja '.$actionName.' nie istnieje.'); //wyrzucane, gdy brakuje metody
 
  $controller->{$actionName}; //wykonanie akcji, możesz zwracać wynik, jeżeli zachodzi taka potrzeba.
}
[PHP] pobierz, plaintext 

A kod kontrollera wygląda np. tak:

[PHP] pobierz, plaintext 
Controller_A {
public function foo() {}//gdy url: www.site.com/a/foo
public function boo() {}//gdy url: www.site.com/a/boo
}
[PHP] pobierz, plaintext 

Radzę zastosować zamiast takich urli: www.site.com/index.php?module=user&action=shownotactive&other=1_2 coś takiego: www.site.com/user/shownotactive?other=1_2 wygląda o wiele ładniej. Wartość parametru other powinna być dostępna z obiektu Request, a nie przekazywana jako parametr metody akcji.

Dzięki wielkie, pomogło

Mógłbyś wytłumaczyć budowę requestu w tym przykładzie ?

Czy getRequest powinien zwracać referencję do tablicy ?

Metoda getRequest() powinna zwracać obiekt klasy Request, która zawiera informacje nt. aktualnego requestu, czyli jej budowa może wyglądać mniej więcej tak:

[PHP] pobierz, plaintext 
class Request {
private $_controllerName;
private $_actionName;
private $_params = array(); //dobrze byłoby, gdyby istniał podział na parametry przekazane za pomocą metody GET i POST
public function getControllerName($default = 'index') {} //jeżeli nie został przekazany w url kontroler, to zostaje zwrócony domyślny
public function getActionName($default  = 'index') {} //analogicznie j.w.
public function getParams($method = Request::ALL) {} //zwraca wszystkie parametry. Tutaj radziłbym zastosować jakieś consty do określenia czy wszystkie wysłane postem, getem, czy ogólnie wszystkie
public function getParam($name, $default = null) {} //zwraca wartość parametry lub $default, gdy parametr nie został przekazany
}
[PHP] pobierz, plaintext 

Oczywiście to tylko sugestia, bo można spokojnie to rozbudować. Dodatkowo pozostaje decyzja, czy parametry są przekazywane, czy sam obiekt dobiera się do nich z tablic globalnych. To samo tyczy się nazw akcji i kontrolera. Czy pobierać je wewnątrz obiektu klasy, czy może przekazywać. Jeżeli chcesz to wszystko ustawiać to raczej proponowałbym takie dane przekazywać w konstruktorze, a nie za pomocą seterów, ponieważ logika podpowiada, że po utworzeniu obiektu klasy request raczej nie powinien się zmienić.
Temat można rozwinąć, jeżeli zakładasz również obsługę kodu z linii poleceń. W takim wypadku doradzałbym wyciągnięcie klasy bazowej (abstrakcyjnej) wyżej i stworzenie Request_HTTP i Request_CLI lub coś w tym stylu.