Discussion :

[calitom]

Bonjour,

Je cherche a faire une fonction en javascript verifiant certaines conditions avnt la validation d'un formulaire...

Je ne m'y connait pas tr�s bien en javascript et je ne sais pas quelle est la syntaxe exacte en javascript...

Quelqu'un pourrait il v�rifier ma fonction car je crois que dans les conditions des if il faut mettre trois �gal parfois...

Voici ma fonction qui me semble bonne hormis la syntaxe car ca ne marche pas du tout:

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
function valider()
{
  // si la valeur des select taille et couleur est non choisie
  if((document.getElementById('taille').value == "Taille non choisie")&&(document.getElementById('couleur').value == "Couleur non choisie"))
  {
	// on affiche un message
    alert("Veuillez choisir la taille et la couleur du produit dans les menu déroulant correspondants!");
    // et on indique de ne pas envoyer le formulaire
    return false;
  }
  elseif((document.getElementById('taille').value != "Taille non choisie")&&(document.getElementById('couleur').value == "Couleur non choisie"))
  {
  	// on affiche un message
    alert("Veuillez choisir la couleur du produit dans le menu déroulant correspondant!");
    // et on indique de ne pas envoyer le formulaire
    return false;
  }
  elseif((document.getElementById('taille').value == "Taille non choisie")&&(document.getElementById('couleur').value != "Couleur non choisie"))
  {
  	//on affiche un message
    alert("Veuillez choisir la taille du produit dans le menu déroulant correspondant!");
    // et on indique de ne pas envoyer le formulaire
    return false;
  }
  else
  {
    // les données sont ok, on peut envoyer le formulaire
    return true;
  }
 
}

[SpaceFrog]

elseif => c'est du javascript �a ?

renseigne toi plutot sur le switch ...

[Bis�n�rs]

@SpaceFrog : Non, en effet, mais "else if", oui.

@calitom :

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
function valider(bouton){
   form = bouton.form;
   if((form.taille.value == "Taille non choisie")&&(form.couleur.value == "Couleur non choisie")){
      alert("Veuillez choisir la taille et la couleur du produit dans les menu déroulant correspondants!");
   }
   else if((form.taille.value != "Taille non choisie")&&(form.couleur.value == "Couleur non choisie")){
      alert("Veuillez choisir la couleur du produit dans le menu déroulant correspondant!");
   }
   else if((form.taille.value == "Taille non choisie")&&(form.couleur.value != "Couleur non choisie")){
      alert("Veuillez choisir la taille du produit dans le menu déroulant correspondant!");
   else{
      form.submit();
      bouton.disabled=true;
   }
}

Et ton code html :

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
5
<form action="monadresse" method="post" name="formulaire">
   <input type="text" name="taille" id="taille" value="Taille non choisie" />
   <input type="text" name="couleur" id="couleur" value="Couleur non choisie" />
   <input type="button" value="Envoyer" name="soumettre" onclick="valider(this)" />
</form>

Tu peux �videmment remplacer les input text par n'importe quel objet du formulaire.

[Hug0_76]

Salut,

tu ne devrais pas faire de condition else!

Puisque tu va vouloir passer par toutes les conditions!
Tu devrais faire simplement :

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
5
6
7
8
9
10
11
12
erreur = 0;
if(!condition1) {
   alert("erreur");
   erreur = 1;
}
 
if(!condition2) {
   alert("erreur");
   erreur = 1;
}
 
return (erreur == 0);

[SpaceFrog]

je persiste � dire qu'un switch serait beaucoup plus clair ...

[Hug0_76]

Hum un switch sur quoi aussi spacefrog??
Parce que le switch ne fonctionne qu'avec des caract�res ou des entiers!

C'est plus propre mais il faudrait attribuer un num�ro d'erreur, faire le test avec switch et afficher un msg en fonction du case....

[calitom]

C'�tait bien le elseif le probl�me...
J'ai remplac� par else if!!

Sinon BisounoursJos t'as solution n'est pas optimum car en utilisant le onclick si l'utilisateur appuie sur entr�e avec le clavier, le formulaire sera qd meme valider...

Le mieu c'est d'utiliser un onsubmit dans la balise form comme ca:

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
 
<script>
function valider()
{
  // si la valeur des select taille et couleur est non choisie
  if(document.getElementById('taille').value == "Taille non choisie" && document.getElementById('couleur').value == "Couleur non choisie")
  {
	// sinon on affiche un message
    alert("Veuillez choisir la taille et la couleur du produit dans les menu déroulant correspondants!");
    // et on indique de ne pas envoyer le formulaire
    return false;
  }
  else if(document.getElementById('taille').value != "Taille non choisie" && document.getElementById('couleur').value == "Couleur non choisie")
  {
  	// sinon on affiche un message
    alert("Veuillez choisir la couleur du produit dans le menu déroulant correspondant!");
    // et on indique de ne pas envoyer le formulaire
    return false;
  }
  else if(document.getElementById('taille').value == "Taille non choisie" && document.getElementById('couleur').value != "Couleur non choisie")
  {
  	// sinon on affiche un message
    alert("Veuillez choisir la taille du produit dans le menu déroulant correspondant!");
    // et on indique de ne pas envoyer le formulaire
    return false;
  }
  else
  {
    // les données sont ok, on peut envoyer le formulaire
    return true;
  }
 
}
</script>
 
 
<form name="form1" method="post" action="/cartsys/add.php" onsubmit="return valider()">
...
</form>

[Bis�n�rs]

@calitom : Au contraire, le formulaire ne sera pas valid� si l'utilisateur appuye sur la touche "Entr�e" de son clavier, parce que c'est un type button et non un type submit.

[calitom]

Autant pour moi j'avais pas fait attention!!

Mais dans le cas o� on utilise un type submit, il vaut mieux utiliser un onsubmit lol!

Merci pour vos r�ponses!