Node MySQL异步方法getDatabaseValues返回undefined问题求助

可以保留你的OOP代码结构，只需调整以下几处即可解决返回值为undefined的问题，同时优化异步逻辑：

关键修改点： #

1. 移除冗余的Promise嵌套
   你的getDatabaseValues已经是async函数，它本身会自动返回Promise，无需手动再包裹一层new Promise(...)。这种嵌套写法不仅冗余，还可能引发错误处理的隐性问题。

2. 将回调式query转为Promise形式
   Node MySQL的回调风格query可以通过util.promisify转换为支持await的Promise版本，和async/await语法更契合，避免回调嵌套带来的可读性问题。

3. 用finally确保连接可靠关闭
   原代码在query回调中关闭连接，改用Promise形式后，通过finally可以保证无论查询成功或失败，数据库连接都会被关闭，避免资源泄漏。

修改后的完整代码： #

const util = require('util');

async getDatabaseValues() {
  const databaseManager = new DatabaseManager();
  try {
    await databaseManager.connect();
    // 转换query方法为Promise并绑定上下文
    const query = util.promisify(databaseManager.connection.query).bind(databaseManager.connection);
    const results = await query("SELECT * FROM live");
    return results[0];
  } catch (error) {
    throw error; // 将错误抛出，交由调用方处理
  } finally {
    databaseManager.close();
  }
}

关于Node MySQL query的使用说明： #

你原有的回调式写法语法上是正确的，但在async/await场景下，回调嵌套会破坏代码的线性可读性，且容易遗漏错误处理。转为Promise形式后，代码逻辑更清晰，也更符合现代异步代码的编写习惯。

调用验证 #

你期望的调用方式完全有效，只需确保调用代码在async函数内：

async yourMethod() {
  const databaseValues = await this.getDatabaseValues();
  const title = databaseValues.title;
  // 后续业务逻辑
}

原代码问题根源 #

原代码并非resolve失效，而是async函数嵌套new Promise的写法存在隐患：当new Promise的异步执行器（async (resolve, reject) => {}）内部出现未捕获错误时，可能无法正确触发reject，导致调用方获取到undefined。同时，若调用时未正确使用await，也会直接得到Promise对象而非结果。

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