Witam
Zastanawiałem się ostatnio jak sprawdzić czy w adresie jest podana wystarczająca ilośc parametrów które trzeba przekazać do metody jako argumenty. Pierwszy problem to przekazanie ich do metody w odpowiedniej kolejności np:

[PHP] pobierz, plaintext 
<?php
class viewAction extends Action {
 
public function execute($idNewsa, $sort, $param3) // to trzeba przekazac w odpowiedniej kolejności
{
 
}
 
}?>
[PHP] pobierz, plaintext 

adres mam w postaci: http://page.pl/controller/action/idNewsa/5...ytitle/param3/n
no i gdy rozdziele adres, tablica jest typu:
array('idNewsa' => 5, 'sort' => 'bytitle', 'param3' => 'n');

No i jak je przekazać do funkcji w poprawnej kolejności. Jeśli inaczej rozwiązaliście ten problem u siebie to piszcie

Pozdrawiam

Osobiście nie lubię dynamicznego wywoływania metod, dlatego takie rozwiązanie nie przypadło mi do gustu.

Pierwszy pomysł, to zbieranie danych w kontenerze, który następnie przekazujemy do akcji w momencie jej wywołania.

Możesz również utworzyć metodę, która posłuży do zbierania parametrów...


Tak czy inaczej, wszystkie te podejścia wymagają pewnego rodzaju mapowania parametrów z URL do ich nazw (przy Twoim podejściu - kolejności) w akcji. Najlepiej zrobić to w Routerze. Tutaj masz pełną dowolność...

No właśnie mam teraz coś w rodzaju kontenera. Router odczytuje parametry i przekazuje do dispathera w postaci tablicy z parametrami, która następnie jest przesyłana do klasy akcji. W klasie akcji dostępna jest metoda getParam($name); która zwraca pożądany parametr. Obiekt akcji tworzy klasa Dispather:

[PHP] pobierz, plaintext 
<?php
// ... dispather
require_once($url_na_podstawie_get);
$_action = new $_action;
$action->_init($HttpContext, $parameters); // ta metoda zapisuje parametry i context w klasie oraz wywołuje metode Execute
// ... end dispather
 
// metoda _init w klasie akcji:
 
final public function _init(HttpContext $context, $params)
{
$this->context = $context;
$this->parameters = $params;
$this->execute();
}
// end
?>
[PHP] pobierz, plaintext 

No i teraz chciałem w metodzie _init() np sprawdzić czy wszystkie parametry zostały podane w przeciwnym razie zostaniemy przekierowani do innej akcji, ale nie wiem jak to poprawnie rozwiązać bo trzeba gdzieś trzymać wymagne parametry, narazie myślałem nad trzymaniem ich w bazie ale to jest raczej niewydajny sposób. Kolejnym moze być utworzenie w klasie akcji składowej $required_params = array('param1', 'paramn'); gdzie będą wypisane wszystkie parametry potrzebne do wywołania akcji.

Co o tym sądzicie ?

Może dobrym pomysłem byłoby poprostu sprawdzanie w każdej akcji osobno czy ma odpowiednie parametry
Możesz dla każdej akcji zrobić "rodzica" który miałby, coś takiego (pomysł zaczerpnięty z Zend_Config):

[PHP] pobierz, plaintext 
<?php
1: class Zend_Config implements Countable, Iterator
 2: {
 3: 
 4: protected $_allowModifications;
 5: protected $_iterationPointerValid;
 6: protected $_data;
 7: 
 8: public function __construct($array, $allowModifications = false)
 9: {
10: $this->_allowModifications = $allowModifications;
11: foreach ($array as $key => $value) {
12: if ($this->_isValidKeyName($key)) {
13: if (is_array($value)) {
14: $this->_data[$key] = new Zend_Config($value, $allowModifications);
15: } else {
16: $this->_data[$key] = $value;
17: }
18: } else {
19: throw new Zend_Config_Exception("Invalid key: $key");
20: }
21: }
22: }
23: 
24: public function __get($name)
25: {
26: $result = null;
27: if (isset($this->_data[$name])) {
28: $result = $this->_data[$name];
29: }
30: return $result;
31: }
32: 
33: public function __set($name, $value)
34: {
35: if ($this->_allowModifications) {
36: if (is_array($value)) {
37: $this->_data[$name] = new Zend_Config($value, true);
38: } else {
39: $this->_data[$name] = $value;
40: }
41: } else {
42: throw new Zend_Config_Exception('Zend_Config is read only');
43: }
44: }
45: }
?>
[PHP] pobierz, plaintext 

potem sprawdzasz, w pliku konkretnej akcji:

[PHP] pobierz, plaintext 
<?php
class Error403Action extends Zend_Config //oczywiście u Ciebie Class Error403Action extends Action 
{
public __construct()
{
if (!isset($this->twojparamter)) throw new NoParamActionExeption();
}
}
?>
[PHP] pobierz, plaintext 

Oczywiście we FrontControlerze łapiesz NoParamActionException i kierujesz usera na odpowiednią stronę.