CODEWITHSUNDEEP
javascriptjson
Mathias Rød
Mathias Rød

Reputation: 83

Unexpected token u in JSON at position 0 (but only sometimes)

Ok, so this is just a part of my code, and actually works as it's supposed to.

var jqxhr = $.getJSON( "main.json", function(data) {
    return data;
});

var json;
window.onload = function() {
    var jsonTxt = jqxhr.responseText;
    json = JSON.parse(jsonTxt);
    ....
}

But every 10th attempt or so i get the following error:

Uncaught SyntaxError: Unexpected token u in JSON at position 0
at JSON.parse (<anonymous>)
at window.onload (profile.js:8)

It's really annoying, because it will leave my page blank. I do think it has something to do with me trying to parse the JSON wrong - or something. I really don't know and I'm looking forward to any kind of answer that could lead me to the fix. Thank you.

Upvotes: 6

Views: 56448

Answers (2)

Christian4423
Christian4423

Reputation: 1796

Insert a try catch block where the JSON.parse() is called.

It is possible that the text itself is not JSON.

For example...

I think this is the issue because if response text is undefined, the char 'u' is at position 0. So JSON.parse() is working with a string. Not a javascript-object-notation text file. 

if(response !== undefined){
    try{
    // do parse
    }
    catch(errror){
        console.error("Not a JSON response")
    }
}

Upvotes: 2

M&#225;t&#233; Safranka
M&#225;t&#233; Safranka

Reputation: 4116

That unexpected "u" is the first letter of the string "undefined". It happens because your two asynchronous operations (i.e. loading the JSON and loading the window) are racing to completion, and if the JSON isn't loaded fast enough, the window.onload method will attempt to parse the JSON string that isn't loaded yet.

A solution is to move the JSON loading inside the onload even handler. Additionally, the whole point of getJSON is that it already parses the response for you as a JSON object, so there is no need for JSON.parse():

window.onload = function() {
    $.getJSON( "main.json", function(json) {
        // process the results here
    });
}

Upvotes: 17

Related Questions