NodeJS中mysql包db.query回调内异步调用参数为undefined问题问询

问题根因与解决方法 #

1. 排查insertedId为undefined的问题 #

• 首先确认你的user表主键是否设置了AUTO_INCREMENT属性，MySQL的result.insertId仅当表存在自增主键时才会返回插入的ID值，若主键为自定义生成，则该字段默认返回0或undefined。
• 在回调中添加打印调试，确认返回的result对象结构，判断是查询未返回ID还是后续处理丢失：

db.query("INSERT INTO user SET ?", [data], async (iErr, result) => {
    if (iErr) {
        res.status(410).jsonp({msg:iErr});
        return next(iErr); // 错误分支添加return，避免后续逻辑继续执行
    }
    console.log('插入结果：', result);
    const insertedId = result.insertId;
    console.log('拿到的自增ID：', insertedId);
    // 其余逻辑
})

• 若调试确认insertedId本身有值，检查getEmailToken方法的参数定义，是否存在拼写错误（比如将参数名写为insertedid小写等）。

2. 排查req.body.name为undefined的问题 #

该问题基本都是请求作用域销毁导致的：Express的请求生命周期结束后，req对象会被回收或被后续请求复用，若你在返回响应之后才异步执行发邮件逻辑，就会出现req.body为空的情况。
解决方法：在路由入口处就将需要用到的请求参数提前存入局部变量，不要在异步回调中直接引用req对象：

// 路由开头先提取所有需要的参数，存在局部变量中
const { name, email } = req.body;
db.query("INSERT INTO user SET ?", [data], async (iErr, result) => {
    // ...省略错误处理
    // 发邮件时直接使用提前存好的局部变量，不再调用req.body.name
    const emailContent = `您好${name}，请点击链接完成邮箱验证...`
})

3. 优化写法避免异步问题 #

你当前混用回调+async/await的写法很容易出现作用域和错误捕获问题，建议将数据库查询promise化，用全async的写法梳理逻辑：

// 封装promise版查询方法（mysql2已原生支持promise，旧mysql包可用util.promisify转换）
const util = require('util');
const query = util.promisify(db.query).bind(db);

// 路由逻辑
router.post('/register', async (req, res, next) => {
    try {
        const { name, email } = req.body;
        const insertResult = await query("INSERT INTO user SET ?", [data]);
        const insertedId = insertResult.insertId;
        const emailToken = await getEmailToken(insertedId);
        // 执行发邮件逻辑
        await sendVerifyEmail(email, name, emailToken);
        res.status(200).jsonp({msg: '注册成功，请查收验证邮件'});
    } catch (err) {
        res.status(410).jsonp({msg: err.message});
        next(err);
    }
})

如果发邮件耗时较长不想阻塞用户响应，可引入轻量任务队列处理邮件发送逻辑，依然要注意提前把需要的参数全部提取出来，不要依赖原始req对象。

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