Question

189

node.js async.series is that how it is supposed to work?

rated 0 times [ 192] [ 3] / answers: 1 / hits: 65383 / 11 Years ago, fri, april 12, 2013, 12:00:00

var async = require('async');



function callbackhandler(err, results) {

    console.log('It came back with this ' + results);

}   



function takes5Seconds(callback) {

    console.log('Starting 5 second task');

    setTimeout( function() { 

        console.log('Just finshed 5 seconds');

        callback(null, 'five');

    }, 5000);

}   



function takes2Seconds(callback) {

    console.log('Starting 2 second task');

    setTimeout( function() { 

        console.log('Just finshed 2 seconds');

        callback(null, 'two');

    }, 2000); 

}   



async.series([takes2Seconds(callbackhandler), 

              takes5Seconds(callbackhandler)], function(err, results){

    console.log('Result of the whole run is ' + results);

})

The output looks like below :

Starting 2 second task

Starting 5 second task

Just finshed 2 seconds

It came back with this two

Just finshed 5 seconds

It came back with this five

I was expecting the takes2Second function to finish completely before the takes5Second starts.
Is that how it is supposed to work. Please let me know.
And the final function never runs. Thanks.

Answers

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

diontajm

Add To Favorites

Follow

Total Points: 391

Total Questions: 104

Total Answers: 104

Location: Netherlands

Member since Mon, Jun 7, 2021

3 Years ago

answered 11 Years ago janjadonb · Accepted Answer

Not quite. You are executing the functions immediately (as soon as the array is evaluated), which is why they appear to start at the same time.

The callback passed to each of the functions to be executed is internal to the async library. You execute it once your function has completed, passing an error and/or a value. You don't need to define that function yourself.

The final function never runs in your case because the callback function that async needs you to invoke to move on to the next function in the series never actually gets executed (only your callbackHandler function gets executed).

Try this instead:

async.series([

    takes2Seconds,

    takes5seconds

], function (err, results) {

    // Here, results is an array of the value from each function

    console.log(results); // outputs: ['two', 'five']

});