动态 require js module

V2EX = way to explore

V2EX 是一个关于分享和探索的地方

已注册用户请登录

这是一个创建于 2101 天前的主题，其中的信息可能已经有所发展或是发生改变。

有一个需求，是去判断一个目录下面有没有 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: 目前为止这段代码运行还没有带来什么问题

3 条回复 2020-01-18 04:20:26 +08:00

libook

2020-01-17 15:07:22 +08:00

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

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

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

zivyou

2020-01-17 15:36:57 +08:00

@libook 感谢

xcstream

2020-01-18 04:20:26 +08:00

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