projektowanie kontrolera Opcje

[pikornecki]

Witam,

Chciałbym zaprojektować wydajny kontroler. Znalazłem taki artykół na stronie http://php.pl/Wortal/Artykuly/Proces-tworz...anie-Kontrolera, w którym switch pełni rolę takiego kontrolera:

[PHP] pobierz, plaintext 
switch ( @$_GET['action'] ) {
    case 'edit':
        include ('actions/edit.php');
    break;
    case 'post':
        include ('actions/post.php');
    break;
    case 'delete':
        include ('actions/delete.php');
    break;
    case 'default':
        include ('actions/view.php');
    break;
}
[PHP] pobierz, plaintext 

I teraz moje pytanie, chciałbym tak jak w tym przykładzie użyć switcha do obsługi menu złożonego z takich imputów:

[HTML] pobierz, plaintext 
<input class="menuButtons" type="submit" value="Załaduj obrazki" name="addImagesButton"/>
[HTML] pobierz, plaintext 

Teraz pytanie, nawet jeśli zastosuję takiego samego name(a) dla 10 przycisków jak zastosować to w switchu? Co dla mnie będzie casem?


EDIT: chciałbym dodać że chce operować POST a nie GET

Ten post edytował pikornecki 15.07.2011, 16:36:39

[tabbi]

a nie można w:

