在nodejs/express 中，如何防止 if 条件在 readdir 完成之前执行？

Question

app.get('/students/:record_id', function (req, res) {
  var found =false;
  var record_id = req.params.record_id;
  var dirname = "students/";


  filenames = fs.readdirSync(dirname);
  filenames.forEach(filename => {
    
    fs.readFile(dirname + filename, "utf-8", function (err, data ) {
      if (err) {
        onError(err);
        return;
      }
      if (data.includes(record_id)) {
        found = true;
        return res.status(200).send(data);
        
      }
    });
  });

  if(found == false){ 
    return res.status(500).send({ "message": "error - no student found" });
  }

});

我正在使用 GET 按学生姓名搜索来检索学生文件。如果我删除 if 条件，则成功。但如果学生不存在，那么它就不起作用。 if 条件通过在文件中找到 find 时将其设为 true 来修复此问题。问题在于 if 条件在读取文件之前执行。 我尝试使用回调，但经过几个小时的研究，我无法弄清楚如何在其中实现它，因为 readfile 回调用于检索学生信息。

我尝试使用承诺，但只有找到学生时，承诺才会兑现，而我不知道如何在找不到学生时兑现承诺。

Answer 1

既然您已经尝试过使用 Promises，我将向您展示如何使用 Promises

使用 async/await 尤其使这变得非常简单

注意：因为我不知道您如何使用 fs 导入/require 只获取 readdir 和 readFile 的承诺版本，并像 readdir 而不是 fs.readdir 使用它 - 所以其他现有代码不会破坏

const {readFile, readdir} = require('fs/promises');

// if you use import (modules) instead of require (commonjs)
// import { readFile, readdir } from 'fs/promises';

app.get('/students/:record_id', async function (req, res) {
    var record_id = req.params.record_id;
    var dirname = "students/";

    try {
        const filenames = await readdir(dirname);
        for (let filename of filenames) {
            const data = await readFile(dirname + filename, "utf-8");
            if (data.includes(record_id)) {
                return res.status(200).send(data);
            }
        }
        res.status(404).send({"message": "error - student not found"});
    } catch (err) {
        onError(err);
        res.status(500).send({"message": "internal server error"});
    }
});

注意：我不会'发送 500 （内部服务器错误）如果找不到学生 - 我更改了逻辑以发送 404（未找到） - 这更合适

如果您想要纯粹的承诺（无异步/等待） - 我相信以下是一种方法它，可能不是最好的代码，但它应该可以工作（虽然还没有测试过）

const {readFile, readdir} = require('fs/promises');

// if you use import (modules) instead of require (commonjs)
// import { readFile, readdir } from 'fs/promises';

app.get('/students/:record_id', function (req, res) {
    var record_id = req.params.record_id;
    var dirname = "students/";
    
    let p = Promise.resolve(false);
    for (let filename of filenames) {
        p = p
        .then(found => 
            found ? found : readFile(dirname + filename, "utf-8")
            .then(
                data => 
                    data.includes(record_id) ? data : false
            )
        );
    }
    .then(data => data ? res.status(200).send(data) : res.status(404).send({"message": "error - student not found"}))
    .catch(err => {
        onError(err);
        res.status(500).send({"message": "internal server error"});
    })
});

并且，最后 - 根本不使用 Promises

app.get('/students/:record_id', function (req, res) {
    let found = false;
    const record_id = req.params.record_id;
    const dirname = "students/";

    const filenames = fs.readdirSync(dirname);
    const count = filenames.length;
    const checkFile = index => {
        if (index < count) {
            const filename = filenames[index];
            fs.readFile(dirname + filename, "utf-8", function (err, data) {
                if (err) {
                    onError(err);
                    res.status(500).send({"message": "internal server error"});
                } else if (data.includes(record_id)) {
                    res.status(200).send(data);
                } else {
                    checkFile(index + 1)
                }
            });
        } else {
            res.status(404).send({"message": "error - student not found"});
        }
    }
    checkFile(0);
});