Discussion :

[TokeinatorWrecK]

Bonjour les gens

Voilà j'ai voulu réaliser un script qui examine ligne par ligne le contenu des champs d'un formulaire en forme de tableau.
En gros:
- avoir au moins tous les champs de la première ligne remplis (ça ça fonctionne )
- si, pour n'importe quelle autre ligne, un champ est rempli, les autres de cette même ligne doivent l'être aussi (ça ça fonctionne pas )

Voici ce qui m'empêche de dormir :

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
function verifForm(formulaire)
{
	if ((formulaire.p1.value == "")||(formulaire.m1.value == "")||(formulaire.c1.value == "")||(formulaire.d1.value == "")||(formulaire.t1.value == ""))
	{
		alert("Vous devez au moins remplir la première ligne!")
	}
	else
	{
		i=1;
		while (i<10)
			{
				if (formulaire.p[i].value == "")
					{
						if ((formulaire.m[i].value != "")||(formulaire.c[i].value != "")||(formulaire.d[i].value != "")||(formulaire.t[i].value != ""))
						{
							alert("Veuillez remplir tous les champs d'une même ligne!");
							break;
						}
						else
						{
							i++;
							continue;
						}
					}
				else if ((formulaire.m[i].value == "")||(formulaire.c[i].value == "")||(formulaire.d[i].value == "")||(formulaire.t[i].value == ""))
					{
						alert("Veuillez remplir tous les champs d'une même ligne!");
						break;
					}
					else
					{
						i++;
						continue;
					}
			}	
		formulaire.submit()
	}
}

Une partie du formulaire html:

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
 
<table width="800" border="0" bgcolor="black">
<caption><h1>Formulaire</h1></caption>
<tr align="center" BGCOLOR="yellow">
<th width="20%">Colonne1</th>
<th width="20%">Colonne2</th>
<th width="20%">Colonne3</th>
<th width="20%">Colonne4</th>
<th width="20%">Colonne5</th>
</tr>
<tr align="center" BGCOLOR="#EEEE00"> /*Cette ligne DU TABLEAU apparait
encore 8 fois, où on incrément
chaque numéro des noms
des champs à chaque fois*/
<td><input type="text" name="p1"></td>
<td><input type="text" name="m1"></td>
<td><input type="text" name="c1"></td>
<td><input type="text" name="d1"></td>
<td><input type="text" name="t1"></td>
</tr>
 
...
 
<tr align="center" BGCOLOR="#CDCD00">
<td colspan="5">
<input type="button" value="Retour à l'accueil" OnClick="redirecAcceuil()">&nbsp; &nbsp;
<input type="button" value="Valider" onClick="verifForm(this.form)">
</td></tr></table>

Alors, z'en pensez quoi?

[E.Bzz]

EDIT : Oups ... sensible le bouton "enregistrer"

[E.Bzz]

Bonjour,

Alors, z'en pensez quoi?

d'abord que les balises [CODE ][/CODE ] rendrait ton post + lisible.
Ensuite que ta syntaxe pour accéder aux objets n'est pas bonne non plus (en restant poli )
Enfin, que sans le html associé, ça serait aléatoire de tenter une correction.

A+

[TokeinatorWrecK]

Je les trouvées pas ces balises ^^

C'est quoi de mauvais dans ma syntaxe d'accés aux objets?

[Shinuza]

Parce qu'on utilise pas le name pour accèder à des éléments

[E.Bzz]

Parce qu'on utilise pas le name pour accèder à des éléments

Et que même, en admettant, la syntaxe correct serait

Code :

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

document.forms['formulaire'].elements['p1'].value

Mais l'utilisation des ID est effectivement largement préférable

Au fait, pour les balises [code ], le bouton est le "#" (à droite de tous les autres).

A+

[TokeinatorWrecK]

Bon après les quelques réglages qui ont suivis suite à vos remarques, et ben ça change rien

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
function verifForm(formulaire)
{
	if ((document.forms['formulaire'].elements['p1'].value == "")||(document.forms['formulaire'].elements['m1'].value == "")||(document.forms['formulaire'].elements['c1'].value == "")||(document.forms['formulaire'].elements['d1'].value == "")||(document.forms['formulaire'].elements['t1'].value == ""))
	{
		alert("Vous devez au moins remplir la première ligne!")
	}
	else
	{
		i=1;
		while (i<10)
			{
				if (document.forms['formulaire'].elements['p[i]'].value == "")
					{
						if ((document.forms['formulaire'].elements['m[i]'].value != "")||(document.forms['formulaire'].elements['c[i]'].value != "")||(document.forms['formulaire'].elements['d[i]'].value != "")||(document.forms['formulaire'].elements['t[i]'].value != ""))
						{
							alert("Veuillez remplir tous les champs d'une même ligne!");
							break;
						}
						else
						{
							i++;
							continue;
						}
					}
				else if ((document.forms['formulaire'].elements['m[i]'].value == "")||(document.forms['formulaire'].elements['c[i]'].value == "")||(document.forms['formulaire'].elements['d[i]'].value == "")||(document.forms['formulaire'].elements['t[i]'].value == ""))
					{
						alert("Veuillez remplir tous les champs d'une même ligne!");
						break;
					}
					else
					{
						i++;
						continue;
					}
			}	
		formulaire.submit()
	}
}

Je me demande si ça viendrais pas de

Code :

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

document.forms['formulaire'].elements['m[i]'].value

le "i" (compteur) est surement pas interprété de la façon dont je voudrais