Discussion :

[Shinjuku]

Bonjour,

J'obtiens quelque chose de bizarre en PHP voici le code :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
/* Ligne de connexion bdd */
mysql_connect(...);
mysql_select_db(...);
 
$query = 'select * from table';
$result = mysql_query($query);
while( $row = mysql_fetch_array($result) ) {
        foreach($row as $cle => $valeur) {
                if( $cle == "champ1" ) {
                        echo "Ma cle est : " . $cle . "<br/>\n";
                }
        }
}

En sortie j'obtiens ceci :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
 
Ma cle est : 0
Ma cle est : champ1
Ma cle est : 0
Ma cle est : champ1
...

Du coup, je n'arrive pas à comprendre pourquoi quand $cle est égale à 0 il entre quand même dans le if.

Bien entendu, si je mets un « === » dans le test du if ça corrige le problème, mais je n'ai jamais eu un tel problème avec des anciennes versions de php, et je vais avoir un peu de mal à corriger les « == » en « === » sur tous mes sites !

Merci d'avance à ceux qui pourront m'aider...

[Gats]

Salut,

c'est normal si tu compares une chaîne à 0 la condition sera bonne vu que la chaîne sera transformé en 0. Une chaîne vaux zéro comparé à un entier.

Code php :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
 
if(0 == "code") echo 'hahaha';
// affiche bien hahaha

[Shinjuku]

Je viens d'essayer encore pire :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
 
if( 0 == 'test' ) {
        echo 'Bogus!!';
}

Et j'obtiens bien un « Bogus!! » en sortie.

Je suis inquiet pour mes sites, cela pourrait engendrer d'enorme faille de sécurité cette histoire.

[Gats]

Vu que tu n'as peut être pas vu ma réponse je rajouterais que c'est bien pour ça
que l'opérateur === existe comme tu l'as fait remarquer.

[Shinjuku]

Ok merci je viens de voir ça sur le site de php, je suis quand même un peu étonné.

La conversion est bizarre je pensais que cétait plutôt le 0 qui était transformé en '0', et non la chaîne transformé en 0.

Bon du coup, faut que je vérifie sur tous mes sites que ça ne pose pas de problème.

Merci pour ton aide.

[FCYPBA]

Salut,

C'est pour des cas comme cela qu'il existe des fonctions de vérifications de type, ou de transformations de type

is_numeric(),intval(), ctype_*

Voilà fait ton choix dans ces fonctions

Edit : Le === fonctionne aussi très bien en effet

[Shinjuku]

Ben ce genre de fonction on y pense pas quand on veut faire ça :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
/* Ligne de connexion bdd */
mysql_connect(...);
mysql_select_db(...);
 
$query = 'select * from table';
$result = mysql_query($query);
while( $row = mysql_fetch_array($result) ) {
        foreach($row as $cle => $valeur) {
                if( $cle == "champ1" ) {
                        echo "Ma cle est : " . $cle . "<br/>\n";
                }
        }
}

Dans le if, je m'attends à faire une comparaison de chaîne, donc pourquoi transtyper. Mais c'est là que j'aurais du faire un « === » étant donné que je m'attend à avoir 2 chaînes de caractères, vaut mieux en être sur avec une triple égale.

Je commence à en mettre, mais mes vieux sites sont pas toujours optimisés...

Le plus étrange c'est que je n'ai jamais eu le problème auparavent, c'était surement une portion de code qui n'a presque été jamais exécuté. Le hasard fait bizarrement les choses.

En tout cas, merci, j'étais un peu surpris de la réaction de php...