Math.round qui n'arrondit pas

Version imprimable

Bonjour à tous.
Je souhaite arrondir une valeur à deux décimales près.
Celle valeur étant calculée à partir d'autres.

Voici le code complet :

Code:

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
39
40
41
42
43
44
45
46
47
48
 
  function calculerMontantTotal(idInput) {
    var isChecked = document.getElementById("echeancier_" + idInput).checked;
 
    var montantTotal = parseFloat(document.getElementById("totalAPayer").value);
    montantTotal = Math.round(montantTotal*100)/100;
 
    var montant = parseFloat(document.getElementById("montant_" + idInput).value);
    montant = Math.round(montant*100)/100;
 
    var cptEcheancier = parseInt(idInput) + 1;
    if(isChecked) {
      document.getElementById("totalAPayer").value = montantTotal + montant;
      toggleEcheancier(cptEcheancier, false, montantTotal);
    } else {
      var newMontantTotal = montantTotal - montant;
      newMontantTotal = Math.round(parseFloat(newMontantTotal)*100)/100;
 
      document.getElementById("totalAPayer").value = newMontantTotal;
      document.getElementById("totalAPayer").value = parseFloat(document.getElementById("totalAPayer").value);
      document.getElementById("totalAPayer").value = Math.round(document.getElementById("totalAPayer").value*100)/100;
 
      toggleEcheancier(cptEcheancier, true, newMontantTotal);
    }
  }
 
 
  function toggleEcheancier(idInput, isDisabled, montantTotal) {
    if(document.getElementById("echeancier_" + idInput) != null) {
      document.getElementById("echeancier_" + idInput).disabled = isDisabled;
 
      if(isDisabled && document.getElementById("echeancier_" + idInput).checked) {
        document.getElementById("echeancier_" + idInput).checked = false;
 
        var montantCourant = parseFloat(document.getElementById("montant_" + idInput).value);
        montantCourant = Math.round(montantCourant*100)/100;
 
        var newMontantTotal = montantTotal - montantCourant;
        newMontantTotal = Math.round(parseFloat(newMontantTotal)*100)/100;
 
        document.getElementById("totalAPayer").value = newMontantTotal;
        document.getElementById("totalAPayer").value = parseFloat(document.getElementById("totalAPayer").value);
        document.getElementById("totalAPayer").value = Math.round(document.getElementById("totalAPayer").value*100)/100;
 
        var cptEcheancier = parseInt(idInput) + 1;
        toggleEcheancier(cptEcheancier, isDisabled, newMontantTotal);
      }
    }

Dans mon cas il y a 3 valeur en tout à additionner, chacune valant 53,3. Tout se passe bien jusqu'à la dernière ou je passe de 106,6 à 1959.8999999999

D'ou cela peut-il venir?
J'espère être assez clair! :)

Merci d'avance

A quoi ça sert d'attribuer 3 fois une value ?

Code:

1
2
3
document.getElementById("totalAPayer").value = newMontantTotal;
      document.getElementById("totalAPayer").value = parseFloat(document.getElementById("totalAPayer").value);
      document.getElementById("totalAPayer").value = Math.round(document.getElementById("totalAPayer").value*100)/100;

???

31/03/2010, 10h01
michel.di

oui je sais que c'est bien con la! :)
mais je cherchais un peu d'ou venais le problème, donc l'écriture passait après...
31/03/2010, 10h08
michel.di

J'ai essayé d'utiliser toFixed aussi mais sans résultat
31/03/2010, 10h17
SpaceFrog

tu n'aurais pas un souci de séparateur decimal
chez moi ça marche très bien
31/03/2010, 10h23
michel.di

Quand je regarde dans Firebug il additionne bien 53,3 et 106,6 mais la valeur affichée est 159.8999999 au lieu de 159.9
31/03/2010, 10h29
SpaceFrog

oui jusque là c'est normal c'est la gestion de la virgule flottante ...
c'est justement sur le resultat qu'il faut appliquer le round ...

Code:

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
39
40
41
 
function calculerMontantTotal(idInput) {
    var isChecked = document.getElementById("echeancier_" + idInput).checked;
 
    var montantTotal = parseFloat(document.getElementById("totalAPayer").value);
 
    var montant = parseFloat(document.getElementById("montant_" + idInput).value);
 
    var cptEcheancier = parseInt(idInput) + 1;
    if(isChecked) {
      document.getElementById("totalAPayer").value = montantTotal + montant;
      toggleEcheancier(cptEcheancier, false, montantTotal);
    } else {
      var newMontantTotal = new Number(montantTotal - montant);
      newMontantTotal = Math.round(newMontantTotal*100)/100;
 
      document.getElementById("totalAPayer").value = newMontantTotal;
 
      toggleEcheancier(cptEcheancier, true, newMontantTotal);
    }
  }
 
function toggleEcheancier(idInput, isDisabled, montantTotal) {
    if(document.getElementById("echeancier_" + idInput) != null) {
      document.getElementById("echeancier_" + idInput).disabled = isDisabled;
 
      if(isDisabled && document.getElementById("echeancier_" + idInput).checked) {
        document.getElementById("echeancier_" + idInput).checked = false;
 
        var montantCourant = parseFloat(document.getElementById("montant_" + idInput).value);
 
        var newMontantTotal = new Number(montantTotal - montantCourant);
        newMontantTotal = Math.round(newMontantTotal*100)/100;
 
        document.getElementById("totalAPayer").value = newMontantTotal;
 
        var cptEcheancier = parseInt(idInput) + 1;
        toggleEcheancier(cptEcheancier, isDisabled, newMontantTotal);
      }
    }
  }

Ne marche pas non plus.... :(

c'est bon ça marche, je suis c** je ne faisais pas le round au bon endroit. mais ce que je ne comprend pas c'est qu'en additionnant 53,3 et 106,6 il trouve 159,899999 et qu'un round est nécessaire!

Code:

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
39
40
41
42
43
44
45
46
 
  function calculerMontantTotal(idInput) {
    var isChecked = document.getElementById("echeancier_" + idInput).checked;
 
    var montantTotal = parseFloat(document.getElementById("totalAPayer").value);
 
    var montant = parseFloat(document.getElementById("montant_" + idInput).value);
 
    var cptEcheancier = parseInt(idInput) + 1;
    var newMontantTotal;
    if(isChecked) {
      newMontantTotal = montantTotal + montant;	
      newMontantTotal = Math.round(newMontantTotal*100)/100;
      document.getElementById("totalAPayer").value = newMontantTotal;
 
      toggleEcheancier(cptEcheancier, false, montantTotal);
    } else {
      newMontantTotal = new Number(montantTotal - montant);
      newMontantTotal = Math.round(newMontantTotal*100)/100;
 
      document.getElementById("totalAPayer").value = newMontantTotal;
 
      toggleEcheancier(cptEcheancier, true, newMontantTotal);
    }
  }
 
 
  function toggleEcheancier(idInput, isDisabled, montantTotal) {
    if(document.getElementById("echeancier_" + idInput) != null) {
      document.getElementById("echeancier_" + idInput).disabled = isDisabled;
 
      if(isDisabled && document.getElementById("echeancier_" + idInput).checked) {
        document.getElementById("echeancier_" + idInput).checked = false;
 
        var montantCourant = parseFloat(document.getElementById("montant_" + idInput).value);
 
        var newMontantTotal = montantTotal - montantCourant;
        newMontantTotal = Math.round(newMontantTotal*100)/100;
 
        document.getElementById("totalAPayer").value = newMontantTotal;
 
        var cptEcheancier = parseInt(idInput) + 1;
        toggleEcheancier(cptEcheancier, isDisabled, newMontantTotal);
      }
    }
  }

Merci de ton aide en tout cas SpaceFrog! ;)

Code:

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
39
40
41
<script type="text/javascript">
function calculerMontantTotal(idInput) {
    var isChecked = document.getElementById("echeancier_" + idInput).checked;
 
    var montantTotal = +document.getElementById("totalAPayer").value;
 
    var montant = +document.getElementById("montant_" + idInput).value;
 
    var cptEcheancier = parseInt(idInput,10) + 1;
    if(isChecked) {
      document.getElementById("totalAPayer").value = Math.round((montantTotal + montant)*100)/100;
      toggleEcheancier(cptEcheancier, false, Math.round((montantTotal)*100)/100);
    } else {
      var newMontantTotal = Math.round((montantTotal - montant)*100)/100;
 
      document.getElementById("totalAPayer").value = newMontantTotal;
 
      toggleEcheancier(cptEcheancier, true, newMontantTotal);
    }
  }
 
function toggleEcheancier(idInput, isDisabled, montantTotal) {
    if(document.getElementById("echeancier_" + idInput) != null) {
      document.getElementById("echeancier_" + idInput).disabled = isDisabled;
 
      if(isDisabled && document.getElementById("echeancier_" + idInput).checked) {
        document.getElementById("echeancier_" + idInput).checked = false;
 
        var montantCourant = +document.getElementById("montant_" + idInput).value;
 
        var newMontantTotal = newMontantTotal = Math.round(newMontantTotal*100)/100;        
 
        document.getElementById("totalAPayer").value = newMontantTotal;
 
        var cptEcheancier = parseInt(idInput,10) + 1;
        toggleEcheancier(cptEcheancier, isDisabled, newMontantTotal);
      }
    }
  }
 
</script>

31/03/2010, 13h35
Bovino

Citation:

Envoyé par michel.di

ce que je ne comprend pas c'est qu'en additionnant 53,3 et 106,6 il trouve 159,899999 et qu'un round est nécessaire!

Citation:

Envoyé par SpaceFrog

c'est la gestion de la virgule flottante ...

:fleche: [ame]http://fr.wikipedia.org/wiki/Virgule_flottante[/ame]
31/03/2010, 13h41
SpaceFrog

ce n'est pas un phénomène spécifique à javascript ...
31/03/2010, 15h05
michel.di

Merci beaucoup pour vos précisions je m'endormirai moins bête! :)
je marque comme résolu