Redirection si $var n'est pas numérique

[vitalizo]

Bonjour

Je voudrais vérifier si une variable est un chiffre ou pas. Si oui, on traite la page, si non je voudrais rediriger vers l'index.

le code:

if(is_numeric($_GET['chiffre']))   
$chiffre=mysql_real_escape_string(htmlentities($_GET['chiffre']));
 
 $data = ma requete
$donnees1 = mysql_fetch_assoc($data); 
     	.......
         else     {
header("Location: index.php");  }

Mais quand je mets par exemple '11aa' dans $chiffre, il me renvoie une page 404.

Qqchose qui cloche dans mon code ?

 

[forty]

si tu veux vérifier que $_GET['chiffre'] ne contient que des chiffres entre 0 et 9, is_numeric n'est pas la bonne fonction car elle accepte les exposants et le séparateur décimal.
tu peux utiliser une expression régulière :

if (preg_match('/^[0-9]+$/', $_GET['chiffre']))

 

[vitalizo]

Salut,

non $chiffre peut être un nombre entre 1 et 99999999.

 

[forty]

il ne manque pas des { } dans ton if ?

 

[vitalizo]

Salut,
ici oui, je les ai zappé en recopiant le code.

Mais même en les mettant, quand je teste avec $chiffre= 11aa, ca me renvoie une 404 alors que je voudrais que ca redirige sur index.

 

[forty]

c'est pas plutôt une règle de réécriture qui ne "match" pas?

 

[spout]

Tu fais la redirection avec header() mais pas de exit(), donc le code après le header("Location: index.php") est exécuté aussi.
Le problème ne viendrais pas de là (on ne sait pas ce qu'il y a après) ?

 

[Eric 2A]

Salut,

Le fichier index.php est t-il bien présent dans le même répertoire que celui du script contenant ce code ?

Sinon, gràce à l'expression rationnelle de Forty, nous somme assurés de n'avoir que des chiffres. Ainsi, l'utilisation des fonctions mysql_escape_string et htmlentities devient superflue.

Comme spout, je pense aussi qu'il est préférable de placer la fonction exit(); après l'appel à la fonction header('Location: index.php');.

if((isset($_GET['chiffre']))and(preg_match('/^[0-9]+$/',$_GET['chiffre']))) {
	$chiffre=$_GET['chiffre'];
	$data = ma requete
	$donnees1 = mysql_fetch_assoc($data);
	// ...
} else {
	header('Location: index.php');
	exit();
}

 

[milkiway]

Ton souci c'est la 404 ou le fait que ta variable n'est pas identifiée comme étant numérique ,

 

[boby55]

avec ça plutot (/)?

header('Location: /index.php');

et garde ton test numérique il est très bien.

is_numeric