switch ( @$_POST['action'] ) {

i odpowiednio zmieniać value w inputach

albo zrobić tablice np. name=mirrors[action][value]

[CuteOne]

Dla każdego guzika będziesz robił oddzielny <form action="xxx"></form>?? Podany przez Ciebie przykład z artykUłu dotyczy bardziej linków niż submitów. Poza tym zamiast switchów wygodniej jest zrobić sobie tablicę z danymi modułów.

[PHP] pobierz, plaintext 
 
$modules = array(
   'main' => array('main.php', 1),
   'logowanie' => array('login.php', 1),
   //itd...
);
 
if(array_key_exists($_GET['action'], $modules)) {
 
   if($modules[$_GET['action']][1] == 1) {
 
        include $modules[$_GET['action']][0];
   }
}
[PHP] pobierz, plaintext 

A jeszcze wygodniejszym rozwiązaniem jest zaangażowanie do tego celu bazy danych


EDIT: fakt zamiast in_array powinno być array_key_exists dzięki fifi

Ten post edytował CuteOne 16.07.2011, 16:34:05

[Sagnitor]

Hm, twojego switch'a da się zastąpić instrukcją if, else przy okazji filtrując dane.

[PHP] pobierz, plaintext 
$sAction = $_GET['action'];  // Tutaj należy dodać filtrowanie zmiennej.
 
if(file_exists('./actions/'.$sAction.'.php'))
{
   include ('./actions/'.$sAction.'.php');
} else {
 include ('./main.php');  // W tym miejscu includujemy lub przekierowujemy na stronę główną
}
 
[PHP] pobierz, plaintext 

To są najprostsze i najbardziej prymitywne przykłady .

Pozdrawiam

[AndyPSV]

uzywam czegos bardziej skomplikowanego - [SPAM]

[PHP] pobierz, plaintext 
function loadMod($m) {
	$uri = explode(';',$m); if(!empty($uri[1])) $m = $uri[0]; $pth = DIR_MOD.strtolower($m).N.INX.PHP; $q = q('SELECT id FROM `'.PRFX.'c0'.LG.'` WHERE url = "'.$m.'" LIMIT 1'); if(n_r($q) == 0) {
		if(file_exists($pth)) include_once($pth); else { header(LOC.URL,true,301); exit; } } else include_once(DIR_MOD.strtolower('pg').N.INX.PHP);
} if(empty($_GET[0]) or substr($_GET[0],0,1) == '?') loadMod(PG_H_MOD); else { if(ctype_digit($_GET[0])) loadMod('ad'); else { if(ctype_digit(end(explode(',',$_GET[0])))) loadMod('ad'); else {
	if(substr($_GET[0],0,3) == 'buy') loadMod(substr($_GET[0],0,3)); elseif(is_dir('m/'.$_GET[0])) loadMod($_GET[0]); else {
		$q = q('SELECT id,t FROM '.PRFX.'ads WHERE uri = "'.$_GET[0].'" LIMIT 1'); if(n_r($q) > 0) { $ad = f($q); exit(header(LOC.URL._url($ad['t']).','.$ad['id'])); } else exit(header(LOC.URL));
	}
}}
[PHP] pobierz, plaintext 

kod umozliwia otworzenie wszystkiego.

Ten post edytował wookieb 16.07.2011, 16:40:05

Powód edycji: [wookieb]:

[Fifi209]

[PHP] pobierz, plaintext 
 
$modules = array(
   'main' => array('main.php', 1),
   'logowanie' => array('login.php', 1),
   //itd...
);
 
if(in_array($_GET['action'], $modules)) {
 
   if($modules[$_GET['action']][1] == 1) {
 
        include $modules[$_GET['action']][0];
   }
}
[PHP] pobierz, plaintext 

Jak dla mnie array_key_exists zamiast in_array

Hm, twojego switch'a da się zastąpić instrukcją if, else przy okazji filtrując dane.

[PHP] pobierz, plaintext 
$sAction = $_GET['action'];  // Tutaj należy dodać filtrowanie zmiennej.
 
if(file_exists('./actions/'.$sAction.'.php'))
{
   include ('./actions/'.$sAction.'.php');
} else {
 include ('./main.php');  // W tym miejscu includujemy lub przekierowujemy na stronę główną
}
 
[PHP] pobierz, plaintext 

To są najprostsze i najbardziej prymitywne przykłady .

Gdzie w tym kodzie filtracja? Jakiś ctype_ czy preg_match ukryty? A może filter_var? Nawet brak podstawy jaką jest basename

Ten post edytował Fifi209 16.07.2011, 14:20:39

[Sagnitor]

@UP

Wstawiłem kod bez filtracji i pokazałem miejsca, w których powinna sie odbywać. Być może słabo sprecyzowałem swoją wypowiedź.

[-=Peter=-]

[PHP] pobierz, plaintext 
function loadMod($m) {
	$uri = explode(';',$m); if(!empty($uri[1])) $m = $uri[0]; $pth = DIR_MOD.strtolower($m).N.INX.PHP; $q = q('SELECT id FROM `'.PRFX.'c0'.LG.'` WHERE url = "'.$m.'" LIMIT 1'); if(n_r($q) == 0) {
		if(file_exists($pth)) include_once($pth); else { header(LOC.URL,true,301); exit; } } else include_once(DIR_MOD.strtolower('pg').N.INX.PHP);
} if(empty($_GET[0]) or substr($_GET[0],0,1) == '?') loadMod(PG_H_MOD); else { if(ctype_digit($_GET[0])) loadMod('ad'); else { if(ctype_digit(end(explode(',',$_GET[0])))) loadMod('ad'); else {
	if(substr($_GET[0],0,3) == 'buy') loadMod(substr($_GET[0],0,3)); elseif(is_dir('m/'.$_GET[0])) loadMod($_GET[0]); else {
		$q = q('SELECT id,t FROM '.PRFX.'ads WHERE uri = "'.$_GET[0].'" LIMIT 1'); if(n_r($q) > 0) { $ad = f($q); exit(header(LOC.URL._url($ad['t']).','.$ad['id'])); } else exit(header(LOC.URL));
	}
}}
[PHP] pobierz, plaintext 

kod umozliwia otworzenie wszystkiego.

Nie zdziwię się jeśli to spaghetti otwiera też puszkę konserwową.

[Fifi209]

Wstawiłem kod bez filtracji i pokazałem miejsca, w których powinna sie odbywać.

Trzeba było podsunąć pomysł na filtrację skoro wiesz o co chodzi.

Równie dobrze możesz dać kluczyki do auta 10-latkowi i powiedzieć, że może jechać gdzie chce.
Dałeś mu kod, jeszcze biedak użyje w takiej wersji bo może nie wie jak przefiltrować? 10 razy nic się nie stanie, za 11 ktoś wykorzysta brak filtracji.