Discussion :

[sam01]

Bonjour,

j'ai un code ou je veux tester des champs.
Si ou moins un des champs est rempli, je teste si il y a au moins un champs vide.
Si c'est le cas, j'envoie un messgae d'erreur. voici mon code

Code :

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

1
2
3
4
5
6
7
8
9
10
 
 
if((!empty($nomf)) or (!empty($prenomf)) or (!empty($ad_ligne1f)) or (!empty($ad_ligne2f)) or (!empty($ad_cpf)) or (!empty($ad_villef)))
									{
										if((empty($nomf)) or (empty($prenomf)) or (empty($ad_ligne1f)) or (empty($ad_ligne2f)) or (empty($ad_cpf)) or (empty($ad_villef)))
										{ 
											$erreur[] = '<center><font color="#FF0000">Si l\'adresse de facturation est différente, il faut repmlir tous les champs</font></center>';
											echo 'erreur01';
										}
									}

le problème c'est que ce code ne focntionne pas...

Quand je rempli un seul champ, il devrai m'envoyer un message d'erreur maisl il ne le fait...

[Biglo]

Salut,

Tu as un problème de parenthésage dans ton deuxième if. Et utilise plutôt || que "or". Plus optimisé et le "or" est surtout utilisé pour des gestions d'erreurs.

Code :

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

if(empty($nomf) || empty($prenomf) || empty($ad_ligne1f) || empty($ad_ligne2f) || empty($ad_cpf) || empty($ad_villef))

[Amara]

Je crois qu'il manque aussi une parenthèse ouvrante juste après le premier if.

Mais il est pas un peu bizarre cet algo, le premier if teste si il y a un champ rempli et le second teste si un champ est vide, c'est bien ce que tu expliques mais ça paraît bizarre comme démarche...

[KneXtasY]

Effectivement, c'est trés bizarre ...

Pour vérifier qu'au moins un des champs a été rempli et donc que l'utilisateur est passé par le formulaire pour accèder à cette page, je te conseil d'utiliser un champ de type hidden.

Si la variable correspondant au champ hidden existe alors à ce moment la, tu peux effectuer la vérification sur l'ensemble des champs.

[Msieurduss]

Pour un code moins bizarre je te conseille un algo du style :

Code :

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

1
2
3
4
5
6
SI "champ 1 vide" OU "champ 2 vide" OU ... {
         Afficher un message d'erreur
} SINON {
         Code si tout va bien...
}

Ca sera moins lourd non ?

[Fladnag]

non, car il cherche a verifier en fait que :
* soit tout les champs sont remplis
* soit aucun champs n'est remplis.

Tu pourrais donc faire un code avec du xor ;o)

Mais restons simple ;o)

Code :

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

1
2
3
4
5
$tab=array($champ1, $champ2, $champ3, $champ4);
$tab=array_unique(array_map('empty', $tab));
if (count($tab) > 1) {
// message d'erreur : il y a a la fois un champ remplis et un champ vide ;o)
}

comment ca pas lisible ? pourtant ca devrait marcher ;o))

[Msieurduss]

non, car il cherche a verifier en fait que :
* soit tout les champs sont remplis
* soit aucun champs n'est remplis.

Ben j'ai bien compris, mais quel intérêt ?
S'il n'y a aucun champ vide, alors tous les champs sont remplis...

[Fladnag]

Ben j'ai bien compris, mais quel intérêt ?
S'il n'y a aucun champ vide, alors tous les champs sont remplis...

Si tu avait lu le message d'erreur qu'il affiche dans son premier message tu aurais la réponse :

Il s'agit d'un formulaire de modification.
Donc tu as des champs disponibles pour rentrer une nouvelle adresse.
=> Soit aucun champ n'est rempli : dans ce cas là on garde l'ancienne adresse
=> Soit au moins un champ est rempli : dans ce cas là c'est un changement d'adresse et il faut que tout les champs soient remplis

[KneXtasY]

Je ne vois pas bien l'interet ...

Autant faire un formulaire de modification qui reprends les valeurs dans les champs avec l'attribut value et faire un update dans tout les cas (modification ou non).

En réflechissant, ça ne sert que si on veut afficher un message à l'utilisateur aprés la modification.

Mais bon aprés, c'est plutôt de la conception que du PHP.

[sam01]

désolé je suis arrivé un peu tard.
Fladnag à tout à fait compris mon problème.
Je vais tester ce que tu m'as proposé.

merci