Node.js跨模块Promise仅首次生效、无法同步数据库最新数据的问题求解

你遇到的结果固定、不随数据库更新的问题，本质是对Node.js模块执行时机和Promise状态特性的误解：

• 你在General.js的顶层代码里直接实例化了personInfo这个Promise。Node.js的模块在第一次被require引入时，会立刻执行文件内所有顶层代码，这个new Promise(...)包裹的数据库查询逻辑，只会在服务启动、模块首次加载的时候跑一次。
• Promise一旦进入fulfilled或者rejected的终态，后续就不会再重新执行内部逻辑，你导出的这个Promise实例会永久保存第一次查询拿到的结果，后续所有调用拿到的都是同一个已经定态的Promise，自然不会随数据库数据变化更新。
• 额外的小问题：你在Account.js的路由处理函数内部写require('./general')其实完全没必要，Node.js有模块缓存机制，多次require拿到的是同一个模块实例，根本不会重新加载文件内容。

不要直接导出已经实例化的Promise对象，要导出一个每次被调用都会重新执行数据库查询、返回全新Promise的普通异步函数，从根源上保证每次请求都能拿到最新的数据库数据。

改造后的General.js代码 #

const Person = require('../models/person');

// 导出可复用的查询函数，每次调用都会生成全新的Promise
module.exports.getPreGroupPerson = () => {
  // 函数被触发时才执行查询，不会在模块加载时提前运行
  return Person.find({ Group: 'pre' })
    .then((result) => {
      if (result.length === 0) {
        // 数据为空时主动抛出错误，让调用方感知异常
        throw new Error('未查询到Group为pre的人员数据');
      }
      return result;
    })
    .catch((err) => {
      console.error('查询pre分组人员数据异常：', err);
      // 一定要把异常向上抛出，不然调用方的catch逻辑根本触发不了
      throw err;
    });
};

改造后的Account.js代码 #

// 模块引入放到文件顶层即可，不需要写在路由处理函数内部
const preGroupPerson = require('./general');

exports.Ac = async (req, res, next) => {
  try {
    // 每次收到账号页面的请求，都重新调用查询函数拿最新数据
    const latestPersonList = await preGroupPerson.getPreGroupPerson();
    return res.render('/myPage/Account', {
      title: 'Account',
      group: latestPersonList
    });
  } catch (err) {
    // 把异常交给全局错误处理中间件，不要只打印日志
    next(err);
  }
};

• 绝对不要把模块加载阶段实例化的Promise直接当导出值：这种写法相当于把第一次查询的结果永久隐式缓存了，完全不符合动态查询的需求。如果确实需要加查询缓存，要在函数内部做可控的缓存逻辑，比如设置缓存过期时间、提供手动刷新缓存的方法，不能依赖模块的单次执行特性做隐式缓存。
• Promise的catch块不是打完日志就完事了：如果catch没有把捕获到的错误重新抛出，后续的调用链是感知不到查询失败的，会出现“查询出错了但页面照常渲染空数据”的诡异问题。
• 不要在请求处理函数里重复写require：这种写法除了让代码变乱，没有任何实际收益，Node.js的模块加载缓存会直接返回第一次加载的模块实例，根本不会重新执行文件内容。
• 原代码里的new require('./general')属于无意义写法：require是Node.js提供的模块加载函数，不是构造函数，加new完全没有作用，还会增加额外的语法解析开销。

内容的提问来源于stack exchange，提问作者Reşit Abdullah Yavuzkol