if dans une boucle (ou l'inverse)

**TokeinatorWrecK** · 06/06/2007, 15h01

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** · 06/06/2007, 15h15

EDIT :

Oups ... sensible le bouton "enregistrer"

**E.Bzz** · 06/06/2007, 15h16

Bonjour,

Envoyé par TokeinatorWrecK

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** · 06/06/2007, 15h32

Je les trouvées pas ces balises ^^

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

**Shinuza** · 06/06/2007, 15h37

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

**E.Bzz** · 06/06/2007, 15h50

Envoyé par Shinuza

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** · 06/06/2007, 17h17

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

**E.Bzz** · 06/06/2007, 17h31

Envoyé par TokeinatorWrecK

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

Gââgné

En fait il est pas interprété du tout, car "en dur" ...

Code :

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

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

devrait résoudre le pb

A+

**TokeinatorWrecK** · 06/06/2007, 19h11

Bon ça règle déjà un problème, mais au final ça n'arrange rien

Je ne sais comment déboguer la chose, sur le plan logique ça me parait juste pourtant

**E.Bzz** · 07/06/2007, 09h04

Envoyé par TokeinatorWrecK

Bon ça règle déjà un problème, mais au final ça n'arrange rien

Je ne sais comment déboguer la chose, sur le plan logique ça me parait juste pourtant

C'est à dire ?
Tu en es où ?
Fonctionnement ? Erreur ? Version actuelle de ton code ?

**TokeinatorWrecK** · 07/06/2007, 12h40

Bon j'ai changer un peu le tout, ça marche toujours pas mais au moins j'aurais essayé

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

Bon la récupération des valeurs des champs est correct, mais là c'est la boucle qui pose problème, elle ne se termine pas

(du moins pas comme je voudrais

) Je vais revoir un peu l'enchainement et les test conditionnels, parce que j'ai fait ça cette nuit, tard, très tard

**TokeinatorWrecK** · 07/06/2007, 12h51

Ah ben enfaite c'est bon ça fonctionne, un petit rajout de break; ici ou là (d'ailleurs je ne sais même pas pourquoi je l'ai avait enlevé

Merci pour votre aide, ça m'as permis de mon coucher moins c.. bête ces derniers jours

Je reviendrais

La fonction, qui fonctionne!

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

if dans une boucle (ou l'inverse)

JavaScript

Discussions similaires

Partager

Partager