架构错误，'没有方法保存'

Question

目前我想做的是，一旦用户登录，他可以转到任何用户的网址并关注他们。我遇到了这个问题

TypeError: Object { _id: 54bd6b90b7d4cc8c10b40cbd,
  name: 'Johnny',
  username: 'batman',
  __v: 0,
  following: [],
  followers: [] } has no method 'save'

schema.js

var UserSchema = new Schema({
    name: String,
    username: { type: String, required: true, index: { unique: true }},
    password: { type: String, required: true, select: false },
    level: String,
    followers: [{ type: Schema.Types.ObjectId, ref: 'User'}],
    following: [{ type: Schema.Types.ObjectId, ref: 'User'}]
});

module.exports = mongoose.model('User', UserSchema);

api.js

        // this is where I'm Stuck ( Should I use put or post? for best practices)

        .post(function(req, res) {

            // find a current user that has logged in
            User.find({ _id: req.decoded.id }, function(err, user) {

                // simply follow the user by matching the id
                user.following = req.params.user_id;

                // save to the database ( The problem )
                user.save(function(err) {
                    if(err) res.send(err);

                    res.json({ message: "Successfully followed!"})
                })
            });
        })

问题出在 .post 中，因为我无法保存。我应该怎么做才能让它保存到数据库中？

Answer 1

您的代码中的问题是 .find() 不返回单个 mongoose 文档，而是返回一个文档数组，即使该数组仅包含一个项目。你可以通过调用.findOne() 甚至.findById 来纠正这个问题，这允许更短的形式。

如前所述，这仍然存在问题，因为无法保证在您发出 .save() after making modifications. For this reason MongoDB provides an.update()` 方法之前服务器上的文档尚未更改，该方法只会通过以下方式对文档进行额外更改指定的运算符。

要添加新的追随者，您需要将$push 元素放到数组中：

.post(function(req, res) {

  User.update(
    { 
        "_id": req.decoded.id, 
        "following": { "$ne": req.params.user_id }
    }, 
    { "$push": { "following": req.params.user_id }}, 
    function(err, user) {
      if (err) return res.send(err);

      res.json({ message: "Successfully followed!"})
    }
  );
})  

我实际上会检查该元素是否不存在，然后仅在出现这种情况时更新。有一个 $addToSet 运算符可以执行此操作，但我还建议修改您的架构以包含 "followingCount": 和类似字段。这些对于查询很有用，在不读取整个文档的情况下返回数组的“长度”并不是一件简单的事情：

var UserSchema = new Schema({
    name: String,
    username: { type: String, required: true, index: { unique: true }},
    password: { type: String, required: true, select: false },
    level: String,
    followers: [{ type: Schema.Types.ObjectId, ref: 'User'}],
    following: [{ type: Schema.Types.ObjectId, ref: 'User'}],
    folloersCount: Number,
    followingCount: Number
});

// And in later code

  User.update(
    { 
        "_id": req.decoded.id, 
        "following": { "$ne": req.params.user_id }
    }, 
    { 
        "$push": { "following": req.params.user_id }},
        "$inc": { "followingCount": 1 }
    },
    function(err, user) {
      if (err) return res.send(err);

      res.json({ message: "Successfully followed!"})
    }
  );
})  

这通过使用$inc 运算符来增加计数器值。要执行相反的操作并删除关注者，请使用 $pull

  User.update(
    { 
        "_id": req.decoded.id, 
        "following": req.params.user_id 
    }, 
    { 
        "$pull": { "following": req.params.user_id }},
        "$inc": { "followingCount": -1 }
    },
    function(err, user) {
      if (err) return res.send(err);

      res.json({ message: "Successfully Un-followed :("})
    }
  );
})  

那是$addToSet在不满足查询条件时不能也不会触碰文档的。

如果您希望在响应中包含修改后的文档，请改用.findOneAndUpdate()。

所以，仅仅因为您使用的是 mongoose，请不要忘记您基本上仍在使用 MongoDB。最好使用数据库运算符来修改文档，而不是在代码中进行这些修改。

阅读文档中的 query 和 update 运算符。事实上，请阅读 all of them，因为它值得学习。

Answer 2

您应该使用 update 方法，因为这将执行原子操作并且不容易出现任何并发问题。这是因为它会同时执行查找查询和保存。

.post(function(req, res) {

  User.update({ _id: req.decoded.id }, { $push: {following: req.params.user_id }}, function(err, user) {
    if (err) return res.send(err);

    res.json({ message: "Successfully followed!"})
  });
})