Node.js中MongoDB的findOneAndUpdate执行异常问题求助

问题重现 #

初始代码（更新无效果） #

exports.dashboardUpdateNote = async (req, res, next) => {
    var updateNote = {
        "title": req.body.title,
        "body": req.body.body1,
    }
    await Note.findOneAndUpdate({ _id: req.body.note_id }, { $set: updateNote })
        .then((Successful) => {
            console.log('Successful: ', Successful)
        })
    return res.redirect('/dashboard')
}

执行后页面正常重定向，但目标笔记未更新。

添加upsert: true后的问题 #

修改后代码：

exports.dashboardUpdateNote = async (req, res, next) => {
    var updateNote = {
        "title": req.body.title,
        "body": req.body.body1,
    }
    await Note.findOneAndUpdate({ _id: req.body.note_id }, { $set: updateNote }, { upsert: true })
        .then((Successful) => {
            console.log('Successful: ', Successful)
        })
    return res.redirect('/dashboard')
}

出现两个问题：

1. 目标笔记未更新，反而新增了一条笔记；
2. 点击这条新增的笔记时，触发错误：

Error [ERR_HTTP_HEADERS_SENT]: Cannot set headers after they are sent to the client
    at new NodeError (node:internal/errors:399:5)
    at ServerResponse.setHeader (node:_http_outgoing:644:11)
    at ServerResponse.header (C:\Users\hp\NODE\Notes\node_modules\express\lib\response.js:794:10)     
    at ServerResponse.contentType (C:\Users\hp\NODE\Notes\node_modules\express\lib\response.js:624:15)
    at ServerResponse.send (C:\Users\hp\NODE\Notes\node_modules\express\lib\response.js:149:14)       
    at done (C:\Users\hp\NODE\Notes\node_modules\express\lib\response.js:1035:10)
    at tryHandleCache (C:\Users\hp\NODE\Notes\node_modules\ejs\lib\ejs.js:280:5)
    at exports.renderFile [as engine] (C:\Users\hp\NODE\Notes\node_modules\ejs\lib\ejs.js:491:10)     
    at View.render (C:\Users\hp\NODE\Notes\node_modules\express\lib\view.js:135:8)
    at tryRender (C:\Users\hp\NODE\Notes\node_modules\express\lib\application.js:657:10)

未更新过的笔记可正常访问。

原因分析 #

1. 初始代码更新无效：MongoDB的_id是ObjectId类型，而req.body.note_id是字符串类型，直接匹配无法找到目标文档，导致更新操作无效果。
2. 新增笔记问题：添加upsert: true后，因为匹配不到文档，MongoDB自动创建了一条新笔记，而非更新原有文档。
3. ERR_HTTP_HEADERS_SENT错误：访问新增笔记时，对应的详情路由处理函数中存在多次发送响应的情况（比如在res.render之前已经用res.send返回了内容，后续又尝试渲染页面），导致重复设置HTTP头。

解决方案 #

1. 修正更新操作，匹配正确的文档 #

• 将字符串类型的note_id转换为MongoDB的ObjectId；
• 移除upsert: true（更新操作不需要自动新增）；
• 统一使用async/await，避免混用then导致逻辑混乱；
• 添加{ new: true }参数，返回更新后的文档便于调试。

修正后的更新代码：

const { ObjectId } = require('mongoose');

exports.dashboardUpdateNote = async (req, res, next) => {
  try {
    const updateNote = {
      title: req.body.title,
      body: req.body.body1,
    };
    // 转换note_id为ObjectId类型
    const noteId = ObjectId(req.body.note_id);
    const updatedNote = await Note.findOneAndUpdate(
      { _id: noteId },
      { $set: updateNote },
      { new: true } // 返回更新后的文档
    );

    if (!updatedNote) {
      return res.status(404).send('目标笔记不存在');
    }

    console.log('更新成功: ', updatedNote);
    return res.redirect('/dashboard');
  } catch (err) {
    console.error('更新失败: ', err);
    next(err); // 交给Express错误处理中间件
  }
};

2. 修复ERR_HTTP_HEADERS_SENT错误 #

检查笔记详情页的路由代码，确保每个请求只发送一次响应。比如存在如下错误代码：

exports.getNoteDetail = async (req, res) => {
  try {
    const note = await Note.findById(req.params.id);
    if (!note) {
      res.send('笔记不存在'); // 发送响应后未终止函数
    }
    res.render('note-detail', { note }); // 再次发送响应，触发错误
  } catch (err) {
    console.error(err);
    res.status(500).send('服务器错误');
  }
};

修正后：

exports.getNoteDetail = async (req, res) => {
  try {
    const note = await Note.findById(req.params.id);
    if (!note) {
      return res.send('笔记不存在'); // 用return终止后续执行
    }
    res.render('note-detail', { note });
  } catch (err) {
    console.error(err);
    return res.status(500).send('服务器错误'); // 确保只发送一次响应
  }
};

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