多重承诺的一个承诺 - 并发问题

时间:2016-05-06 21:55:53

标签: javascript node.js npm gulp

在我的“gulpfile.js”的方法“myMethod”中,我想创建多个Promises。数量取决于数组大小(方法参数)。当我打电话给方法时,我想确保在继续之前完成所有承诺。在所有承诺都实现之前,我宁愿回来。

请查看最后五行代码。

依赖

var promiseAll = require('gulp-all');
var del = require('del');
var deleteEmpty = require('delete-empty');

gulp-all | del | delete-empty

帮手方法

var oneForAllPromises = function(promises){
    var promAll = promiseAll(promises);
    promAll.then(function(param) {
        console.log('foo');
    }, function(err) {
        console.error('foo');
    });
    return promAll;
}

有问题的代码

var myMethod = function(array1, array2){
    var promise = del(array1, {force: true});
    promise.then(paths => {console.log('foo');});

    var promises = [];
    promise.then(()=>{
        for(var i=0; i<array2.length; i++){
            promises[i] = new Promise(function(resolve, reject) {
                deleteEmpty(array2[i], {force: true},
                    function(err, deleted){
                        if(err){
                            console.log('foo');
                            reject
                        }else{
                            console.log('foo');
                            resolve
                        }
                    }
                );
            });
        }
    });

    // PROBLEM: Returns empty promises array
    console.log("promiesesLENGTH: "+promises.length); // promiesesLENGTH: 0

    // Create one promise for all the promises
    return oneForAllPromises(promises);
}

1 个答案:

答案 0 :(得分:3)

在console.log时,第一个promise promise = del(array1, {force: true});尚未完成,因此then中的所有代码都没有执行。这就是为什么你的承诺是空的。

你可以简单地回到另一个承诺:

var myMethod = function(array1, array2){
    var promise = del(array1, {force: true});

    return promise.then(() => {
        return Promise.all(array2.map(array2value => {
            return new Promise(function(resolve, reject) {
                deleteEmpty(array2value, {force: true}, (err, deleted) => {
                    if (err) {
                        reject(err);
                    } else{
                        resolve()
                    }
                });
            });
        }
    });
}