Total Price variable limit to 2 decimals using ajax with php

Question

I am using the following code snippet to calculate a total price. This works great except #totalPrice on some occasions expands out to for example $267.9999999999. How do I reformat #totalPrice within this function to just round to two decimals as is standard in dealing with price.

function getTotalCost(inventory) {
    if(inventory) {
        getTotalParts(inventory);
        getTotalMarkup(inventory);
    }

    var labor = $('#labor').val() * 1;
    var totals = 0;
    for(i in totalMarkup) {
        totals += totalMarkup[i];
    }
    totalCost = totals+labor;
    /*if(totals == 0) {
        totalCost = 0;
    }*/
    $('#totalPrice').html(totalCost);
}

duplicate answered here i believe http://stackoverflow.com/questions/1726630/javascript-formatting-number-with-exactly-two-decimals — colonelclick, Jul 03 '12 at 19:09
`$('#labor').val() * 1` what a strange way, for casting to `Number` ??!!! — Engineer, Jul 03 '12 at 19:16
i actually like it :) if the value is not a number you'll get NaN — fmsf, Jul 03 '12 at 19:17
@fmsf You could get the same with `+$('#labor').val()` either. — Engineer, Jul 03 '12 at 19:19
@Engineer that could make a concat on other scopes while *1 will always convert it to a number — fmsf, Jul 03 '12 at 19:23
let us [continue this discussion in chat](http://chat.stackoverflow.com/rooms/13383/discussion-between-fmsf-and-engineer) — fmsf, Jul 03 '12 at 19:49

score 1 · Answer 1 · edited Jul 03 '12 at 19:14

1

When working with javascript the floating points are always a bad. Best you can do is, round it up.

But in this case you can do

(totalCost).toFixed(2);

edited Jul 03 '12 at 19:14

Raab

31,764
4
47
63

answered Jul 03 '12 at 19:11

fmsf

33,514
48
140
190

score 1 · Accepted Answer · answered Jul 03 '12 at 19:20

1

You can have:

$('#totalPrice').html(totalCost.toFixed(2));

See:

https://developer.mozilla.org/en/JavaScript/Reference/Global_Objects/Number/toFixed

Notice that toFixed method returns a formatted number, therefore converts the number to a string. It's not a problem here because html wants a string, but it's keep it in mind that in order to avoid concatenation of string when you expects sum of numbers. I believe you use $('#labor').val() * 1; for this very reason. However it's not necessary, it's better use method like parseFloat or the unary plus operator:

var labor = +$('#labor').val();

answered Jul 03 '12 at 19:20

ZER0

22,173
4
45
51

excellent, thanks for the help. Just so I know why is it better to use parseFloat or unary? – Rocco The Taco Jul 03 '12 at 19:33
It depends from what is the input you have. I usually prefer the unary plus because it's an operator therefore is not a function call. But sometimes `parseFloat` does a better job if I want actually "parse" (as the method said) the input string and not just convert. Ti give some concrete example, if you have `+"31.40 USD"` you will get `NaN`. Instead, with `parseFloat("31.40 USD")` you will have `31.4` as number. See the link I gave in my answer for further details – ZER0 Jul 03 '12 at 19:53

score 0 · Answer 3 · answered Jul 03 '12 at 19:11

0

You can use Math.round function in JavaScript like this.

totalCost = Math.round(totalCost*100)/100;
$('#totalPrice').html(totalCost);

answered Jul 03 '12 at 19:11

Sylvain B

522
3
9

Total Price variable limit to 2 decimals using ajax with php

3 Answers3