Discussion :

[Maze45]

Bonjour,

Je viens vous demander de l'aide vis à vis d'un petit bug que j'ai lors de la vérification d'un formulaire via JavaScript. J'ai donc une inclusion de mon JS dans la page, le formulaire contient le paramètre :

Code :

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

onSubmit="return FormTest();"

et donc le js se présente sous la forme suivante :

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
function FormTest()
{
	 if (document.form.year.value == 0)
	  {
			alert("Alert 1");
			return false;
	  }
 
	else if (document.form.name.value == 0)
	  {
		  alert("Alert 2");
		  return false;
	  }
 
	 else if (document.form.cards.value == 0)
	  {
		  alert("Alert 3 ");
		  return false;
	  }
          else
          {
              return true;
           }
}

(j'ai du changer le nom des valeurs et en mettre qu'un extrait pour des problèmes de confidentialité et de taille (presque 50 champs a tester))

Donc lorsque je valide le formulaire a vide, j'ai une alert 1. Je remplis donc le champs Year et revalide a nouveau et la tout les autres champs ne sont pas testé et le formulaire est envoyé.

Si quelqu'un a connu le même problème ou pourrait m'aiguiller.
Merci d'avance,
Knut

[Bovino]

Code :

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

else if (document.form.name.value == 0)

form.name, ça demande l'attribut name du formulaire et de surcroit un attribut name ne possède pas de propriété value.
Tu obtiens donc une erreur et le reste du script n'est pas évalué.

[Maze45]

Ha mince désolé... J'ai mis name parce que c'est le premier truc qui m'es passé par la tête mais dans les vrais test la mention name n'apparait pas admettons que ce soit autre chose que name :

Code :

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

else if (document.form.champs2.value == 0)

J'ai vérifier avec firebug et je n'ai pas d'erreur de syntaxe au niveau du code.

[Bovino]

Tu n'as pas de champs avec des valeurs par défaut par hasard ?

[Maze45]

Non aucun, se sont des champs bien spécifique et aucun n'est commun du tout par exemple :

Code :

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

	 else if (document.add_atm.gfa-total-acct-ndc.value == 0)

C'est vraiment du javascript basique et je me demande vraiment d’où vient le bug ...

[Maze45]

Est-ce que le problème pourrait provenir du browser ?

[Maze45]

Il y as-t-il une manière plus propre pour tester un formulaire ayant de nombreux champs ?

[fab256]

Pour connaitre les valeurs de tes champs

Code :

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

1
2
3
4
for(i=0; i<document.form.elements.length; i++)
{
document.write("Le nom du champs est : " + document.form.elements[i].name + " et sa valeur est: " + document.form.elements[i].value + ".<br />");
}

Si tes champs sont de type texte tu peux les tester comme ceci

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
function FormTest ( form )
{
for(i=0; i<form.elements.length; i++)
{
if (form.elements[i].value == "") {
    alert( "Champs '"+form.elements[i].name+"' vide" );
    form.elements[i].focus();
    return false ;
  }
}
return true ;
}

et l'appel sera fera de cette manière

Code :

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

<form action="mailto:nom@domaine.com" method="post" onsubmit="return FormTest(this);">

[Bovino]

Code :

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

1
2
3
4
for(i=0; i<document.form.elements.length; i++)
{
document.write("Le nom du champs est : " + document.form.elements[i].name + " et sa valeur est: " + document.form.elements[i].value + ".<br />");
}

document.write() ?

[fab256]

Code :

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

1
2
3
4
for(i=0; i<document.form.elements.length; i++)
{
document.write("Le nom du champs est : " + document.form.elements[i].name + " et sa valeur est: " + document.form.elements[i].value + ".<br />");
}

document.write() ?

Tu peux mettre 'alert' si tu veux, c'est juste pour la phase de teste.