cs_garfield90
Messages postés388Date d'inscriptionlundi 7 juillet 2003StatutWebmasterDernière intervention10 février 2009 28 févr. 2007 à 18:26
J'ai copier-coller mon com de l'autre code, car tout le monde répond ici.
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)
Je pense pas avoir dis trop de connerie .
Tout comme les autres, aucune note
Ilsundal
Messages postés79Date d'inscriptionlundi 23 décembre 2002StatutMembreDernière intervention 1 mars 2011 28 févr. 2007 à 18:20
meme commentaire que précédemment, mais en plus ton code n'est absolument pas cohérent : Tu ne peux pas mettre un echo avec un header() --> Warning: Cannot modify header information.
Il y de plus aucun intérêt de mètre un texte avant le header (en plus c'est interdit par php) étant donné que personne ne pourra le lire
cerede2000
Messages postés71Date d'inscriptionvendredi 12 janvier 2007StatutMembreDernière intervention 6 septembre 2013 28 févr. 2007 à 15:03
Effectivement tout a fait d'accord avec BTAJV! Pas une grande invention, en plus de ca erreur possible et même pire on peut injecter une page qui n'est pu site donc on peut faire ce que l'on veut!
kankrelune
Messages postés1293Date d'inscriptionmardi 9 novembre 2004StatutMembreDernière intervention21 mai 2015 28 févr. 2007 à 15:02
Tout comme BTAJV... ça n'apporte rien et en plus il y a des erreurs... qui plus est ce n'est pas une source... ça pourrait être mis sous forme de tutoriel mais il faudrait sacrement approfondir... .. .
@ tchaOo°
BTAJV
Messages postés145Date d'inscriptionjeudi 2 février 2006StatutMembreDernière intervention17 avril 2008 28 févr. 2007 à 14:17
Et surtout le pire si tu rentre une variable action pas définie avec ton code sa renvoi une ereur donc je pense que tu peu approfondire sa
BTAJV
Messages postés145Date d'inscriptionjeudi 2 février 2006StatutMembreDernière intervention17 avril 2008 28 févr. 2007 à 14:14
Euh moi je dirai Mouai la j'ai lu et tu ma rien appri lol donc je ne note pas je preferai mais se que tu as mis la c inutile dsl
28 févr. 2007 à 18:26
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)
Je pense pas avoir dis trop de connerie .
Tout comme les autres, aucune note
28 févr. 2007 à 18:20
Il y de plus aucun intérêt de mètre un texte avant le header (en plus c'est interdit par php) étant donné que personne ne pourra le lire
28 févr. 2007 à 15:03
28 févr. 2007 à 15:02
@ tchaOo°
28 févr. 2007 à 14:17
28 févr. 2007 à 14:14