aide debug fonction

**calitom** · 25/09/2008, 16h53

Bonjour,

J'ai fait une fonction js, et je n'arrive pas a savoir ou est mon erreur...
En fait elle teste si les valeurs de deux menus déroulants (taille et couleur) ont été choisies...

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
 
function valider()
{
    if(valider0()==false){return false;}
    if(valider1()==false){return false;}
    if(valider2()==false){return false;}
    return true;
}
 
function valider0()
{
 if(document.getElementById('taille0').value == "Taille non choisie" && document.getElementById('couleur0').value == "Couleur non choisie")
  {
	// sinon on affiche un message
    alert("Veuillez choisir la taille et la couleur du produit dans les menus déroulants 0!");
    // et on indique de ne pas envoyer le formulaire
    return false;
  }
  else if((document.getElementById('taille0').value != "Taille non choisie") && (document.getElementById('couleur0').value == "Couleur non choisie"))
  {
  	// sinon on affiche un message
    alert("Veuillez choisir la couleur du produit dans le menu déroulant 0!");
    // et on indique de ne pas envoyer le formulaire
    return false;
  }
  else if(document.getElementById('taille0').value == "Taille non choisie" && document.getElementById('couleur0').value != "Couleur non choisie")
  {
  	// sinon on affiche un message
    alert("Veuillez choisir la taille du produit dans le menu déroulant 0!");
    // et on indique de ne pas envoyer le formulaire
    return false;
  }
  else
  {
    // les données sont ok, on peut envoyer le formulaire
  }
}

valider1 et valider2 sont exactement pareilles que valider0...

En fait je cherche a faire des lots de produits...
Donc valider0 correspond au menu deroulant du produit0, valider1 au menu deroulant du produit1...

La fonction valider est appelée onsubmit de mon formulaire...

Mon problème, c'est que la fonction bloque l'envoi du formulaire pour le produit 0 si il n'y a pas de taille ou de couleur choisie, mais pour le produit 1, le formulaire s'envoi meme si je n'ai pas choisi de taille ou de couleur...

**E.Bzz** · 25/09/2008, 17h03

Bonjour,
2 questions :

Code :

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

1
2
3
4
  else
{
 // les données sont ok, on peut envoyer le formulaire
}

OK, mais se termine-t-elle bien par un "return true;" ?

Comment as-tu coder le onsubmit ? On peut voir le tag <form> ?

Et une suggestion :

Code :

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

1
2
3
4
function valider()
{
    return (valider0() && valider1() && valider2());
}

A+

**calitom** · 25/09/2008, 17h28

Voici le form:

Code :

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

1
2
 
<form name="form" method="post" action="/cartsys/add_moteur_3.php" onsubmit="return valider();">

J'ai essayé les modif que tu m'as conseillé, ca marche toujours pas...

**E.Bzz** · 25/09/2008, 17h33

Envoyé par calitom

J'ai essayé les modif que tu m'as conseillé, ca marche toujours pas...

Ce n'était pas une correction ...

Par contre, tu n'as pas répondu aux 2 questions : donne le code de valider0() en entier ...

A+

**calitom** · 25/09/2008, 17h41

Oui j'ai mis return true...

En fait la fonction valider et avant une boucle php, et valider0, 1... sont dans une boucle...

En gros la boucle correspond à une boucle sur le nombre de produits du lot...

Donc je fais une fonction valider$idboucle pour chaque produit, de maniere a gerer la taille et la couleur de chaque prod...

Donc la j'ai mis une version simplifiée de ma fonction, car sinon personne aurait la motivation de lire tout mon code...

**calitom** · 25/09/2008, 17h46

C'est bon, en fait il y avait un produit ou il n'y avait que une taille, et j'avais oublié de mettre un input type hidden avec le nom de la taille pour gérer le cas où il n'y avait qu'une couleur a choisir...