Promise拒绝未触发输出问题：数据库更新失败却返回成功提示

问题描述 #

当数据库更新失败时，预期能捕获到错误，但实际却收到了“Success: successfully changed Wallet Names”的提示。按照Promise机制，Promise/Promise链中的throw应该自动触发Promise/Promise链拒绝，但调用changeWalletName后并未触发catch方法执行，这是为什么？

相关代码 #

API Handler代码

import type { NextApiRequest, NextApiResponse } from 'next'
import { changeWalletName } from '../../../utils/postgresQueries'

// Handles POST requests to /api
export default async function handler(req: NextApiRequest, res: NextApiResponse) {
  try {
    const { walletNickNames, userId } = JSON.parse(req.body)
     
    changeWalletName({ walletNickNames, userId }).then(onFulfilled =>
      res.status(200).json({ message: "Success: successfully changed Wallet Names" })
    ).catch(error => {
      res.status(500).json({ message: error })
    })

  } catch (error) {
    res.status(500).json({ message: "COULD NOT CHANGE WALLET NAME" })
  }
}

数据库操作函数代码

export async function changeWalletName({ walletNickNames, userId }) {
  try {
    // update db
  } catch (error) {
    throw error;
  }
}

问题原因及解决方案 #

核心原因 #

• 异步操作未被正确等待：changeWalletName是async函数，但内部的数据库更新操作若为异步（比如返回Promise的数据库API调用），却未添加await关键字，那么这个异步操作的错误无法被当前的try/catch捕获。try/catch仅能捕获同步代码和被await的异步代码的错误，未被await的异步操作抛出的错误会成为未处理的Promise拒绝，而changeWalletName会提前返回已resolved的Promise，导致后续.then被执行，而非.catch。
• 外层try/catch无法覆盖Promise链错误：handler函数虽为async，但未用await调用changeWalletName，而是采用.then/.catch链式调用。外层的try/catch只能捕获同步代码（如JSON.parse）的错误，无法捕获该Promise链中的错误，但这并非当前问题的主因。

修复方案 #

1. 修正数据库操作函数

确保异步的数据库操作被await，让try/catch能正确捕获错误：

export async function changeWalletName({ walletNickNames, userId }) {
  try {
    // 示例：假设db.update是返回Promise的异步更新操作，必须添加await
    await db.update(/* 你的数据库更新逻辑 */);
  } catch (error) {
    throw error;
  }
}

2. 优化API Handler的错误处理

推荐统一使用await配合外层try/catch，避免混合两种异步写法，提升代码可读性：

import type { NextApiRequest, NextApiResponse } from 'next'
import { changeWalletName } from '../../../utils/postgresQueries'

export default async function handler(req: NextApiRequest, res: NextApiResponse) {
  try {
    const { walletNickNames, userId } = JSON.parse(req.body)
    // 用await等待数据库操作完成
    await changeWalletName({ walletNickNames, userId })
    res.status(200).json({ message: "Success: successfully changed Wallet Names" })
  } catch (error) {
    // 统一处理所有错误：JSON解析错误、数据库操作错误等
    const errorMsg = error instanceof Error ? error.message : "COULD NOT CHANGE WALLET NAME"
    res.status(500).json({ message: errorMsg })
  }
}

补充说明 #

若坚持使用.then/.catch链式调用，只要changeWalletName内部正确await异步操作，其返回的Promise就会在数据库错误时变为rejected，从而触发.catch执行。但async/await的写法更贴合同步思维，代码逻辑更清晰，也更便于统一错误处理。

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