problem z parametrami public static Opcje

[zephyr7]

Próbuję napisać coś na wzór frameworka,
mam funkcję, która wywołuje:

$uri[0]::$uri[1](); //np. pages::view();

jak przekazywać dodatkowe parametry?

Jeśli mój url wygląda tak: pages/view/1/2

funkcja powinna wywołać:

$uri[0]::$uri[1]($uri[2],$uri[3]);

jak poradzić sobie ze zmienną liczbą parametrów?


Do głowy przychodzi mi tylko coś takiego:


$params="";
for ($q=2;$q<count($uri)-1;$q++){
$params.='$uri['.$q.'],';
}
//usuwa przecinek.... no comment....
$params=strip_tags(substr($params,0,-1));
eval ('$uri[0]::$uri[1]('.$params.');');


będę wdzięczny za bezlitosną krytykę i pomysły, jak można napisać to po ludzku....

[thek]

Podziel string po stosowanym rozdzielaczu w adresie, czyli / a co do zmiennej liczby parametrów funkcji, to jak widzę nie uważałeś podczas czytania manuala (IMG:style_emoticons/default/winksmiley.jpg) Lepiej obadaj func_get_args
Dzięki temu możesz ręcznie odbierać tyle parametrów w funkcji ile chcesz. Taki nieco kulawy rodzaj przeciążania funkcji w PHP mających tę samą nazwę, ale inną liczbę parametrów.

Powód edycji: [thek]: @bottom: A ja Ci "synu" błogosławię żeś wybrał ścieżkę dobra. Idź i nie grzesz więcej :D

[zephyr7]

Faktycznie! Baran ze mnie!

Przecież gdzieś to widziałem, w którymś z tutoriali!

(już biorę się za czytanie)

[Crozin]

Skorzystaj z ReflectionMethod. Swoją drogą nie powinieneś najprawdopodobniej korzystać tutaj z metod statycznych.

[zephyr7]

No, zagwozdka normalnie...

chciałbym tylko wywołać metodę z tyloma parametrami, ile mam w URI...

func_get_args z tego co wiem, zwraca już istniejące argumenty.

Kiedy mam w URL 2 parametry (pomijając uri[0] i [1], czyli kontroller i metodę)

chciałbym wywołać funkcję podając dwa parametry, np. $uri[0]::$uri[1]($uri[2],$uri[3]), co daje, np. page::view(2,3)

wtedy w kontrolerze mógłbym odwołać się prosto, np. function index($id,$page)...

chodzi tylko o tą przeklętą zmieniającą się liczbę argumentów
$uri[0]::$uri[1]($uri[2],$uri[3])

jeśli będę miał 1 parametr, np. pages/view/2
a funckja wywołana będzie jak wyżej, powstanie błąd "undefined index"

Mógłbym ewentualnie (co chyba jest jeszcze gorszym pomysłem)

podawać ciąg parametrów i zerować te, których nie ma w URL, np. pages/view/2

for($q=2;$q<count($uri)-1;$q++){
if (!isset($uri[$q]))$uri[$q]=0;
}

$uri[0]::$uri[1]($uri[2],$uri[3]....$uri[20])

kretyńskie, prawda?

a swoją drogą, proste i bez kombinowania.

Przecież nikt nie potrzebuje więcej, niż np. 5-10 parametrów w url

[thek]

Pokombinuj z func_num_args i switch. Skoro jakaś metoda obsługuje ściśle określoną liczbę parametrów maksymalnie, to brakujące musisz ustawić na elementy "neutralne" (czyli albo false, albo 0 ) by wychwycić ich brak. Jak?
Przypuśćmy możesz w funkcji maksymalnie obsłużyć 5 parametrów. Jeśli będzie ich mniejsza ilość - uzupełniasz je poprzez false, jeśli równa - ok, jeśli większa - ignorujesz te powyżej. Jak by to mogło wyglądać?

[PHP] pobierz, plaintext 
function test() {
  $args = func_get_args();
  switch( func_num_args() ) {
    case 0:
      $args[0] = false;
    case 1:
      $args[1] = false;
    case 2:
      $args[2] = false;      
    case 3:
      $args[3] = false;
    case 4:
      $args[4] = false;
    case 5:
      break;
    default:
      $args = array_slice($args, 0, 5);
  }
  //tu robisz co chcesz już na tablicy $args
}
[PHP] pobierz, plaintext 

Switch jest tu najważniejszy, ponieważ on modyfikuje tak listę parametrów, by uzupełnić brakujące parametry poprzez false. Zwróć uwagę na brak break aż do case 5. To ważne, ponieważ dla 5 parametrów jedynie nastąpi wyjście ze switcha. A co jeśli jest parametrów mniej, jak choćby 3? Ano tabela $args uzupełni $args[3] (4 parametr) na false i przejdzie do case 4 (dlatego brak break!) i uzupełni $args[4](5 parametr) na false oraz przejdzie do case 5 i dopiero tu wyskoczy ze switcha (IMG:style_emoticons/default/winksmiley.jpg) Defaultem jedynie potraktowałem jeśli masz więcej parametrów, bo wtedy nastąpi obcięcie ich do wymaganych 5 (IMG:style_emoticons/default/smile.gif) Zrobiłem coś więc o charakterze "normalizatora" liczby parametrów.

