current community

your communities

more stack exchange communities

Stack Exchange
tour help
stack overflow careers
Take the 2-minute tour ×
Stack Overflow is a question and answer site for professional and enthusiast programmers. It's 100% free, no registration required.
up vote 125 down vote favorite
12

I am trying to parse two values from a datagrid.
The fields are numeric, and when they have a comma (ex. 554,20), I can't get the numbers after the comma.
I've tried parseInt and parseFloat. How can I do this?

share|improve this question
    
see also: stackoverflow.com/questions/24318654 –  dreftymac Jun 20 '14 at 1:42

6 Answers 6

up vote 177 down vote accepted

If they're meant to be separate values, try this:

var values = "554,20".split(",")
var v1 = parseFloat(values[0])
var v2 = parseFloat(values[1])

If they're meant to be a single value (like in French, where one-half is written 0,5)

var value = parseFloat("554,20".replace(",", "."));
share|improve this answer
    
What about where you have a comma used as a thousand seperator? E.g. 1234 written as 1,234 –  Chris B Jun 26 '09 at 10:33
6  
You could just remove the comma, then parse it. (eg. replace it with "") –  Jesse Rusak Jun 30 '09 at 11:25
4  
it's not very safe, as sconto = parseFloat("5,,5".replace(",", ".")); returns 5 leading one to believe it's a valid number, but you loose the .5 part or you could consider it is not a valid number –  max4ever Apr 1 '11 at 9:02
1  
@max4ever If you're worried about silently accepting invalid numbers, you can use +(string) instead of parseString(string) as suggested by @Stev below. So, the first example would be v1 = +(values[0]); v2 = +(values[1]);. –  Jesse Rusak Nov 8 '12 at 2:10
    
FAIL, what if my number is 6.000.000. Function replace only replace first comma separator –  GusDeCooL Sep 17 '13 at 23:41
up vote 22 down vote

Have you ever tried to do this? :p

var str = '3.8';ie
alert( +(str) + 0.2 );

+(string) will cast string into float.

Handy!

So in order to solve your problem, you can do something like this:

var floatValue = +(str.replace(/,/,'.'));
share|improve this answer
1  
I really like the +(str) solution - parseFloat ignores invalid characters after the number, +(str) returns NaN in those cases, which is exactly what I need. –  Alex May 28 '12 at 12:25
up vote 16 down vote

Replace the comma with a dot.

This will only return 554:

var value = parseFloat("554,20")

This will return 554.20:

var value = parseFloat("554.20")

So in the end, you can simply use:

var fValue = parseFloat(document.getElementById("textfield").value.replace(",","."))

Don't forget that parseInt() should only be used to parse integers (no floating points). In your case it will only return 554. Additionally, calling parseInt() on a float will not round the number: it will take its floor (closest lower integer).

share|improve this answer
up vote 10 down vote

If you extend String object like this..

String.prototype.float = function() { 
  return parseFloat(this.replace(',', '.')); 
}

.. you can run it like this

"554,20".float()
> 554.20

works with dot as well

"554.20".float()
> 554.20

typeof "554,20".float()
> "number"
share|improve this answer
3  
Although usually it's considered bad form to modify the base object prototypes. What if another framework also tried to do that but the functionality differed? –  phreakhead Jan 22 '13 at 1:02
up vote 1 down vote

@GusDeCool or anyone else trying to replace more than one thousands separators, one way to do it is a regex global replace: /foo/g. Just remember that . is a metacharacter, so you have to escape it or put it in brackets (\. or [.]). Here's one option:

var str = '6.000.000';
str.replace(/[.]/g,",");
share|improve this answer
up vote 0 down vote

I had the same problem except I did not know in advance what were the thousands separators and the decimal separator. I ended up writing a library to do this. If you are interested it here it is : https://github.com/GuillaumeLeclerc/number-parsing

share|improve this answer

protected by Community♦ Mar 21 '13 at 10:52

Thank you for your interest in this question. Because it has attracted low-quality answers, posting an answer now requires 10 reputation on this site.

Would you like to answer one of these unanswered questions instead?