Node.js+Express API中函数返回值赋值外部变量显示undefined问题

你在开发基于Node.js和Express的API时，尝试将MySQL查询结果赋值到函数外部的data变量，但最终data显示为undefined。你的核心函数和调用代码如下：

核心查询函数 #

function get_orders(callback) {
  dbConnection.query(
    "CALL orders.getOrders()",
    async (err, rows, fields) => {
      if (!err) {
        if (rows.length !== 0) {
          return await callback(rows[0][0]);
        }
      } else {
        throw err;
      }
    }
  );
}

调用代码 #

function call_orders(req, res) {
  let data;
  get_orders(async (result) => {
    data = await result;
  });
}

问题原因 #

• 异步执行顺序混乱：dbConnection.query是异步操作，call_orders函数会在回调执行前就完成执行，所以当你后续访问data时，它还没被赋值，自然是undefined。
• 不必要的await滥用：rows[0][0]是同步返回的查询结果，你传入的async回调和里面的await完全多余，反而会打乱执行逻辑。
• 错误处理不规范：回调内直接throw err无法被外部捕获，会导致未处理的异常，甚至程序崩溃。

解决方案 #

方案1：规范使用回调（确保逻辑在回调内执行）

先简化get_orders的错误处理，把错误传递给回调，再在调用时将依赖data的逻辑放在回调内部：

// 优化后的get_orders
function get_orders(callback) {
  dbConnection.query(
    "CALL orders.getOrders()",
    (err, rows, fields) => {
      if (err) {
        return callback(null, err); // 错误优先的回调模式
      }
      if (rows.length !== 0) {
        callback(rows[0][0], null);
      } else {
        callback(null, new Error("No orders found"));
      }
    }
  );
}

// 正确调用
function call_orders(req, res) {
  get_orders((data, err) => {
    if (err) {
      return res.status(500).json({ error: err.message });
    }
    // 在这里处理数据，比如返回给客户端
    res.json(data);
  });
}

方案2：Promise化函数（推荐，更符合现代异步编程习惯）

将get_orders改造成返回Promise的函数，配合async/await让异步代码更易读：

// Promise化的get_orders
function get_orders() {
  return new Promise((resolve, reject) => {
    dbConnection.query(
      "CALL orders.getOrders()",
      (err, rows, fields) => {
        if (err) {
          return reject(err);
        }
        rows.length !== 0 ? resolve(rows[0][0]) : reject(new Error("No orders found"));
      }
    );
  });
}

// 用async/await调用
async function call_orders(req, res) {
  try {
    const data = await get_orders();
    // 这里可以正常使用data
    res.json(data);
  } catch (err) {
    // 统一处理错误
    res.status(500).json({ error: err.message });
  }
}

总结 #

不要试图在异步回调外部直接获取尚未赋值的变量，异步操作的结果必须在其完成后才能使用。优先选择Promise+async/await的写法，代码更简洁易维护，错误处理也更规范。

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