interrompre une boucle sur erreur

**flyingfr53** · 26/01/2011, 23h14

Bonjour,

je souhaite faire le total des valeurs de champs input dont le nombre varie en javascript ( l'utilisateur peut rajouter des lignes ) mais je n'en connais pas le nombre et dans ma boucle il arrive un moment ou le champ input n'etant pas défini le programme se bloque. je voudrais simplement que ma boucle s’arrête a ce moment la pour pouvoir récupérer mon total,

mes champs input ont pour nom paf1, paf2 ,....... ( mon indice s’arrête à une valeur inférieure à 12 que je ne connais pas)

Code :

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

1
2
3
4
5
6
total =0 ;
for(i=1;i<=12;i++)
{tabaf[i]=+eval("document.formulaire.paf"+i+".value");
total+=tabaf[i];
}
alert (total);

j'espère avoir été clair.
merci de votre aide

**Myst.** · 26/01/2011, 23h47

Déjà, si tu ne connais pas d'avance le nombre de fois que tu vas boucler, pourquoi utiliser un for ?
Par définition, tu utilises un for quand tu connais (ou que tu as un moyen de connaître) le nombre d'itérations que tu feras. Dans le cas contraire, tu devrais plutôt opter pour un while.

Code :

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

1
2
3
4
5
6
7
i = 0;
while(document.getElementById("paf"+i))
{
   tabaf[i] = eval("document.formulaire.paf"+i+".value");
   i++;
}
alert(i - 1);

Tu n'as pas besoin de plus normalement...
La methode getElementById te renverras vrai si l'élément que tu passes en paramètre existe.

**b_zakaria** · 26/01/2011, 23h51

(pas très optimisé mais ça fait l'affaire)

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
<html>
	<head>
		<script type="text/javascript">
			function calcul(){
				var valeur = 0;
				var total = 0;
				var elements = document.getElementById("formulaire").elements;
				var nombreElements = elements.length;
				for(i=0;i< nombreElements;i++)
				{
					if (elements[i].id.indexOf('paf')!=-1){
						if (elements[i].value !="")
							total += parseInt(elements[i].value);
					}
				}
				alert(total);
			}
		</script>
	</head>
	<body>
		<form id="formulaire">
			<input type="text" id="paf1" /><br/>
			<input type="text" id="paf2" /><br/>
			<input type="text" id="paf3" /><br/>
			<input type="button" value="calcul" onclick='calcul()'/>
		</form>
	</body>
</html>

**SpaceFrog** · 27/01/2011, 08h25

sinon un break fait sortir d'une boucle

**SpaceFrog** · 27/01/2011, 08h33

Code :

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

1
2
3
4
for(i=0;i<10;i++){
if(i==5){break}
}
alert(i)

**javatwister** · 27/01/2011, 17h37

Je ne comprends pas pourquoi tu n'insères pas tes input dans un conteneur quelconque; un simple <p id="inputs"> </p> fait l'affaire;
du coup, tu n'as plus qu'à boucler sur le nombre d'input dans la balise, sans souci et sans risque d'erreur;

**Bovino** · 27/01/2011, 20h40

Code :

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

tabaf[i]=+eval("document.formulaire.paf"+i+".value");

Pourquoi cet horrible eval ?

**flyingfr53** · 27/01/2011, 22h05

Bonsoir,

A Zakaria,

merci pour ta réponse que j'ai testé et qui fonctionne bien, je n'en ai pas encore compris toutes les subtilités mais je vais y reflechir.

A Bovino,
je n'ai pas reussi à obtenir mes valeurs d'une autre manière sachant que mes champs input sont générés en php avec un indice variable.

A Spacefrog,

Ta solution est efffectivement la plus raisonnable, je ne pensais pas pouvoir recuperer la valeur de mon dernier indice, mais en reflechissant je dois pouvoir y arriver et ce serait quand même plus satisfaisant.

A Myst

comme j'utilisais document.formulaire.name.value, dès que je tombais sur un name qui n'existait pas celà bloquait immediatement mon javascript, et ma boucle while se bloquait pareillement. Il semble que avec document.getElementById cela ne genere pas d'erreur.

Enfin merci à tous.

**b_zakaria** · 28/01/2011, 00h36

Quelles lignes te semblent incompréhensibles dans mon code?

**javatwister** · 28/01/2011, 12h02

j'allais le dire, zakaria

d'ailleurs, je reprenais le même principe, simplement en évitant de boucler sur tout ce qui bouge dans le formulaire;

par ailleurs, on ne sait tjrs pas comment flyingfr53 insère ses nouveaux champs..., ce qui peut s'avérer intéressant;

**Bovino** · 28/01/2011, 13h15

Envoyé par flyingfr53

A Bovino,
je n'ai pas reussi à obtenir mes valeurs d'une autre manière sachant que mes champs input sont générés en php avec un indice variable.

Code :

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

tabaf[i]=+document.formulaire.elements['paf'+i+].value;

interrompre une boucle sur erreur

JavaScript

Discussions similaires

Partager

Partager