Discussion :

[kof_eve]

Enfaite, j'ai trouvé la solution.
Ça vient comme je l'imaginé de cette foutu boucle
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.

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
 
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

[NoSmoking]

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

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 :

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
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.

[kof_eve]

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