Wywołanie kontrolera Opcje

[cer98]

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 

[bastard13]

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.