Node.js中Delete功能执行成功但返回捕获错误信息问题

• 变量名冲突：then回调里的参数res和外层Express的响应对象res重名了。数据库删除操作成功后，你试图调用res.status(200)，但此时的res是Sequelize destroy方法返回的结果（通常是受影响的行数），不是Express的响应对象，调用status方法会直接抛出错误，进而触发catch分支返回错误信息。

方法1：修改回调参数名 #

把then的参数改成非res的名称，比如affectedRows：

app.delete('/delete/items/:id', async(req, res) => {
  const id = req.params.id;
  items.destroy({
    where:{
      id: id
    }
  }).then(affectedRows => {
    res.status(200).json("Items deleted")
  }).catch(err => {
    console.error(err); // 打印错误日志，方便排查其他问题
    res.status(500).json("Can't delete Items")
  })
 })

方法2：改用async/try-catch写法（更推荐） #

既然已经声明了async，直接用await配合try-catch，避免回调嵌套和变量名冲突问题：

app.delete('/delete/items/:id', async(req, res) => {
  try {
    const id = req.params.id;
    const affectedRows = await items.destroy({
      where:{
        id: id
      }
    });
    // 额外判断：如果没有找到对应ID的记录，返回404
    if (affectedRows === 0) {
      return res.status(404).json("Item not found");
    }
    res.status(200).json("Items deleted");
  } catch (err) {
    console.error(err);
    res.status(500).json("Can't delete Items");
  }
})

• 永远在catch分支里打印错误日志，这样即使遇到其他意外问题，也能快速定位根源。
• 判断destroy返回的受影响行数，可以区分“删除成功”和“找不到对应记录”两种情况，返回更准确的HTTP状态码，符合REST API规范。

内容的提问来源于stack exchange，提问作者info.dava