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
– Harish Kommuri, Commented Feb 4, 2018 at 5: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
– user9263373, Commented Feb 4, 2018 at 5:27
Possible duplicate of How do I return the response from an asynchronous call? — Hassan Imam
– Hassan Imam, Commented Feb 4, 2018 at 5:29
No, do not use async:false. It's deprecated, it's lazy and it's poor programming practice. — user9263373
– user9263373, Commented Feb 4, 2018 at 5:32

brk · Accepted Answer · 2018-02-04 05:26:25Z

1

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]);
    })

answered Feb 4, 2018 at 5:26

brk

50.3k10 gold badges59 silver badges85 bronze badges

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

Comments

Aaditya Thakkar · Accepted Answer · 2018-02-04 05:27:35Z

0

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.

answered Feb 4, 2018 at 5:27

Aaditya Thakkar

1,81015 silver badges14 bronze badges

Collectives™ on Stack Overflow

JSON objects wont transfer into js array using ajax

2 Answers 2

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related