cs_garfield90
Messages postés388Date d'inscriptionlundi 7 juillet 2003StatutWebmasterDernière intervention10 février 2009 28 févr. 2007 à 17:54
Je ferais plutot
<?php
// permet de définir l'action si celle ci existe
$action = isset($_GET['action']) ? $_GET['action'] : 'erreur' ;
switch ( $action ){
case 'action1' :
case 'action2' :
case 'action3' :
$url = 'url/vers/'.$action1.'/index.php';
break;
default :
$url = 'url/vers/erreur.php';
break;
}
header('Location: '.$url);
die(); // juste pour être sûr
?>
Je trouve ca plus clair que t'es if a répétition si tu as x action possible.
tu tappes moins de code ( tu fais un seul header)
Erreur :
tu ne fais pas de vérification sur l'existence ou non de $_GET['action']
un echo('') inutile voir erroné car si tu envois des données avant ton header, celui ci ne fonctionnera pas
J'utiliserai pas un header pour ca mais plutot un include ou assimilé. (redirection inutile a mon gout) ou je ferai ca via URL Rewriting (google est ton ami si tu connais pas)
28 févr. 2007 à 17:54
<?php
// permet de définir l'action si celle ci existe
$action = isset($_GET['action']) ? $_GET['action'] : 'erreur' ;
switch ( $action ){
case 'action1' :
case 'action2' :
case 'action3' :
$url = 'url/vers/'.$action1.'/index.php';
break;
default :
$url = 'url/vers/erreur.php';
break;
}
header('Location: '.$url);
die(); // juste pour être sûr
?>
Je trouve ca plus clair que t'es if a répétition si tu as x action possible.
tu tappes moins de code ( tu fais un seul header)
Erreur :
tu ne fais pas de vérification sur l'existence ou non de $_GET['action']
un echo('') inutile voir erroné car si tu envois des données avant ton header, celui ci ne fonctionnera pas
J'utiliserai pas un header pour ca mais plutot un include ou assimilé. (redirection inutile a mon gout) ou je ferai ca via URL Rewriting (google est ton ami si tu connais pas)
Je pense pas avoir dis de connerie .