Probleme Php et condition if
Fermé
Tydher
Messages postés
123
Date d'inscription
vendredi 5 juin 2009
Statut
Membre
Dernière intervention
17 mai 2011
-
8 juin 2009 à 11:19
Yoan Messages postés 11795 Date d'inscription mardi 1 février 2005 Statut Modérateur Dernière intervention 10 décembre 2023 - 8 juin 2009 à 11:33
Yoan Messages postés 11795 Date d'inscription mardi 1 février 2005 Statut Modérateur Dernière intervention 10 décembre 2023 - 8 juin 2009 à 11:33
A voir également:
- Probleme Php et condition if
- Excel cellule couleur si condition texte - Guide
- Easy php - Télécharger - Divers Web & Internet
- Condition et excel - Guide
- Excel copier une cellule si condition ✓ - Forum Excel
- Excel colorer une cellule sous condition d'une autre cellule ✓ - Forum Excel
2 réponses
easycomp
Messages postés
86
Date d'inscription
samedi 6 juin 2009
Statut
Membre
Dernière intervention
4 juillet 2009
30
8 juin 2009 à 11:25
8 juin 2009 à 11:25
Salut,
Normal pour tes tests tu utilise l'opérateur d'affectation (=) au lieu de l'opérateur de comparaison (==) :)
De plus tu écris:
if (is_null($sql8))
Qui ne sera jamais null puisque c'est ta requête et pas le retour de ta requête.
Ce serait plutôt
if(is_null($req9))
Ou encore mieux
if(mysql_num_rows($req9) == 0)
Bon courage.
Normal pour tes tests tu utilise l'opérateur d'affectation (=) au lieu de l'opérateur de comparaison (==) :)
De plus tu écris:
if (is_null($sql8))
Qui ne sera jamais null puisque c'est ta requête et pas le retour de ta requête.
Ce serait plutôt
if(is_null($req9))
Ou encore mieux
if(mysql_num_rows($req9) == 0)
Bon courage.
Yoan
Messages postés
11795
Date d'inscription
mardi 1 février 2005
Statut
Modérateur
Dernière intervention
10 décembre 2023
2 330
8 juin 2009 à 11:33
8 juin 2009 à 11:33
Houlala, c'est un peu bricolé tout ça, hein ? ^^
Le principal problème est que la comparaison n'est pas = mais ==
Donc ça c'est à la limite pas bien violent à corriger, mais le reste je trouve ça pas très propre.
En réaction à ça : is_null($sql8)
au lieu de tester ça regarde plutôt ce qu'a renvoyé mysql ( if(mysql_num_rows($req8)>0) ), parce que si jamais ta requête n'est pas nulle mais qu'elle ne renvoie aucun n-uplet, tu va rentrer dans le if quand-même.
Ensuite, n'utilise pas 'NULL' et 'PAS NULL'. C'est comme si tu utilisais $nom='JEAN-PIERRE' ou 'PAULETTE', ça marche aussi mais c'est pas vraiment parlant, utilise plutôt un booléen : $requete_valide=TRUE ou FALSE
Ensuite, je ne vois pas l'intérêt de tester $test='NULL' ou 'PAS NULL' puisque tu fais la même chose à l'intérieur. Si c'est parce que tu n'as pas fini d'écrire ton code, pense à sortir le code commun aux deux conditions. D'ailleurs plutôt que deux if() ce serait plutôt ici un if() else ou if() elseif()
EDIT : Donc c'est une redite :)
Le principal problème est que la comparaison n'est pas = mais ==
Donc ça c'est à la limite pas bien violent à corriger, mais le reste je trouve ça pas très propre.
En réaction à ça : is_null($sql8)
au lieu de tester ça regarde plutôt ce qu'a renvoyé mysql ( if(mysql_num_rows($req8)>0) ), parce que si jamais ta requête n'est pas nulle mais qu'elle ne renvoie aucun n-uplet, tu va rentrer dans le if quand-même.
Ensuite, n'utilise pas 'NULL' et 'PAS NULL'. C'est comme si tu utilisais $nom='JEAN-PIERRE' ou 'PAULETTE', ça marche aussi mais c'est pas vraiment parlant, utilise plutôt un booléen : $requete_valide=TRUE ou FALSE
Ensuite, je ne vois pas l'intérêt de tester $test='NULL' ou 'PAS NULL' puisque tu fais la même chose à l'intérieur. Si c'est parce que tu n'as pas fini d'écrire ton code, pense à sortir le code commun aux deux conditions. D'ailleurs plutôt que deux if() ce serait plutôt ici un if() else ou if() elseif()
EDIT : Donc c'est une redite :)