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!