Express自引用调用registerUser报TypeError非函数问题排查

• 导出直接被覆盖了：Node的CommonJS模块里，exports只是module.exports初始值的引用，你一开始往exports上挂了registerUser，但文件最后直接写module.exports = router，相当于把整个模块对外的出口直接换成了Express路由对象，之前挂在原出口上的registerUser直接被丢弃，不管是内部还是外部模块，拿到的导出对象里根本没有这个方法。
• 没必要的自引用：你在自己写的文件里，本来registerUser就是文件作用域内的函数，直接写函数名就能调，非要通过require('./user')把自己这个文件重新加载一遍拿方法，完全是多此一举。这种写法不仅增加冗余加载开销，还可能碰到模块还没加载完成的时序问题，就算导出逻辑没问题，也可能拿到个没初始化完的模块对象，触发找不到方法的报错。

先把registerUser定义为文件内的独立函数，删掉多余的自require，再调整导出逻辑，确保路由能力和可复用的方法都能正常暴露：

// user.js // 对应api/user/signup接口的Express路由文件
const router = require('express').Router();
const userModel = require('../models/user');

// 定义独立的可复用注册逻辑
const registerUser = async (req, res, next) => {
    console.log('in registeruser')
}

router.post("/signup", (req, res, next) => {
    // 直接调用本地函数即可，不需要自引用加载
    registerUser(req, res, next);
    console.log('in signup')
    next();
})

// 把可复用方法挂到路由对象上，避免被导出覆盖丢失
router.registerUser = registerUser;
module.exports = router;

// 如果外部模块需要更清晰的导出结构，也可以替换为对象形式导出：
// module.exports = {
//   router,
//   registerUser
// }

• 只要执行了module.exports = 任意新值的赋值，之前通过exports.xxx挂载的属性会直接失效，因为导出对象的引用已经被替换。如果要同时导出多个内容，要么在最终导出的对象上直接挂载属性，要么最后再做整体替换。
• 同一个文件内的逻辑调用，优先用本地作用域的函数引用，不要通过自require的方式获取，从根源上规避模块加载时序的问题。

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