PS: Oczywiście uzycie switch ma sens jedynie dla małej liczby akceptowanych parametrów. Przy dużej można użyć do tego celu jak powyżej -> pętli.

[Crozin]

A patrzyłeś w ogóle na mój post? To podpowiem: metoda invokeArgs()

[thek]

Crozin... na nic fleksja się nie zda, skoro błąd będzie na poziomie samej funkcji wywołanej w refleksji. Skoro będzie ona przyjmowała kilka parametrów, a nie dostanie nawet jednego to co będzie? Nie widziałem by refleksja to obsługiwała poprzez uzupełnienie nie istniejących jakoś (IMG:style_emoticons/default/winksmiley.jpg) Ona też dostaje określoną listę parametrów i musi ona być kompletna. Jedyne co możesz zrobić to za jej pomocą pobrać liczbę argumentów funkcji i uzupełnić brakujące false'ami. Moim zdaniem dobre jest to do jakiegoś systemu kontroli, wbicie w helper czy cuś, ale na szybko można to prosto rozwiązać poprzez wariację na switch czy w pętli na zasadzie:

[PHP] pobierz, plaintext 
$args = func_get_args();
if( func_num_args() < 5 ) {
  for($i=func_num_args(); $i < 5; $i++) {
    $args[$i] = false;
  }
} else if( func_num_args() > 5 ) {
  $args = array_slice($args, 0, 5);
}
[PHP] pobierz, plaintext 

Można też zamiast for użyć choćby

[PHP] pobierz, plaintext 
$args = array_merge( $args, array_fill( func_num_args(), 5-func_num_args(), false ));
[PHP] pobierz, plaintext 

Cudować można na wiele sposobów (IMG:style_emoticons/default/winksmiley.jpg)

PS: Zapomniałem dodać, że 5 to oczywiście wartość jakaś dowolna, która określa liczbę parametrów funkcji.

[Crozin]

Skoro będzie ona przyjmowała kilka parametrów, a nie dostanie nawet jednego to co będzie?

To błąd będzie. A jeżeli jakiś argument ma być opcjonalny, to... niech będzie opcjonalny:

[PHP] pobierz, plaintext 
public function doSth($a = null)
[PHP] pobierz, plaintext 

Swoją drogą, jeżeli już ma być coś uzupełniane wartościami domyślnymi (sic!) to powinny to być NULLe, nie FALSE.

[Athlan]

Ja tylko się zastanawiam, dlaczego nie użyjecie do przekazania tych parametrów jednej tablicy. Rozwiązanie jest o tyle elastyczne, że można w przyszłości oddzielić argument przekazujący parametry url'a od dodatkowych argumentów (np. flag, etc). Poza tym to nieco szybszy sposób, łatwo się na tym operuje, dane masz takie same, jak w przypadku func_get_args().

[rodia]

[PHP] pobierz, plaintext 
function hello()
{
	$num = func_num_args();
	$args = func_get_args();
 
	foreach ($args as $arg)
		echo $arg.'<br>';
}
[PHP] pobierz, plaintext 

Powyższe rozwiązanie odbiera zmienną ilość argumentów, jednak problemem jest wywołanie tej funkcji z różną ilością argumentów - bo ciężko jest to już zrobić z automatu.

Czy nie lepiej mając URL w postaci jak autor pisał:

[PHP] pobierz, plaintext 
 
$url = 'a/b/c/d';
[PHP] pobierz, plaintext 

zamienić to na tablicę i wywolać funkcję przetwarzającą URL?

[PHP] pobierz, plaintext 
executeURL(explode('/', $url));
[PHP] pobierz, plaintext 

a funkcja ta przyjmowałaby zawsze tylko jeden argument, którym byłaby tablica o dowolnej ilości parametrów z URLa

[PHP] pobierz, plaintext 
 
function executeURL($args)
{
	foreach ($args as $arg)
		echo $arg.'<br>';
}
 
[PHP] pobierz, plaintext 

Pozdrawiam, R.


Ten post edytował rodia 27.12.2010, 16:11:16

[thek]

Podejść na rozwiązane tego problemu jest masę. Najwygodniejsza jest tablica. Potem kombinowanie z func_get_args. Co do NULL bądź innych wartości to zależne to od tego co funkcja robić ma. Ja chciałem tylko zwrócić Ci Crozin uwagę, że wbijanie tam refleksji pasuje jak pięść do oka (IMG:style_emoticons/default/winksmiley.jpg) Nie po to ten mechanizm wymyślono by kontrolował liczbę parametrów funkcji czy tego typu "manewry". Są prostsze rozwiązania tego problemu. Do problemu zmiennej liczby parametrów tak naprawdę tylko 2 sposoby są sensowym rozwiązaniami: func_***_arg(s) i tablice/obiekty. Przy czym to drugie jest elastyczniejsze, gdyż możemy jeszcze się bawić.