使用SailsJS+Waterline编辑MongoDB条目时遇到问题

嘿，我一眼就看到你控制器里的Promise回调写法出问题了——这是Waterline新手常踩的坑！

核心错误：Promise回调参数顺序搞反了 #

Waterline的异步方法（比如findOneById）返回的Promise，then()的参数顺序是第一个是成功回调（拿到查询到的数据），第二个才是错误回调，但你现在把顺序写反了：

// 你原来的错误写法
Safety.findOneById(id).then((err, safety) => { ... })

这会导致err变量其实是查询到的safety数据，而safety反而变成undefined，直接让你的后续逻辑全走偏了！

修正后的基础版本控制器 #

先把回调顺序改对，再补上完整的更新逻辑：

editSafety: function editSafety(req, res) {
  const id = req.params.id;

  Safety.findOneById(id)
  // 第一个参数是成功获取到的记录
  .then(safety => {
    if (!safety) {
      return res.notFound('找不到对应的安全记录');
    }
    // 从请求体拿到要更新的数据
    const updateData = req.body;
    // 执行更新操作
    return Safety.updateOneById(id, updateData);
  })
  // 处理更新后的结果
  .then(updatedSafety => {
    res.ok(updatedSafety);
  })
  // 统一捕获所有错误
  .catch(err => {
    sails.log.error('编辑记录出错:', err);
    res.serverError('服务器内部错误');
  });
}

额外优化与检查点 #

除了核心错误，还有几个点可以帮你避免后续问题：

• 用REST语义化的HTTP方法：编辑资源更推荐用PUT或PATCH，而不是POST，你可以把路由改成：

  'put /edit/safety/:id': { controller: 'SafetyController', action: 'editSafety' }
  

  当然如果业务场景必须用POST也没问题，但PUT/PATCH更符合REST规范。
• 校验MongoDB ID格式：如果你的id是MongoDB的ObjectId，要确保前端传的ID格式正确，否则查询会失败。可以用validator库做校验：

  const { isMongoId } = require('validator');
  if (!isMongoId(id)) {
    return res.badRequest('无效的记录ID');
  }
  

• 过滤更新字段：不要直接把req.body全部传给更新方法，避免用户修改不允许的字段（比如createdAt），可以用lodash的pick方法过滤：

  const _ = require('lodash');
  const allowedFields = ['title', 'content', 'status']; // 允许更新的字段
  const filteredData = _.pick(req.body, allowedFields);
  

• 用async/await简化代码：Promise链式调用多了容易乱，用async/await会更清晰：

  editSafety: async function editSafety(req, res) {
    try {
      const id = req.params.id;
      // ID格式校验
      const { isMongoId } = require('validator');
      if (!isMongoId(id)) {
        return res.badRequest('无效的记录ID');
      }
      // 查询记录
      const safety = await Safety.findOneById(id);
      if (!safety) {
        return res.notFound('找不到对应的安全记录');
      }
      // 过滤更新字段
      const _ = require('lodash');
      const allowedFields = ['title', 'content', 'status'];
      const filteredData = _.pick(req.body, allowedFields);
      if (_.isEmpty(filteredData)) {
        return res.badRequest('没有需要更新的字段');
      }
      // 执行更新
      const updatedSafety = await Safety.updateOneById(id, filteredData);
      // 返回结果
      res.ok(updatedSafety);
    } catch (err) {
      sails.log.error('编辑记录出错:', err);
      res.serverError('服务器内部错误');
    }
  }
  

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