Promise.all() 和 .map 的问题，意外结果

Question

我已经阅读了有关承诺的所有内容，我从多个线程测试了多个解决方案，但我仍然无法让它工作

我正在尝试遍历文件夹中的文件，获取 xml 内容，使用 xml-js 模块将其转换为 js 对象，然后将它们添加到数组中。 这是我最后一次尝试（大约 30 次中的一次）

fs.readdir(folderPath, async(err, files) => {
    let xmlArray = await Promise.all(
        files.map((file) => {
            fs.readFile(filePath, 'utf8', async(err, data) => {
                return await xmljs.xml2js(data , {compact : true).RequestFileHatasDM;
            })
        })
    )
    console.log(xmlArray);
})

我错过了什么？ 我认为 readdir 被调用，然后回调返回，这就是我等待 Promise.all 的地方，然后使用 .map 迭代所有内容并返回值，该值应该（？）返回并承诺解决。

然后在一切都解决之后，它才应该继续到console.log()，但由于某种原因，当我运行服务器时它立即跳转到console.log()，好像所有的promise都已经解决了！

将 console.log() 放入 readFile 方法时，它仍在运行。

我很困惑，谁能指出我的错误？

Answer 1

这里有几件事。 First Promise.all 接受一系列你没有给它的承诺。在您的 files.map 中，您不会在该地图回调函数中返回任何内容。所以试试这个：

files.map((file) => {
        return fs.readFile(filePath, 'utf8', async(err, data) => {
            return await xmljs.xml2js(data , {compact : true).RequestFileHatasDM;
        })
    })

此外，当您在异步函数中返回承诺时，您可以这样做：

return xmljs.xml2js()

代替：

return await xmljs.xml2js()

很容易错过。我鼓励您设置一个调试环境，以便您可以单步执行该函数并观察代码中的不同值。它确实可以为您节省一些调试代码的时间。

我喜欢 VSCodes 调试器：

https://code.visualstudio.com/docs/nodejs/nodejs-debugging

Answer 2

这种代码可能非常令人困惑。它可以更轻松地创建一个返回 Promise 并接受文件路径作为输入的函数。

我们可以使用Util.Promisify 来创建fs.readFile 的promisified 版本，生成的代码更容易阅读，也更紧凑。

然后我们可以在通过这个函数映射文件路径的结果上使用 Promise.all。

例如：

const fs = require('fs');
const path = require('path')
const xmljs = require('xml-js');
const { promisify } = require ('util');

const readFilePromise = promisify(fs.readFile);
const readDirPromise = promisify(fs.readdir);

// Replace as appropriate.
const folderPath = 'test_dir';

async function xmlFile2js(path) {
    const data = await readFilePromise(path, { encoding: 'utf-8'});
    return xmljs.xml2js(data, { compact: true});
}

async function readXmlFiles(folderPath) {
    try {
        const files = await readDirPromise(folderPath);
        const filePaths = files.map(file => path.join(folderPath, file));
        const promiseArray = filePaths.map(filePath => xmlFile2js(filePath));
        const xmlArray = await Promise.all(promiseArray);
        return xmlArray;
    } catch (err) {
        console.error("readXmlFiles: An error occurred:", err);
    }
}

async function testReadXmlFiles() {
    const xmlArray = await readXmlFiles(folderPath);
    console.log("testReadXmlFiles: xmlArray:", JSON.stringify(xmlArray, null, 4));
}

testReadXmlFiles();