Convert a wrong JSON string to JSON array

Question

I have this wrong Json string:

({loc: '(49.150892549035454, 9.22825813293457)' , shopName: 'Otest', nameID: '1', apID: 'Loa', gender: 'male'}); ({loc: '(12.150892549035454, 12.22825813293457)' , shopName: 'OtestRe', nameID: '1', apID: 'Loaa', gender: 'female'});

So now I tried to correct the string with:

        var res = this.markerlist.replace(/\)|\(/g, '');
        var res2 = res.replace(/;/g, ',');
        var jsonList = JSON.stringify('[' + res2 + ']';

[{loc: '(49.150892549035454, 9.22825813293457)' , shopName: 'Otest', nameID: '1', apID: 'Loa', gender: 'male'}, ({loc: '(12.150892549035454, 12.22825813293457)' , shopName: 'OtestRe', nameID: '1', apID: 'Loaa', gender: 'female'}]

Then I try to parse it:

   var jsonRs = JSON.parse(jsonList);


     for (var rowData in jsonRs) // for acts as a foreach
    {
        console.log(jsonRs[rowData]);

    }

And I get every character as output like:

And so on.

You should not iterate over an array with for... in but with for (var i = 0; i < myArray.length; i++) — LoremIpsum
– LoremIpsum, Commented Dec 19, 2015 at 17:08
It seems like you are using a JSONP service and you forgot the callback URL parameter. Like this: http://example/com/get.json?callback=test — Ismael Miguel
– Ismael Miguel, Commented Dec 19, 2015 at 17:46

loxxy · Accepted Answer · 2015-12-19 18:02:36Z

4

Although as everyone suggested, knowing the format is important to solve your problem. So your approach to replace & parse is wrong & error prone.

Your Json string is actually javascript... So even an eval could work.

EDIT : As pointed out by Biwise Creative, it would need bit of treatment :

this.markerlist.split(";").slice(0,-1).map(function(t) {return eval(t);})

edited Dec 19, 2015 at 18:02

answered Dec 19, 2015 at 17:44

loxxy

13.2k2 gold badges28 silver badges57 bronze badges

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

1 Comment

Bitwise Creative Over a year ago

This won't work well with his string as-is since there are multiple objects. He'll end up with a single object with the values of the last data set. You'd still need to split and format the string data sets.

madox2 · Accepted Answer · 2015-12-19 20:03:02Z

You can parse your input like this:

var input = "({loc: '(49.150892549035454, 9.22825813293457)' , shopName: 'Otest', nameID: '1', apID: 'Loa', gender: 'male'}); ({loc: '(12.150892549035454, 12.22825813293457)' , shopName: 'OtestRe', nameID: '1', apID: 'Loaa', gender: 'female'});";

var result = input
             .split(";") // spliting ';' for each object
             .slice(0, -1) // removing last portion after ';'
             .map(function(item) {
                 var cleaned = item
                     .trim() // removing white spaces from the ends
                     .slice(1, -1) // getting rid of ()
                     .replace(/'/g, '"') // replacing ' to "
                     .replace(/(\w+):/g, '"$1":'); // surrounding word before ':' with double quotes
                 return JSON.parse(cleaned);
             });

console.log(result);

salc2 · Accepted Answer · 2015-12-19 18:10:31Z

Here another version with loc property as Array so you could use lattitude ang longitude

 var markerlist = "({loc: '(49.150892549035454, 9.22825813293457)' , shopName: 'Otest', nameID: '1', apID: 'Loa', gender: 'male'}); ({loc: '(12.150892549035454, 12.22825813293457)' , shopName: 'OtestRe', nameID: '1', apID: 'Loaa', gender: 'female'});";

    var arStr = markerlist.split(";");
    var jsons = [];

    for(var i=0;i < arStr.length;i++){
            var o = eval(arStr[i]);
                    if(o){
                            o.loc = o.loc
                                    .replace(/\(|\)/g, "")
                                    .split(',');
                            jsons.push(o);
                    }
    }
    console.log(jsons);

Collectives™ on Stack Overflow

Convert a wrong JSON string to JSON array

3 Answers 3

1 Comment

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

1 Comment

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related