Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

Waiting for multiple callbacks in Node.js

I have a Node.js application where multiple funcions might be called, depending on several factors, but only one last function is called after the last callback.

This is a simplified version of what I got:

if(foo === bar){
    function1(arg1, function(val1){
        doWhatever(val1, function(){
            res.end("Finished");
        });
    });
}else if(foo === baz){
    function2(arg2, function(val2){ 
        doWhatever(val2, function(){
            res.end("Finished");
        });
    });
}else{
    function3(arg3, function(val3){
        doWhatever(val3, function(){
            res.end("Finished");
        });
    });
}

And this is what im doing:

var finished = false;

if(foo === bar){
    function1(arg1, function(val1){
        result = val1;
        finished = true;
    });
}else if(foo === baz){
    function2(arg2, function(val2){ 
        result = val2;
        finished = true;
    });
}else{
    function3(arg3, function(val3){
        result = val3;
        finished = true;
    });
}

var id = setInterval(function(){
    if(finished === true){
        clearInterval(id);
        doWhatever(result, function(){
            res.end("Finished");
        });
    }
}, 100);

I guess this can be simplified by using promises, however im not sure how should I implement them.

like image 290
user435943 Avatar asked Mar 18 '14 16:03

user435943


2 Answers

You could also do it using when and promises, which IMHO is the easiest to read.

var promises = [];

if(x) {
    var deferred1 = when.defer();
    doSomethingAsync({ callback: deferred1.resolve });
    promises.push(deferred1.promise);
} else if(y) {
    var deferred2 = when.defer();
    doSomethingAsync({ callback: deferred2.resolve });
    promises.push(deferred2.promise);
} else if(z) {
    var deferred3 = when.defer();
    doSomethingAsync({ callback: deferred3.resolve });
    promises.push(deferred3.promise);
}

when.all(promises).then(function () {
    console.log('Finished Promises');
});
like image 174
jgillich Avatar answered Oct 11 '22 15:10

jgillich


Here's one way with async series.

https://github.com/caolan/async#series

async.series([
    function(callback){
        if(foo === bar){
            function1(arg1, function(val1){
                callback(null, val1);
            });
        }else if(foo === baz){
            function2(arg2, function(val2){ 
                callback(null, val2);
            });
        }else{
            function3(arg3, function(val3){ 
                callback(null, val3);
            });
        }
    }  
], function(error, valArray){
       doWhatever(valArray[0], function(){
           res.end("Finished");
       });
});
like image 29
aembke Avatar answered Oct 11 '22 13:10

aembke