REDIRIGE PAGE SI ACTION = XXXXXX (PHP)

BTAJV Messages postés 145 Date d'inscription jeudi 2 février 2006 Statut Membre Dernière intervention 17 avril 2008 - 28 févr. 2007 à 14:14
cs_garfield90 Messages postés 388 Date d'inscription lundi 7 juillet 2003 Statut Webmaster Dernière intervention 10 février 2009 - 28 févr. 2007 à 18:26
Cette discussion concerne un article du site. Pour la consulter dans son contexte d'origine, cliquez sur le lien ci-dessous.

https://codes-sources.commentcamarche.net/source/41672-redirige-page-si-action-xxxxxx-php

cs_garfield90 Messages postés 388 Date d'inscription lundi 7 juillet 2003 Statut Webmaster Dernière intervention 10 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és 79 Date d'inscription lundi 23 décembre 2002 Statut Membre Derniè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és 71 Date d'inscription vendredi 12 janvier 2007 Statut Membre Derniè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és 1293 Date d'inscription mardi 9 novembre 2004 Statut Membre Dernière intervention 21 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és 145 Date d'inscription jeudi 2 février 2006 Statut Membre Dernière intervention 17 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és 145 Date d'inscription jeudi 2 février 2006 Statut Membre Dernière intervention 17 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
Rejoignez-nous