本地节点 API 未发布到 Mongoose DB

Question

谁能告诉我为什么我的 POST 方法没有通过 Mongoose 保存到我的 MongoDB 中？

我的 Angular 控制器

$scope.saveUpdate = function(id){
    $http.post('/api/entry/' + id)
        .success(function(data){
            $scope.entry = data;
        })
        .error(function(data){
            console.log('There was a problem saving your entry: ' + data);
        });
    // update page with remaining entries
    $http.get('/api/entries').then(function(response){
        $scope.entries = response.data;
    });
}

我的 API

app.post('/api/entry/:entry_id', function(req, res){
    if (req.params) {
      Entries.findByIdAndUpdate({
        _id : req.params,
        // the properties we're updating and the new values
        username: req.body.username,
        date: req.body.date,
        income: req.body.income
      }, function(err, entry){
        if (err) {
          res.send(err) }
          else {
            res.send('Success!');
          }
      })
    }
});

视图中的提交按钮

<button type="submit" class="btn" ng-click="saveUpdate(entry._id)">Update</button>

当点击按钮时，更新的条目会到达 DOM，但当它到达 Angular 核心代码时，它会恢复到原始状态而不更新数据库。也不会抛出任何错误。

Answer 1

上面的代码有几处错误：

1. 整个req.paramsobject 被传递到_id 字段而不是req.params.entry_id
2. 将参数传递给findByIdAndUpdate() 的方式不正确
3. 请求正文永远不会在您的 $http.post() 中发送，但您希望 req.body 包含您的路由中的数据

req.params 指向请求中的整个 params 对象。您只想从参数中获取 ID，然后将其传递给您的 mongoose 模型。

假设您传递的是entry_id，那么您将传递您的第一个条件if(req.params)，因为参数确实存在。但是，当您将 req.params 传递给 Entries 模型的 _id 字段时，您实际上是在传递整个对象 { entry_id: '123' } 而不仅仅是 123。

此外，您将值传递给findByIdAndUpdate 方法的方式不正确。 findByIdAndUpdate(id, [update], [options], [callback]) 需要 4 个参数，id 是唯一必填字段。您传入整个对象以根据 id 查找并更新单个参数中的值。您需要从要更新的字段中拆分出entry_id。

app.post('/api/entry/:entry_id', function(req, res) {

  // Param Existence Checking
  if (!req.params.entry_id)
    return res.status(400).send('an entry_id must be provided');
  if (!req.body.username)
    return res.status(400).send('a username must be provided');
  if (!req.body.date)
    return res.status(400).send('a date must be provided');
  if (!req.body.income)
    return res.status(400).send('an income must be provided');

  var updateData = {
    username: req.body.username,
    date: req.body.date,
    income: req.body.income
  };

  Entries.findByIdAndUpdate(req.params.entry_id, updateData, function(err, entry){
    if (err)
      return res.status(500).send(err)

    return res.status(200).send('Success!');
  })

});

同样基于您问题中的示例代码，我看不到您在执行$http.put() 时将值传递给req.body 的位置。可以肯定的一件事是，如果req.body 不包含username、date 和income，您将获得undefined 分配给这些字段。

要通过$http.post() 提供请求正文，请将其传递给第二个参数data。

$http.post('/api/entry/' + id, {
  username: 'username',
  date: new Date(),
  income: 10000.00
})
  .then(function(res, status){ 
    console.log(res.data);
  })
  .catch(function(err) {
    console.log(err);
  });

另外，不要在你的承诺链中使用.success()，that approach is deprecated。在处理您的回复时，您应该使用 A+ 标准 .then() 和 .catch()。