Calcul total = prix * quantité

Enfaite, j'ai trouvé la solution.
Ça vient comme je l'imaginé de cette foutu boucle :calim2:
C'est un ami jparia d'un autre forum qui me le fait remarqué.

Je poste le script au cas ou ça sert a quelqu'un d'autre.
Apparemment l'ancienne boucle (for) demande trop de ressource. 8O

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16

function add_me(){ var a = 0; var b = 0; var c = 0; var i = 1;   while ( document.getElementById("qte1_" + i) ){ a = parseFloat(document.getElementById("qte1_" + i).value); b = parseFloat(document.getElementById("prixu_" + i).value); a = Math.round(a) * Math.round(b); c = c + a; i++; } document.getElementById("total").value = c.toString(); }

---

Un grand merci a vous deux NoSmoking, SpaceFrog :ccool:

Citation:

---

Envoyé par kof_eve

Enfaite, j'ai trouvé la solution.
Ça vient comme je l'imaginé de cette foutu boucle :calim2:
....
....
Apparemment l'ancienne boucle (for) demande trop de ressource. 8O

---

que NENNI, pas du tout, problème de condition de fin de boucle...
...dans le code ci dessus comme dans celui que je t'ai proposé, dans mon message précédent, on note que la condition de sortie de boucle se base sur l'existence des éléments.

C'est cela qui visiblement faisait planter ton script, la condition i <= 300.

J'ai fait un test rapide avec 1000 INPUT quantité et prix...et cela marche bien, il n'y a pas de problème de ressources, si ce n'est que c'est un peu long....

Et justement à ce sujet il conviendrait de mettre en mémoire les objets dans un array pour ne pas avoir, à chaque changement de valeur, à faire appel à la méthode document.getElementById.

cela pourrait donner

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19

var tElem = []; // tableau de stockage function add(){ // initialisation la 1st fois if( !tElem.length){ for( i = 1; ( oQte = document.getElementById('qte1_' +i)) && ( oPrix = document.getElementById('prixu_' +i)); i++){ tElem[i] = []; tElem[i][0] = oQte; tElem[i][1] = oPrix; } } // le calcul proprement dit var a = b= 0; var i, nb = tElem.length; for( i = 1; i < nb; i++){ a = tElem[i][0].value * tElem[i][1].value; b +=a; } document.getElementById('total').value = b; }

---

l'initialisation pourrait d'ailleurs se faire sur le onlaod de la page.

Oui c'est vrai ca vient du boucle qui plante avec la méthode document.getElementById s'il est vide :aie:
Voila pourquoi il faut passer par une variable ou un tableau :ccool:
Votre script marche aussi.
Un grand merci a vous, bonne journée ;)