problem parsing JSON Strings

Question

var records = JSON.parse(JsonString);

for(var x=0;x<records.result.length;x++)
{
  var record = records.result[x];
  ht_text+="<b><p>"+(x+1)+" "
    +record.EMPID+" "
    +record.LOCNAME+" "
    +record.DEPTNAME+"  "
    +record.CUSTNAME
    +"<br/><br/><div class='slide'>"
    +record.REPORT
    +"</div></p></b><br/>";
}

The above code works fine when the JsonString contains an array of entities but fails for single entity. result is not identified as an array! Whats wrong with it?

http://pastebin.com/hgyWw5hd

Mickael Lherminez · Accepted Answer · 2020-01-05 19:07:07Z

2

result is not an array. Do you see any square brackets in your JSON? no you do not. it does not contain any arrays.

{
  "result": {
    "ID": "30",
    "EMPID": "1210308550",
    "CUSTID": "1003",
    "STATUS": "2",
    "DATEREPORTED": "1273234502",
    "REPORT": "this is one more report!",
    "NAME": "Sandeep Savarla",
    "CUSTNAME": "Collateral",
    "LOCID": "4",
    "LOCNAME": "Vijayawada",
    "DEPTNAME": "SALES"
  }
}

Can you show me what your "valid" json looks like when the function above works?

edited Jan 5, 2020 at 19:07

Mickael Lherminez

6951 gold badge11 silver badges30 bronze badges

answered May 7, 2010 at 16:38

mkoryak

58.1k64 gold badges204 silver badges262 bronze badges

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

1 Comment

SLaks Over a year ago

It works for that because that is an array. If it's not an array, you can't loop over it.

Peter Bailey · Accepted Answer · 2010-05-07 16:43:14Z

1

Just make sure it's an array before you iterate

if ( 'undefined' == typeof records.result.length )
{
  records.result = [records.result];
}

answered May 7, 2010 at 16:43

Peter Bailey

106k32 gold badges186 silver badges206 bronze badges

Comments

SLaks · Accepted Answer · 2010-05-07 16:41:01Z

0

Your code is looping through records.result as if it were an array.
Since it isn't an array, your code does not work.

This simplest solution is to force it into an array, like this:

var array = 'length' in records.result ? records.result : [ records.result ];

for(var x = 0; x < array.length; x++) {
    var record = array[x];
    ...

answered May 7, 2010 at 16:41

SLaks

891k182 gold badges1.9k silver badges2k bronze badges

Comments

Mickael Lherminez · Accepted Answer · 2020-01-05 19:04:18Z

0

In your code result is an object, not an array. Wrap it's value in square brackets to make it an array:

{
  "result": [{
    "ID": "30",
    "EMPID": "1210308550",
    "CUSTID": "1003",
    "STATUS": "2",
    "DATEREPORTED": "1273234502",
    "REPORT": "this is one more report!",
    "NAME": "Sandeep Savarla",
    "CUSTNAME": "Collateral",
    "LOCID": "4",
    "LOCNAME": "Vijayawada",
    "DEPTNAME": "SALES"
  }]
}

edited Jan 5, 2020 at 19:04

Mickael Lherminez

6951 gold badge11 silver badges30 bronze badges

answered May 7, 2010 at 16:42

Matt

75.4k26 gold badges156 silver badges181 bronze badges

Collectives™ on Stack Overflow

problem parsing JSON Strings

4 Answers 4

1 Comment

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

1 Comment

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related