Javascript Parse Float Error

I am trying to get sum of rows of my table: td1 val = $5,000.00; td2 val = $3000.00; And I am using the following code: var totalnum = 0; $('.num').each(function(){ totalnum+=

Solution 1:

Try:

var totalnum = 0;
$('.num').each(function(){
   totalnum+= parseFloat($(this).html().substring(1).replace(',',''));
});
$('.total_num').html('$' + totalnum);

This will remove the $ (or whatever currency symbol) from the beginning and all commas before doing the parseFloat and put it back for the total.

Alternatively you could use the jQuery FormatCurrency plugin and do this:

totalnum+= $(this).asNumber();

Solution 2:

If you add $ to the value, it is no longer an integer, and can no longer be calculated with.

Trying to make the formatted value back into a number is a bad idea. You would have to cater for different currency symbols, different formattings (e.g. 1.000,00) and so on.

The very best way would be to store the original numeric value in a separate attribute. If using HTML 5, you could use jQuery's data() for it:

<td class="num" data-value="1.25">$1.25</td>
....

var totalnum = 0;
$('.num').each(function(){
  totalnum+= parseFloat($(this).data("value"));
});
$('.total_num').html(totalnum);

this way, you separate the formatted result from the numeric value, which saves a lot of trouble.

Solution 3:

Try removing $ and any other character not part of the float type:

var totalnum = 0;

$('.num').each(function(){
    var num = ($(this).html()).replace(/[^0-9\.]+/g, "");
    totalnum+= parseFloat(num);
});

$('.total_num').html(totalnum);

Edit: updated replace to remove all non-numerical characters (except periods) as per this answer.