动态 require js module

[zivyou]

有一个需求，是去判断一个目录下面有没有 js 文件，如果有的话，就 require 它(后续肯定会用到这些 js 文件里面的一些函数)。 现在我的写法是:

const files = readdirSync(path);
const jsFiles = files.filter((f) => f.endsWith('.js'));

jsFiles.forEach((f) => {
    const jsFile = `${path}/${f}`;
    const mapping = require(jsFile);
   // do something with mapping
});

eslint 会提示 dynamic require 是一种不好的设计，查了一些资料，也有明确说不能动态 require js 文件的。 请问下各位老哥，这种情况有什么更好的方案吗?

PS: 目前为止这段代码运行还没有带来什么问题

[libook]

运行肯定是可以运行的，功能也能满足要求，ESLint 规则只是建议，并不一定代表程序一定有问题。

动态 require 的问题可能有：
1. 如果你用 Webpack 等打包工具分析代码依赖的时候，无法妥善处理动态 require 的代码，只能处理静态引用。
2. 如果引用的文件有被替换或篡改的可能性（如上传功能），或如果引用的路径有被篡改的可能性，那么在文件第一次被 require 之前，可能会被攻击，并最终导致运行恶意代码。

暂时想到的只有这么多，如果你的使用场景不具备以上风险的前提条件，就可以用。

[zivyou]

@libook 感谢

[xcstream]

可以先扫描一下目录 写出一个 js 文件里面都是 require， 再 require 它