在 Array.forEach 块中的所有承诺都已解决后返回

return after all promises in Array.forEach block has been resolved

本文关键字:承诺 解决 返回 forEach Array      更新时间:2023-09-26

我正在一个简单的例子中尝试nodejs中的es6-promisify模块,类似于es6-promisify的npm页面上的示例。但是我想使用 forEach 循环来检查条目的统计信息,如果它们是目录,则将它们推送到单独的数组:

var fs=require('fs');
var promisify=require('es6-promisify');
var path='../test_folder/';
function listSubfolderNamesSync(fpath){
    var arrayOfFolders=[];
    var array=fs.readdirSync(fpath);
    array.forEach(function(element,index,array){
        var stat=promisify(fs.stat);
        stat(path+element).then(function(stats){
            if (stats.isDirectory()){
                arrayOfFolders.push(element);
                console.log('pushed');
            }
        })
    });
    return arrayOfFolders;
}
console.log(listSubfolderNamesSync(path));

对于 2 个子文件夹,输出将为:

[]
pushed
pushed

我知道为什么我的代码不能像我想要的那样工作,但我无法弄清楚如何优雅地解决这个问题。我已经看到了带有Promise.all()的代码示例和Array.map()的使用,以拥有一系列承诺,并等待所有这些承诺解决,然后再在链中前进。我看到的问题是,我不能/不想在每次检查统计数据并将这些承诺放入数组中以使用 Promise.all(array) 时显式创建一个承诺。我觉得该解决方案需要相当多的解决方法,并且会大大降低代码的可读性。也许最好在forEach循环中使用它,这是我在我想尝试承诺之前的工作代码:

if(fs.statSync(path+element).isDirectory()){
        arrayOfFolders.push(element);
}

可能是因为我缺乏编程经验,但似乎混合承诺、回调和同步代码并不像书上写的那么简单?

> 由于stat()返回一个承诺,因此您的listSubfolderNamesSync()函数也必须返回,使其listSubfolderNamesAsync()。除非有stat()的同步替代方案,否则这是不可避免的。

listSubfolderNamesAsync() 中,您需要将 fs.readdirSync() 返回的数组映射到一个承诺数组,然后使用 Promise.all() 将这些承诺(以及它们提供的数据)聚合到一个承诺中。

var fs = require('fs');
var promisify = require('es6-promisify');
var stat = promisify(fs.stat);
var path = '../test_folder/';
function listSubfolderNamesAsync(fpath) {
    var promises = fs.readdirSync(fpath).map(function(element) {
        return stat(path + element).then(function(stats) {
            return stats.isDirectory() ? element : null;
        });
    });
    return Promise.all(promises).then(function(results) {
        return results.filter(function(res) {
            return res !== null;
        });
    });
}

现在使用.then()处理结果:

listSubfolderNamesAsync(path).then(function(folders) {
    console.log(folders);
});

使用返回单独的文件夹文件列表的更通用函数不会对性能造成很大影响。你可以写这样的东西:

function listSubElementsAsync(fpath) {
    var promises = fs.readdirSync(fpath).map(function(element) {
        return stat(path + element).then(function(stats) {
            var type = stats.isDirectory() ? 'folder' : 'file';
            return {type:type, element:element};
        });
    });
    return Promise.all(promises).then(function(results) {
        var files = results.filter(obj => obj.type === 'file').map(obj => obj.element);
        var folders = results.filter(obj =>  obj.type === 'folder').map(obj => obj.element);
        return {files:files, folders:folders};
    });
}
listSubElementsAsync(path).then(function(list) {
    console.log(list.files);
    console.log(list.folders);
});