Node.js两个文件互相require是否正常？有什么可替代的解决方案？

循环require实现的合理性判定 #

Node.js的CommonJS模块规范虽然原生支持循环引用，但你当前的实现是不推荐的不合理写法，存在两个核心问题：

• 存在时序风险：模块加载阶段如果AWS.js先引入还未执行完导出逻辑的index.js，会拿到空的导出对象，访问index.sns_NameSpace会直接得到undefined，触发预期外的逻辑错误，这类问题偶现且排查成本很高
• 模块强耦合：两个模块互相依赖对方的内部变量，后续任意一方的变量改动都会直接影响另一方，维护成本极高

更优解决方案 #

你当前的场景本质是AWS.js的逻辑依赖index.js中的运行时变量，只需要将AWS.js的逻辑封装为可接收入参的函数，调用时由index.js传入对应变量即可，完全不需要循环引用，改造后的代码如下：

改造后AWS.js代码 #

// 将依赖sns参数的逻辑封装为可入参的函数
function getKeyFilter(sns_NameSpace) {
  if(sns_NameSpace == "AWS/S3") {
    return ["BucketName", "StorageType"]
  }
  // 可补充其他命名空间对应的返回逻辑，或设置默认返回值
  return []
}

// 导出方法供外部调用
exports.getKeyFilter = getKeyFilter

改造后index.js代码 #

var AWS = require('./AWS.js');

var sns = JSON.parse(event.Records[0].Sns.Message);
var sns_MetricName = sns.Trigger.MetricName;
var sns_NameSpace = sns.Trigger.Namespace;

// 需要用到keyFilter时调用AWS方法传入对应参数即可
const keyFilter = AWS.getKeyFilter(sns_NameSpace)

该方案的优势：

• 彻底消除循环引用，规避模块加载时序导致的异常问题
• 模块职责清晰，AWS.js不再依赖index.js的内部实现，可单独复用、单独测试
• 可扩展性更强，后续如果有其他命名空间的判断逻辑，只需要修改AWS.js中的getKeyFilter方法即可

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