Javascript parseInt() returning NaN on string literal containing a number

Question

["57300","730","17300","17330","17340","70","220","320","380","420","340","130","50","280","360","550","43110","44340","400","620","440","20","72850"]

Is an array of strings being passed back into a javascript function which proceeds to do work on it to break it up into an array of strings (using the split function with "," as a delimiter).

The problem here, is I need to convert each of those after that into an integer. Sounds easy okay.

So I proceed to do a:

$.each(data, function(i, item)

On it. If I console log the item, I can see it being "57300", "730" etc etc. Which is fine. But if I try to do this:

var number = parseInt(item, 10);

And console log what number is... its NaN (should it not be 57300, 730, etc without the quotes?). Is some hidden character messing with me causing parseInt to break?

They seem to be ["\"57300\"","\"730\"", ...] rather than ["57300","730", ...] — Esailija
– Esailija, Commented Nov 28, 2012 at 17:56
Please provide the actual code. Why do you need to split? Why is your "array" a string? Don't you decode your JSON data? — phant0m
– phant0m, Commented Nov 28, 2012 at 17:57
If you had to use split to break it up, then it wasn't an array of strings, it was just one big string. It sounds like you maybe should be using JSON.Parse. — Barmar
– Barmar, Commented Nov 28, 2012 at 17:58

Selvakumar Arumugam · Accepted Answer · 2012-11-28 18:12:21Z

2

I think your initial string is like below,

var result = '"57300","730","17300","17330"';

Since you are splitting with , as delimiter, you will end up getting like below,

data = ["\"57300\"","\"730\"","\"17300\"","\"17330\""];

Basically a string with quotes.. And when you parse using parseInt

parseInt("\"57300\"", 10); //throws NaN

Solution:

Instead you can wrap that data with [] and instead of split use parseJSON function. See below,

var result = '"57300","730","17300","17330"';

result = '[' + result + ']';

var data = $.parseJSON(result);

$.each(data, function (i, item) {
    var number = parseInt(item, 10);
    alert(number);
});

DEMO: http://jsfiddle.net/mYysX/

answered Nov 28, 2012 at 18:12

Selvakumar Arumugam

79.9k15 gold badges123 silver badges139 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Tim Booker · Accepted Answer · 2012-11-28 18:05:45Z

2

According to your post you are splitting the string on comma which means you are leaving the quotes around the numbers. This means you are doing the equivalent of parseInt("\"573007\""). You would need to strip the quotes before calling parseInt.

edited Nov 28, 2012 at 18:05

answered Nov 28, 2012 at 17:54

Tim Booker

2,7891 gold badge25 silver badges36 bronze badges

6 Comments

Selvakumar Arumugam Over a year ago

I don't think so.. the one of the purpose of parseInt is to convert it to number. Parses a string argument and returns an integer of the specified radix or base. - Definition from MDN

Tim Booker Over a year ago

Yes, it is correct. He is doing the equivalent of parseInt("\"573007\"") because he has split the string on comma without removing quotes. Result is NaN. jsfiddle.net/EFwcx

Selvakumar Arumugam Over a year ago

@apathetic from OP array of strings passed back into a javascript function - I believe he has an array already. It it not a single string.

Tim Booker Over a year ago

@Vega from OP: "using the split function with "," as a delimiter"

cHao Over a year ago

@apathetic: If it had to be split, it isn't an array -- it's a string. And if it's a string that looks like the stuff above, it's actually JSON and should be parsed -- which would eliminate the quotes anyway.

|

Collectives™ on Stack Overflow

Javascript parseInt() returning NaN on string literal containing a number

2 Answers 2

Comments

6 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

6 Comments

Your Answer

Sign up or log in

Post as a guest

Related