JavaScript array not iterable

Question

I hit a really weird issue with a simple JavaScript array last night. I am working on a React Native app powered by Firebase and getting data from the real-time database with the SDK.

Here is the code:

    var app = this

    this.props.fb.auth().onAuthStateChanged(function(user) {
        if (user) {
            app.props.fb.database().ref('/users/' + user.uid + "/timeline").once('value').then(function(snapshot) {
                var posts = snapshot.val()
                return posts
            }).then((posts) => {                
                var statuses = [];
                for (i in posts) {

                    app.props.fb.database().ref('/posts/' + posts[i]).once('value').then(function(snapshot) {
                       statuses.push(snapshot.val())
                    });
                }

                console.log(statuses)
            })
        }

    });

The above code is supposed to get the data from the timeline of each user, iterate through each of the posts on the timeline and then get the data of the post from posts. It then simply pushes the data to the statuses array, which is being console logged at the end.

This is what shows up on the console.

Until the array is expanded, the console doesn't show the items in the array. Additionally, when I try to get the length of the array, it returns 0 and I also can't iterate through the items on the array.

What am I doing wrong?

99/100 JS errors seem to be async issues - this looks the same. u cannot log in the console an array that is being populated by an async call. — Sten Muchow
– Sten Muchow, Commented Oct 25, 2017 at 10:31
You shouldn't iterate an array using a for..in construct. Use Array.prototype.forEach(). — Phylogenesis
– Phylogenesis, Commented Oct 25, 2017 at 10:34

Sten Muchow · Accepted Answer · 2017-10-25 10:45:35Z

2

If posts is an iterable I would look into using some version of q.all and then when all promises are resolved you can reliably log the result of push all those pieces into the statuses array.

some pseudo code:

if (user) {
        app.props.fb.database().ref(`/users/${user.uid}/timeline`)
        .once('value')
        .then(snapshot => snapshot.val())
        .then(posts => {                
            const statuses = [];

            const promises = Object.keys(posts).map(key => {
                return app.props.fb.database().ref(`/posts/${posts[key]}`).once('value')
                    .then(snapshot => statuses.push(snapshot.val()));
            }

            return Promises.all(promises)
        })
        .then(() => console.log(statuses));
    }

edited Oct 25, 2017 at 10:45

answered Oct 25, 2017 at 10:33

Sten Muchow

6,7314 gold badges38 silver badges46 bronze badges

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

4 Comments

hackerman Over a year ago

How would that work? I am kinda new to this so would be really helpful if you could show me an example

Sten Muchow Over a year ago

there ya go... seems we have Promises.all to work with -> developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/…

hackerman Over a year ago

Thank you so much!

Sten Muchow Over a year ago

I added some es6-ification of your code as well, makes it a bit sexier with template strings, arrows functions, auto return statements ;)

Collectives™ on Stack Overflow

JavaScript array not iterable

1 Answer 1

4 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

4 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related