JSON objects wont transfer into js array using ajax

Question

I get undefined when i try to console.log it

var farr = [];
$.ajax({
    url: "https://whispering-cliffs-33347.herokuapp.com/employees",
    type: "GET",
    contentType: "application/jsonp"
}).done(function(employees) {
    for(let i in employees){
        farr.push(employees[i]);
    }
})
console.log(farr[8]);

Any ideas?

First console `employees`. Know whether it is array or object. If it is array, you may need to parse the JSON. And use console in `.done` function. — Harish Kommuri, Feb 04 '18 at 05:24
Move the `console.log` call into your `.done()` callback. Remember that `$.ajax` is asynchronous and any return values or variables you're setting in the callback will be undefined outside of the callback. — user9263373, Feb 04 '18 at 05:27
Possible duplicate of [How do I return the response from an asynchronous call?](https://stackoverflow.com/questions/14220321/how-do-i-return-the-response-from-an-asynchronous-call) — Hassan Imam, Feb 04 '18 at 05:29
No, do not use `async:false`. It's deprecated, it's lazy and it's poor programming practice. — user9263373, Feb 04 '18 at 05:32

score 1 · Answer 1 · answered Feb 04 '18 at 05:26

console.log(farr[8]); will be executed even before the response is available.So in first done push all the elements in the local array & once that is done in the next done log the value

var farr = [];
    $.ajax({
        url: "https://whispering-cliffs-33347.herokuapp.com/employees",
        type: "GET",
        contentType: "application/jsonp"
    }).done(function(employees) {
        employees.forEach(function(item){
            farr.push(item)
        })
    }).done(function(elem){
        console.log(farr[8]);
    })

score 0 · Answer 2 · answered Feb 04 '18 at 05:27

You can not iterate the object with standard for loop. To be able to do that, you should first get object keys in an array and iterate over that.

const keys = Object.keys(employees);
keys.forEach((i) => {
     farr.push(employees[i]);
 }
 console.log(farr[8]);. // You should put this in the call back itself

Or you can directly iterate over the object using lodash's forEach.

JSON objects wont transfer into js array using ajax

2 Answers2