Getting error while Convert string to json object

Question

var result = "";
result += "[";
for(i=0;i<=10;i++)
  {
    result += "{ 'key': 'keyvalue" + i + "', 'values': [";
    for(j=0;j<=10;j++)
      {
        result += "{ 'key': 'subkeyvalue"+j+"', 'value':"+j+"}, ";
      }
    result += "]}, ";
  }

result += "]";

console.log(result);
console.log(JSON.stringify(result));
console.log(JSON.parse(result));

If i try to convert String to JSON.parse. I am getting below error.

JSON.parse: expected property name or '}' at line 1 column 4 of the JSON data

can you please any one resolve this problem.

BTW why you are doing this? As you can do the same with JSON.stringify passing JavaScript object; — itzmukeshy7
– itzmukeshy7, Commented Oct 19, 2016 at 4:55
why you are creating string ? make proper object and add to array and then use JSON.stringify() to make json string — R.K.Saini
– R.K.Saini, Commented Oct 19, 2016 at 4:56
@Tushar said Wrap keys and values in double quotes not in single; — itzmukeshy7
– itzmukeshy7, Commented Oct 19, 2016 at 4:58
after converting string to JSON object i need to create chart — G Boomanikandan
– G Boomanikandan, Commented Oct 19, 2016 at 4:58

itzmukeshy7 · Accepted Answer · 2016-10-19 04:59:43Z

1

In this snippet ( result += "]}, ";) , "," (comma) is getting appended at the last, so the json will be like "},]" where you will be expecting like "}]"

edited Oct 19, 2016 at 4:59

itzmukeshy7

2,6781 gold badge24 silver badges30 bronze badges

answered Oct 19, 2016 at 4:58

Surye Prakash

312 bronze badges

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

3 Comments

itzmukeshy7 Over a year ago

You should add this in comment as this is not a complete answer;

itzmukeshy7 Over a year ago

May be result += '{ "key": "subkeyvalue'+j+'", "value":'+j+'}, '; then is this statement is correct?

G Boomanikandan Over a year ago

based on my requirement result is coming. but its coming string format

nnnnnn · Accepted Answer · 2016-10-19 05:16:40Z

Don't create JSON strings manually through string concatenation. Just...don't.

Create an array, create appropriate objects and add them to the array, then pass the array to your charting library as is.

Or if you actually need a JSON string use JSON.stringify() on the array.

And note that there is no such thing as a "JSON object".

var result = [];
var vals;
for(var i=0;i<=10;i++) {
  vals = [];
  for(j=0;j<=10;j++) {
    vals.push( { key: 'subkeyvalue'+j, value: j } );
  }
  result.push( { key: 'keyvalue' + i, values: vals } );
}

// someChartFunction(result);

console.log(result);
console.log(JSON.stringify(result));

itzmukeshy7 · Accepted Answer · 2016-10-19 05:02:03Z

0

I think you can use JSON.stringify() or use key and value in (') quote

var result = "";
result += "[";
for(i=0;i<=10;i++)
  {
    result += "{ 'key': 'keyvalue" + i + "', 'values': [";
    for(j=0;j<=10;j++)
      {
        result += "{ 'key': 'subkeyvalue"+j+"', 'value':"+j+"}, ";
      }
    result += "]}, ";
  }

result += "]";

console.log(result);

console.log(JSON.stringify(result));

edited Oct 19, 2016 at 5:02

itzmukeshy7

2,6781 gold badge24 silver badges30 bronze badges

answered Oct 19, 2016 at 5:00

user4501357

1 Comment

G Boomanikandan Over a year ago

i tried JsON.stringify again it showing same string format result

Amrinder · Accepted Answer · 2016-10-19 05:12:18Z

Try to use the code given below. You should not append the , (Comma) to the last element while iterating

var result = "";
result += "[";
for(i=0;i<=10;i++)
  {
    result += '{ "key": "keyvalue' + i + '", "values": [';
    for(j=0;j<=10;j++)
      {
        result += '{ "key": "subkeyvalue'+j+'", "value":'+j+'}';
        if(j!=10) {
          result += ','
        }
      }
    result += ']} ';
    if(i!=10) {
    	result += ','
    }
  }

result += ']';

console.log(result);

console.log(JSON.parse(result));

Collectives™ on Stack Overflow

Getting error while Convert string to json object

4 Answers 4

3 Comments

Comments

1 Comment

Comments

Linked

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

3 Comments

Comments

1 Comment

Comments

Linked

Related