Express REST API更新字段失败排查：PUT接口无法修改文档字段

我来帮你拆解下这段代码的问题，以及怎么修复：

核心问题点 #

• 没有持久化修改：你通过findByIdAndUpdate拿到beer文档后，直接修改了beer.name，但没有调用beer.save()把修改同步到数据库，所以数据库里的数据根本没变化。
• 缺少响应与错误处理：你的接口没有给客户端返回任何响应（比如res.json或res.send），也没处理查询可能出现的错误，导致前端请求会一直处于挂起状态，收不到任何反馈。
• findByIdAndUpdate的用法有误：其实这个方法本身就支持直接传入更新字段，不需要手动修改文档再保存，这样写法更简洁高效。

修复后的代码示例 #

方式1：使用回调函数（和你原代码风格一致） #

app.put('/api/beers/:id/comment', (req, res) => {
  Beer.findByIdAndUpdate(
    req.params.id,
    { name: 'newName' }, // 直接传入要更新的字段对象
    { 
      new: true, // 返回更新后的文档（默认返回更新前的）
      runValidators: true // 触发模型的验证规则，确保数据合法性
    },
    (err, updatedBeer) => {
      if (err) {
        // 错误时返回400状态码和错误信息
        return res.status(400).json({ error: err.message });
      }
      if (!updatedBeer) {
        // 找不到对应文档时返回404
        return res.status(404).json({ error: 'Beer not found' });
      }
      // 成功返回更新后的文档
      res.json(updatedBeer);
    }
  );
});

方式2：使用async/await（更易读的现代写法） #

app.put('/api/beers/:id/comment', async (req, res) => {
  try {
    const updatedBeer = await Beer.findByIdAndUpdate(
      req.params.id,
      { name: 'newName' },
      { new: true, runValidators: true }
    );

    if (!updatedBeer) {
      return res.status(404).json({ error: 'Beer not found' });
    }

    res.json(updatedBeer);
  } catch (err) {
    res.status(400).json({ error: err.message });
  }
});

额外说明 #

你原来的代码里，findByIdAndUpdate的回调拿到的beer是更新前的旧文档，就算你调用beer.save()，也只是把旧文档的修改保存回去，不是最优写法。而直接通过findByIdAndUpdate传入更新对象，MongoDB会直接在数据库层面执行更新，性能更好。

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