Verification syntaxe fonction javascript

Version imprimable

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:

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;
  }
 
}

12/09/2006, 14h46
SpaceFrog

elseif => c'est du javascript ça ?

renseigne toi plutot sur le switch ...

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

@calitom :

Code:

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:

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.

12/09/2006, 15h23
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:

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);
12/09/2006, 15h40
SpaceFrog

je persiste à dire qu'un switch serait beaucoup plus clair ...
12/09/2006, 15h45
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....

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:

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>

12/09/2006, 15h58
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.
12/09/2006, 16h02
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!