Question

76

Can't acess variable outside forEach loop JavaScript

rated 0 times [ 83] [ 7] / answers: 1 / hits: 6379 / 4 Years ago, fri, september 18, 2020, 12:00:00

I want to get the allItems variable outside the function, but I just can get an empty array. The first console.log shows the array I wanted, but the second it's just [].

var allItems = [];    

orderPads.forEach(async  element => {

        

        element.orders_fk.forEach(async elem => {

            

            let itemsArray = await OrderedItem.find({ order_fk: elem });

            

            

            itemsArray.forEach(async e => {

                 

                let temp = e.toObject();

                const mesa = await Table.findById(element.mesa);

                temp.mesa = mesa.name;

                

                allItems.push(temp)

                console.log(allItems)

                

            })

        })

    });

console.log(allItems)

I've already looked for answers online, but nothing seems to work in this case. Thank you for the help!

Answers

Only authorized users can answer the question. Please sign in first, or register a free account.

agustindejonm

Add To Favorites

Follow

Total Points: 738

Total Questions: 84

Total Answers: 84

Location: Northern Ireland

Member since Mon, Nov 14, 2022

2 Years ago

agustindejonm questions

1 Discord.js how to fetch message by content

Fri, Jun 25, 21, 00:00, 3 Years ago

1 Security warning in the console of BrowserWindow (electron ^9.2.0)

Sat, Aug 15, 20, 00:00, 4 Years ago

1 Filter array of objects based on the input passed: Javascript

Wed, Jul 15, 20, 00:00, 4 Years ago

1 how to refresh component in react js

Sat, May 16, 20, 00:00, 4 Years ago

1 How to use proxy in package.json in next.js?

Thu, May 14, 20, 00:00, 4 Years ago

View All

answered 4 Years ago marcos · Accepted Answer

Async functions don't do what you think they do in forEach. Foreach will wait for every loop, but since it won't return a Promise, you can't actually wait for all loops (the forEach) to be done - the method will just return the current null immediately. Then you log that once. You might notice that your last log is logged first, and your others later, indicating that forEach was completed, but the loops completed later, and your array only filled up after your first log.

Personally, I would suggest using a for loop itself instead, which does not actually take you out of the async flow:

const allItems = [];   



for( const element of orderPads ){

for( const elem of element.orders_fk ){



  const itemsArray = await OrderedItem.find({ order_fk: elem });

  

  for( const e of itemsArray ){

  

    const temp = e.toObject();

    const mesa = await Table.findById(element.mesa);

    temp.mesa = mesa.name;



    allItems.push(temp)

    console.log(allItems)

    

  }

  

}}



console.log(allItems)

for loops are core constructs in Javascript, while forEach is in essence a simplistic wrapper for a for loop in Arrays that was never intended for asynchronous use.