在Promise的.then方法中使用fs.readFileSync不生效该如何解决

问题原因 #

• 核心错误：fs.readFileSync 是Node.js提供的同步文件读取API，不支持传入回调函数，你写的回调逻辑永远不会被触发。同步API会直接阻塞线程返回文件内容，只有异步的fs.readFile才需要传入回调处理结果。
• 语法错误：.then() 需要接收一个函数作为回调参数，你当前直接执行了items.forEach(...)()，相当于把forEach执行后的返回值（undefined）传给了.then，执行逻辑完全不符合预期。
• 写法冗余：你已经用了async/await语法管理异步流程，没有必要再混用.then()回调，徒增逻辑复杂度。

修正后代码 #

const fs = require('fs');
// 确保你已经引入了其他依赖：parser、downloadFile、generateChecksum、deleteFile

(async () => {
    const feed = await parser.parseURL('https://www.nasa.gov/rss/dyn/breaking_news.rss');
    console.log(feed.title);
    const items = [];

    // 等待所有异步任务执行完成
    await Promise.all(feed.items.map(async (currentItem) => {
      // some code here to create data
      items.push(data);
    }));

    // 遍历处理所有item
    for (const element of items) {
      const file = downloadFile(element.url);
      let checksumValue;
      try {
        // 同步读取直接返回内容，不需要回调
        const data = fs.readFileSync(file);
        checksumValue = generateChecksum(data);
        console.log(`The checksum is: ${checksumValue}`);
        // 删除下载的文件
        deleteFile(file);
      } catch (error) {
        console.error(error);
      }
    }
})();

如果你更习惯用异步API的写法，也可以把fs.readFileSync换成fs.promises.readFile配合await使用：

// 遍历部分修改为异步读取写法
for (const element of items) {
  const file = downloadFile(element.url);
  let checksumValue;
  try {
    const data = await fs.promises.readFile(file);
    checksumValue = generateChecksum(data);
    console.log(`The checksum is: ${checksumValue}`);
    deleteFile(file);
  } catch (error) {
    console.error(error);
  }
}

提示：如果你的downloadFile方法是异步返回文件路径的，需要在前面加await等待下载完成再执行后续读取逻辑。

内容的提问来源于stack exchange，提问作者Nooshin