Can't parse string with JSON.parse

Question

I have the following code that splits a string on the newline character and then calls JSON.parse on each line.

var lines = [];
var ks = data.split("\n");
_(ks).each(function(line){
    lines.push(JSON.parse(line));
}, this);
return lines;

The problem is after the split each line is like this

"{"id":"123","attr1":"abc"}"

However in order for JSON.parse to work my string needs to be surrounded my single quotes like

'{"id":"123","attr1":"abc"}'

Is there a way to perform this conversion on each line?

You seem to be conflating three different things. String literal syntax is not the same as what the JavaScript console will display when logging strings which is, again, not the same as the data stored within the string. If the data really started with ' and ' then JSON.parse would fail because ' isn't a string delimiter in JSON. — Quentin
– Quentin, Commented Jan 16, 2015 at 13:14
...and even disregarding the confusion over quotes, if you split a string by newlines characters then you will break JSON. — symcbean
– symcbean, Commented Jan 16, 2015 at 13:21

score 4 · Accepted Answer · 2015-01-16 13:27:52Z

4

If the string is structured like this...

"{"id":"123","attr1":"abc"}
{"id":"123","attr1":"abc"}
{"id":"123","attr1":"abc"}
{"id":"123","attr1":"abc"}"

... then a quicker alternative to parsing each line as a separate JSON might be to do this:

JSON.parse("["+data.replace(/^\n+|\n+$/g, "").replace(/\n+/g, ",")+"]");

Doing this replaces each sequence of newlines with a comma, and wraps the resulting line in an array literal, so you have a fresh new array to iterate through. :)

edited Jan 16, 2015 at 13:27

answered Jan 16, 2015 at 13:20

user458541

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

3 Comments

Pattle Over a year ago

Awesome, this solved my problem. Although I did have to just make one change to remove the last comma of the end of the string

user458541 Over a year ago

Ah right, yeah, sorry... probably should've stripped leading/trailing whitespace first. xD My bad. I'll edit my answer, haha.

JazzBrotha Over a year ago

Saved my day, bro!

Collectives™ on Stack Overflow

Can't parse string with JSON.parse

1 Answer 1

3 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

3 Comments

Your Answer

Sign up or log in

Post as a guest

Related