Node.js将其他文件函数作为路由参数提示未定义如何解决

错误根因 #

报错是两个问题叠加导致的：

• 循环依赖导致导出未初始化：index.js第一行就加载路由文件，此时index.js还没执行到authenticateToken的定义和导出步骤，路由文件反向引入index.js时只能拿到未完成初始化的空导出对象，authenticateToken取值为undefined
• 导出引入语法不匹配：index.js是默认导出authenticateToken，但路由文件用命名导出的解构语法取值，就算没有循环依赖也无法拿到正确的函数

修复方案 #

按照模块化拆分的原则，把通用鉴权中间件抽成独立文件，从根源避免循环依赖：

1. 新建middlewares/auth.js，存放鉴权中间件逻辑：

const jwt = require('jsonwebtoken');

const authenticateToken = (req, res, next) => {
    const authHeader = req.headers['authorization'];
    const token = authHeader && authHeader.split(' ')[1]
    if (token == null) return res.sendStatus(401);

    jwt.verify(token, process.env.ACCESS_TOKEN_SECRET, (err, user) => {
        if (err) return res.sendStatus(403);
        req.user = user
        next()
    })
}

module.exports = authenticateToken;

2. 修改routes/getPosts.js的引入逻辑，不再依赖index.js：

const express = require('express');
const router = express.Router();
const db = require('../databaseConnection');
const authenticateToken = require('../middlewares/auth')

router.get('/', authenticateToken, (req, res) => {
        db.query('SELECT * FROM posts', 
        (err, result) => {
            if (err) {
                res.send({errorMessage: err});
                return
            }
            res.send(result);
        });
    }
}); 

module.exports = router;

额外优化：原代码中db.query用了回调写法，不需要加await关键字，同时报错分支要加return避免后续逻辑重复执行

3. 清理index.js的冗余代码，移除原有authenticateToken的定义和导出，只保留路由注册逻辑：

// 原有express实例化、环境变量加载等初始化逻辑保留
//Routers 
const getPostsRouter = require('./routes/getPosts');
app.use('/getPosts', getPostsRouter);

内容的提问来源于stack exchange，提问作者Kevin. J