Node.js using async.forEach - how to pass a list to the final function

Question

I am using the node async lib - https://github.com/caolan/async#forEach and would like to iterate through an array (arrayOne), push the result to another(productList), and pass that array to the final function. However the printed result is an empty array.

function Async(callback) {
    var productList = [];
    async.forEach(
        arrayOne,
        function(item, callback){
            getClientsNb({id:item["row"][0]},function (result) {
                productList.push(result)
            });
            callback();
        }, 
        function(err) {
            console.log(productList);
        }    
    );
}

the "getCLientsNb" function is a call to a Neo4j database.

I would like to understand why the result is an empty array. Since I am a beginner in node.js, please be very clear in your answer :)

is it even iterates inside getClientsNb callback? have you tried to put console.log(result) near productList.push(result) ? — llamerr
– llamerr, Commented Mar 12, 2016 at 15:03

Robert Moskal · Accepted Answer · 2016-03-12 16:50:15Z

1

Your problem is where you place your callback.

function(item, callback)         
 { getClientsNb({id:item["row"][0]},function 
    (result) { productList.push(result) });
    callback(); }

The outer function returns before the callback for getClientsNb() is executed, so you never build up your list. You need to move the callback inside that function:

function(item, callback){ getClientsNb({id:item["row"][0]},
    function (result) { 
            productList.push(result) 
            callback()});
}

If you used async.map instead of forEach, then you wouldn't need to create a productList. Just pass the result in the second parameter to the callback and it would be passed to the final function as the second parameter.

edited Mar 12, 2016 at 16:50

answered Mar 12, 2016 at 15:14

Robert Moskal

22.7k8 gold badges69 silver badges91 bronze badges

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

Collectives™ on Stack Overflow

Node.js using async.forEach - how to pass a list to the final function

1 Answer 1

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Related