为什么在./src/Client.js中导入index.js后调用bindClient报错不是函数

问题原因 #

你遇到的是Node.js CommonJS模块规范下的循环依赖问题：
你的./src/index.js中导出的Client: C必然是从./src/Client.js中引入的，加载流程如下：

1. 入口文件加载./src/index.js，开始执行该文件代码
2. index.js执行到引入Client.js的逻辑时，会暂停自身执行，转而加载Client.js
3. Client.js执行到顶部引入同目录index.js的逻辑时，Node识别到二者属于循环依赖，不会重新加载index.js，只会返回当前index.js已经导出的内容——此时index.js还没执行到module.exports = {...}的赋值语句，导出的是默认的空对象{}
4. Client.js执行完成后回到index.js，才会完成模块导出的赋值，但Client.js中已经拿到的空对象不会自动更新，因此调用bindClient时就会报方法不存在的错误。

解决方法 #

方案1：调整index.js的代码执行顺序 #

先完成module.exports的赋值，再引入Client依赖，确保循环导入时导出的对象已经包含所有方法：

// ./src/index.js 调整后代码
// 先完成导出赋值
module.exports = {
    Send,
    MessageDelete,
    Command,
    Client: null,
    bindClient(client) {
        c = client;
        return (
            c.on("messageCreate", msg => {
            const command = commands.find(c => msg.content.startsWith(c.name))
            if (command) {
                command.send ? Send(command.send, msg.channel) : null
                command.delete ? MessageDelete(msg) : null
                command.emit("execute", msg, command)
            }
        })
        );
    }
}
// 再引入Client并赋值
const C = require('./Client.js')
module.exports.Client = C

方案2：延后index.js的导入时机 #

不要在Client.js的顶部导入index.js，改为在用到bindClient方法的位置再导入：

// ./src/Client.js 示例
// 顶部删除const index = require('./index.js')的逻辑
class Client {
    constructor() {
        // 用到的时候再导入
        const index = require('./index.js')
        index.bindClient(this)
    }
}

此时导入index.js时，index.js已经完成了全量导出，拿到的就是包含所有方法的完整对象。

方案3：拆分公共逻辑彻底规避循环依赖 #

把bindClient这类公共方法拆分到独立的工具文件（比如./src/common.js），让index.js和Client.js都依赖该工具文件，二者不再互相引用，从根源上避免循环依赖问题。

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