javascript - Promise.all() 解析，但它不应该(node.js)

Question

我现在正在 node.js 中尝试这样的事情:

var exec = require('child_process').exec
var write = require('fs-writefile-promise')

function run() {
    var myArray = [];   
    var execs = [];

    for (var i = 1; i <= 7; i++) {
        (function(cntr) {
            write('file-' + i + '.txt', someString)
            .then(function (filename) {
                execs.push(new Promise(function(resolve, reject) {
                    exec('cat ' + 'file-' + cntr + '.cnf', function(error, stdout, stderr) {
                        console.log(cntr + ' ' + stdout);
                        if (stdout.search(/\bsomeString\b/) > -1) {
                            myArray.push(cntr);
                            resolve();
                        } 
                        else {
                            resolve();
                        }   
                    })
                }))
            })
            .catch(function (err) {
                console.error(err);
            });
        })(i);
    }
    return Promise.all(execs).then(function() {
        return new Promise(function(resolve) {
            resolve(myArray);
        }) 
    })
}

run().then(function(result) {
    console.log(result);
});

如您所见，我正在创建多个运行 exec() 的 Promise，并且每个 Promise 在 exec() 完成时解析。然后我等待每个 Promise 在 Promise.all(execs) 中解析以返回 myArray 作为 Promise。然而，当我最后执行 run() 函数时，它返回一个空数组。我想这与 Promise.all() 有关，因为即使 execs 中的某些 Promise 尚未解决，它也会解决，但我不确定，就是这样为什么我真的需要一些帮助。有谁知道我在代码中哪里犯了错误？

非常感谢您!

#编辑 1

var exec = require('child_process').exec
var write = require('fs-writefile-promise')

function run() {
    var myArray = [];   
    var execs = [];

    for (var i = 1; i <= 7; i++) {
        (function(cntr) {
            return new Promise(function(resolve, reject) {
                fs.writeFile('file-' + i + '.txt', someString, (err) => {
                    if (err) {
                        reject();
                    }
                    else {
                        resolve();
                    }
                });
            })
            .then(function (filename) {
                execs.push(new Promise(function(resolve, reject) {
                    exec('cat ' + 'file-' + cntr + '.cnf', function(error, stdout, stderr) {
                        console.log(cntr + ' ' + stdout);
                        if (stdout.search(/\bsomeString\b/) > -1) {
                            myArray.push(cntr);
                            resolve();
                        } 
                        else {
                            resolve();
                        }   
                    })
                }))
            })
            .catch(function (err) {
                console.error(err);
            });
        })(i);
    }
    return Promise.all(execs).then(function() {
        return new Promise(function(resolve) {
            resolve(myArray);
        }) 
    })
}

run().then(function(result) {
    console.log(result);
});

Answer 1

您的两次尝试都存在很多问题。第一次尝试的问题是您在异步操作后填充 execs 数组，因此当您实际将数组传递给 Promise.all() 时它什么也没有，因此 Promise .all() 无需等待。

此外，您不只是在使用已经创建的 promise ，因此您最终会做出比需要更多的 promise 。

一般来说，最好是在你的主要逻辑之外“ promise ”你的异步操作，然后让你的所有逻辑都是 promise 驱动的，而不是将 promise 与普通回调混合和匹配。这是一个试图解决这些问题的版本:

var exec = require('child_process').exec
var write = require('fs-writefile-promise')

// make promisified version of exec
function execP(file, options) {
    return new Promise(function(resolve, reject) {
        exec(file, options, function(err, stdout, stderr) {
            if (err) return resolve(err);
            resolve({stdout: stdout, stderr: stderr});
        });
    });
}

function run() {
    var promises = [];
    for (var i = 1; i <= 7; i++) {
        promises.push(write('file-' + i + '.txt', someString).then(function(filename) {
            return execP(filename);
        }));
    }
    return Promise.all(promises).then(function(results) {
        // results is an array of {stdout: xxx, stderr: yyy} objects
        // process those results into a new array of just indexes
        var final = [];
        results.forEach(function(data, index) {
            if (data.stdout.search(/\bsomeString\b/) > -1) {
                final.push(index);
            }
        });
        return final;
    });
}

run().then(function(results) {
    // array of indexes that contained the desired search string
}, function(err) {
    // process error here
});

注意:这将并行运行您的所有 exec 操作，这是您的原始代码所做的。如果您想按顺序运行它们，也可以这样做，但需要进行一些调整。