Problème avec condition if + is_int

Bonjour,

J'ai un problème avec ma condition if, je fais ça :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15

$forumSujet=htmlentities(mysql_real_escape_string($_GET['forumSujet'])); if ( $forumSujet>=1 AND $forumSujet<=11 AND is_int($forumSujet) ) { instruction ;} else { echo'flute' ;}

---

$_GET['forumSujet'] est récupéré par l'URL et est un nombre entier, par exemple 5.

Si j'enlève is_int($forumSujet) de la condition if, alors l'instruction est effectuée, mais si je le laisse et je dois le laisser, alors c'est l'instruction else{} qui est effectuée.

Qu'est-ce que je n'ai pas pigé avec is_int, s'il-vous plait ?

regarde ici cela devrait repondre a ta question

Salut Bebel,

J'ai lu le lien mais honnêtement je ne vois pas où il y a une erreur dans mon code, à priori ils utilisent is_int() de la même manière dans leurs exemple (si ce n'est un qui place la fonction comme suit (is_int($var)) à l'intérieur du if(), mais j'ai essayé ça ne change rien au problème :?

Dans le lien donné par Bebel, on peut lire :

Note : Pour tester si une variable est un nombre ou une chaîne numérique (comme les entrées de formulaire, qui sont toujours des chaînes), vous devez utiliser la fonction is_numeric().

Oui mais je me suis trompé je veux tester si la variable est un nombre entier, pas seulement un nombre.

Tu peux aller voir du côté des conversions de chaînes

Sinon, baguette magique RegExp :

Code:

---

1 2 3 4

if(ereg("^[0-9]+$", $forumSujet) { ... }

---

Bon d'accord je vais me débrouiller, mais quand même y'a un truc qui m'échappe.

is_int($var) est censé tester si $var est un nombre entier. Comme la fonction est un bool ça me renvoit true ou false le cas échéant.
Normalement c'est simplissime ça devrait fonctionner.

Avec la regexp, ça fonctionnera.

Sinon :

Citation:

---

is_int -- Détermine si une variable est de type nombre entier

---

type nombre entier, et toi tu récupères un type string.
Du coup is_int() te retourne false et l'instruction passe directement au else.

Qu'est-ce que tu veux dire quand tu dis que je récupère un string (sans jeu de mot douteux hein) ?

Normalement ma variable a pour valeur un entier numérique. Je devrai faire is_int(5) par exemple ?
J'ai rien pigé, j'ai un truc à comprendre je crois.

1) parentheses autour de CHAQUE condition dans le if (if ((cond1)&&(cond2)&&(cond3)) {...})
2) && au lieu de AND

Permet moi aussi de te dire que ton code est illisible je trouve... tu choisis les conventions que tu veux, mais en général on fait pas comme ca...

Admettons que mon code soit illisible, j'ai pas la prétention d'être un pro.
Toi tu ferais comme ça ?

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17

$forumSujet=htmlentities(mysql_real_escape_string($_GET['forumSujet'])); if ( ($forumSujet>=1 ) && ($forumSujet<=11 ) && (is_int($forumSujet))   ) { instruction ;} else { echo'flute' ;}

---

Je trouve pas ça plus lisible mais bon. Pourquoi && à la place de AND et pourquoi placer chaque condition entre (), s'il te plait ?

**edit** en tout cas même écrit comme ça ça foire quand même.

Salut,

$var = "5";
$var est un string.
is_int($var) retourne false.

$var = 5;
$var est un int.
is_int($var) retourne true.

Bye

Citation:

---

Envoyé par Djakisback

Salut,

$var = "5";
$var est un string.
is_int($var) retourne false.

$var = 5;
$var est un int.
is_int($var) retourne true.

Bye

---

Bon là c'est clair, maintenant moi je récupère 5 depuis l'url, donc dans $_GET['$var'].
Ensuite par commodité je fais $var=$_GET['$var'] avec htmlentities et mysql_real_escape_string au passage.

Comment se fait-il que mon 5 soit devenu un string entretemps ?

Tout ce que tu passes en get ou post est converti en string sauf si tu utilises sérialize() il me semble.

Citation:

---

Envoyé par JackBeauregard

Comment se fait-il que mon 5 soit devenu un string entretemps ?

---

string mysql_real_escape_string ( string unescaped_string [, resource link_identifier] )

string htmlentities ( string string [, int quote_style [, string charset]] )

Pour ça !

Pour revenir au && a la place du AND : Quelle est la difference ? L'utilité ? Car sinon je m'y met au && :)

Citation:

---

Envoyé par JackBeauregard

Toi tu ferais comme ça ?

---

Non, comme ca :

Code:

---

1 2 3 4 5 6

$forumSujet=htmlentities(mysql_real_escape_string($_GET['forumSujet'])); if (($forumSujet>=1 ) && ($forumSujet<=11 ) && (is_int($forumSujet))) { instruction; } else { echo'flute'; }

---

ou :

Code:

---

1 2 3 4 5 6 7 8 9

$forumSujet=htmlentities(mysql_real_escape_string($_GET['forumSujet'])); if ( ($forumSujet>=1 ) && ($forumSujet<=11 ) && (is_int($forumSujet))) {   instruction; } else { echo'flute'; }

---

maintenant, oui, il est normal que is_int renvoie FALSE car ce n'est pas de type INT, c'est de type STRING et ca contient un INT. Donc is_numeric est plus approprié. Pour t'en convaincre, il suffit d'ajouter

Code:

---

var_dump($forumSujet);

---

et tu devrais obtenir qqchose comme String(1) : "1"
et non integer : 1

Si tu veux vraiment interdire les valeurs décimales, tu peux convertir d'abord et faire la condition ensuite :

Code:

---

1 2 3 4 5 6 7 8 9 10

if (is_numeric($forumSujet)) { $forumSujet+=0; // ca va convertir en nombre // $forumSujet = int_val($forumSujet); // autre possibilité // $forumSujet = (int)$forumSujet; // autre possibilité // $forumSujet*=1; // autre possibilité }   if (... is_int($forumSujet)...) { ... }

---

Citation:

---

Envoyé par Elwyn

Pour revenir au && a la place du AND : Quelle est la difference ? L'utilité ? Car sinon je m'y met au && :)

---

http://fr2.php.net/manual/fr/languag...ors.precedence

La différence est dans les priorités... mais si tu met des parentheses explicite, tu ne te pose pas de question sur les priorités, donc tu peux utiliser celui que tu veux...

maintenant je prefere reserver && et || aux operations logiques dans les if, et utiliser "or" dans les lignes genre mysql_... or die(...)

Quand au "and"... et bien je ne l'utilise pas ;o)

mais en fait, je suis d'accord, ca depend de chacun... je pensais qu'il s'agissait des operateurs binaires mais je me suis trompé, les operateurs binaires sont & et |