在构造函数内导入模块是否可行？该做法是否为不良实践？需实现动态模块重载

一、构造函数内部导入模块的影响 #

咱们先拆解这个行为带来的几个关键影响：

• 模块错误延迟暴露：通常模块导入放在文件顶部时，应用启动阶段就会加载所有依赖，一旦模块不存在、有语法错误这类问题会立刻报错；但放在构造函数里的话，只有当你实例化这个类的时候才会触发错误，可能导致应用运行一段时间后才崩溃，增加了调试成本。
• 重复require但不会重复加载模块：Node.js的require自带缓存机制，同一个模块路径第一次被require时，会加载文件、执行模块代码并缓存导出对象；后续再调用require同一个路径，直接返回缓存结果。所以哪怕你每次实例化Product都调用require，也不会重复读取文件或重新执行模块的顶级代码，只是多了几次缓存查找的微小开销（几乎可以忽略）。
• 模块副作用仅触发一次：如果依赖的模块里有顶级执行的代码（比如初始化日志、创建数据库连接），这些副作用只会在第一次require时执行，后续require不会重复触发，这点和顶部导入的表现一致。

二、构造函数中使用require是否属于不良实践？ #

大部分常规场景下，确实属于不太推荐的实践，原因有这几点：

• 依赖关系不直观：其他开发者看代码时，需要钻进构造函数里找依赖，不如顶部导入能一眼看清类的依赖清单，增加了代码理解成本。
• 启动阶段错误检测失效：如果依赖的模块有问题，顶部导入会在应用启动时立刻暴露，方便快速排查；而构造函数导入要等到实例化才报错，可能导致线上服务运行后才出问题。
• 微小的性能冗余：虽然缓存存在，但每次实例化都调用require还是会多做路径解析、缓存查找的操作，虽然影响极小，但不符合代码优化的最佳习惯。

但有例外场景：比如你现在需要的动态模块重载、根据条件加载不同模块（比如按环境变量加载配置）、或者模块体积极大且很少被实例化，延迟加载能减少启动时间，这时候在构造函数里导入就是合理的选择。

三、实现动态模块重载的改进方案 #

你当前的代码里，require的缓存机制会导致模块文件更新后，this.product还是原来的导出对象，没法实现重载。要解决这个问题，需要手动清除对应模块的缓存，再重新加载。

方案1：实例化时可选重载（每次实例化加载最新模块） #

修改构造函数，添加重载开关，需要时先清缓存再加载：

class Product {
  constructor(shouldReload = false) {
    // 如果需要重载，先清除对应模块的缓存
    if (shouldReload) {
      const productModulePath = require.resolve('../db/models/_product');
      delete require.cache[productModulePath];
      const helpersModulePath = require.resolve('./index');
      delete require.cache[helpersModulePath];
    }
    this.product = require('../db/models/_product');
    this.helpers = require('./index');
  }

  getProducts(qty) {
    return this.product.find().limit(qty);
  }

  findProduct(product_id) {
    return this.product.findById(product_id);
  }

  searchProduct(searchTerm) {
    return this.product.find({$text: {$search: searchTerm, $caseSensitive: false}});
  }
}

使用时，传入new Product(true)就能加载最新的模块内容。

方案2：添加专门的重载方法（运行时按需触发） #

如果不需要每次实例化都重载，而是在运行时按需更新，可以单独加一个重载方法：

class Product {
  constructor() {
    this.loadModules();
  }

  // 抽离模块加载逻辑
  loadModules() {
    this.product = require('../db/models/_product');
    this.helpers = require('./index');
  }

  // 手动触发重载
  reload() {
    // 清除缓存
    const productModulePath = require.resolve('../db/models/_product');
    delete require.cache[productModulePath];
    const helpersModulePath = require.resolve('./index');
    delete require.cache[helpersModulePath];
    // 重新加载模块
    this.loadModules();
  }

  getProducts(qty) {
    return this.product.find().limit(qty);
  }

  findProduct(product_id) {
    return this.product.findById(product_id);
  }

  searchProduct(searchTerm) {
    return this.product.find({$text: {$search: searchTerm, $caseSensitive: false}});
  }
}

之后调用productInstance.reload()就能更新到最新的模块内容了。

注意事项 #

• 清除缓存后重新加载模块，模块内的顶级代码会重新执行，所以如果模块有初始化逻辑（比如创建数据库连接），要确保重复执行不会导致问题（比如避免重复创建连接）。
• require.resolve()会返回模块的绝对路径，能精准定位到缓存中的模块条目，避免路径匹配错误。

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