Router.use() 需要一个中间件函数，但有一个未定义的

Question

我的 index.js 中有这段代码

...
import userRoutes from './src/routes/userRoutes';
import invoicesRoutes from './src/routes/invoicesRoutes';
import authMiddleware from "./src/middlewares/authMiddleware";
...
const app = express();
...
app.use('/user', authMiddleware, userRoutes(app));
app.use('/invoices', authMiddleware, invoicesRoutes(app));

我的身份验证中间件

const authMiddleware = (req, res, next) => {
    if (req.headers && req.headers.authorization && req.headers.authorization.split(' ')[0] === 'JWT') {
        console.log(req.headers.authorization.split(' ')[0]);
        next();
    } else {
        console.log('else');
        next();
    }
};

export default authMiddleware;

和我的路线之一：

import { register, login } from '../controllers/authController';

const userRoutes = (app) => {
    app.route('/user/signup')
        .post(register);
    app.route('/user/login')
        .post(login);
};

export default userRoutes;

我收到一个错误：

throw new TypeError('Router.use() 需要一个中间件函数，但得到了一个 ' + gettype(fn)) ^

类型错误：Router.use() 需要一个中间件函数，但有一个未定义的

怎么了？

Answer 1

您将路由初始化作为中间件功能app.use('/user', authMiddleware, userRoutes(app));

在这行代码中， userRoutes(app)应该返回的是function(req,res[,next]){}

它应该是这样的

app.use('/user', authMiddleware, (req, res, next) => {
        userRoutes(app);
        next()
})

而你所做的是

app.use('/user', authMiddleware, (app) => {
    app.route('/user/signup').post(register);
    app.route('/user/login').post(login);
})

那是错误的

这里似乎有一些不好的逻辑，因为据我所知，您将不得不调用http://localhost/user来初始化路由，并且由于包含 express 的中间件而无法正常工作。

再次根据我的理解，您正在尝试做的事情应该看起来更像这样

...
import userRoutes from './src/routes/userRoutes';
import invoicesRoutes from './src/routes/invoicesRoutes';
...
const app = express();
userRoutes(app);
invoicesRoutes(app);

和

import { register, login } from '../controllers/authController';
import authMiddleware from "./middleware";
const userRoutes = (app) => {
    app.post('/user/signup', authMiddleware, register);
    app.post('/user/login', authMiddleware, login);
};

export default userRoutes;

Answer 2

Express 已经提供了Router 类来分隔不同子模块中的路由，因此您可以导入userRoutes模块而无需将app变量传递给它，这是一个示例：

用户路由.js

import { register, login } from '../controllers/authController';
import express from 'express';

const router = express.Router();

router.route('/signup')
        .post(register);

router.route('/login')
        .post(login);

export default router;

索引.js

...
import userRoutes from './src/routes/userRoutes';
import invoicesRoutes from './src/routes/invoicesRoutes';
import authMiddleware from "./src/middlewares/authMiddleware";
...
const app = express();
...
app.use('/user', authMiddleware, userRoutes);
app.use('/invoices', authMiddleware, invoicesRoutes);

希望这可以帮助

Answer 3

检查所有 JavaScript 文件以确保没有遗漏以下行：

module.exports = router

Answer 4

在我的 route.js

我把它像这样module.exports = routes;

我打算做的是导出路由，像下面这样更改后，我的错误消失了module.exports = {routes};