如何从Express路由中将流程控制权传递至外部函数？

问题描述 #

开发Node.js Express应用时，路由代码（index.js）如下：

var express = require('express');
var router = express.Router();

const { codeExecute } = require('../controllers/userController');

router.get('/', function(req, res, next) {
    if (req.query.code){    // 检测GET请求中是否存在code参数
        console.log(req.query.code)    // 此处能正确打印code参数
        codeExecute(req,res,next)    // 希望将流程控制权传递给该函数
    }else{
        res.render('index', { title: 'La mia Home Page' });
    }
});

对应的控制器函数（userController.js）：

async function codeExecute(req, res, next) {

    try {
        console.log(req.body.code)    // 此处输出undefined
        // 执行逻辑，比如渲染页面

    } catch (err) {
        console.log(err);
        next(createError(500));
    }
}

疑问：期望路由检测到GET请求的code参数时，将流程控制权交给codeExecute函数，但该函数中req.body.code输出为undefined。请问能否在路由中将流程控制权交给另一个负责页面渲染的函数？

问题原因 #

GET请求不存在请求体（req.body），你在URL中传递的code参数属于查询字符串，只能通过req.query.code获取，所以控制器里调用req.body.code会得到undefined。

解决方案 #

1. 修正参数获取方式 #

修改控制器函数，改用req.query.code获取参数：

async function codeExecute(req, res, next) {
    try {
        const code = req.query.code;
        console.log(code); // 现在能正确获取到code参数
        // 执行页面渲染逻辑，示例：
        res.render('code-processed', { code: code, title: '处理结果' });
    } catch (err) {
        console.log(err);
        next(createError(500));
    }
}

2. 优化路由控制权传递方式 #

完全可以将路由的流程控制权交给负责渲染的函数，你当前的codeExecute(req,res,next)写法是正确的，也可以用更清晰的路由结构实现：

var express = require('express');
var router = express.Router();

const { codeExecute } = require('../controllers/userController');

// 中间件判断：无code参数时直接渲染首页
router.get('/', (req, res, next) => {
    if (!req.query.code) {
        return res.render('index', { title: 'La mia Home Page' });
    }
    next(); // 有code参数时，传递控制权给下一个处理函数
}, codeExecute); // 这里的codeExecute就是负责渲染的函数

这种写法将判断逻辑和业务逻辑分离，代码更易维护。

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