Mongoose - 使用嵌套模型更新文档答案

【问题标题】：Mongoose - Upserting Documents with Nested ModelsMongoose - 使用嵌套模型更新文档
【发布时间】：2012-09-30 18:50:48
【问题描述】：

我的快递应用中有一个带有“checked_in”标志的基本文档：

module.exports = Book= mongoose.model('Book', new Schema({
 name : String,
 checked_in : Boolean
},{ collection : 'Book' }));

我想记录书籍的签入和签出时间，因此我想出了另一个架构：

var action = new Schema({
 checked_in: Boolean,     
});

module.exports = Activity = mongoose.model('Activity', new Schema({
 book_id: String,
 actions: [action]
},{ collection : 'Activity' }));

“book_id”应该是一本书的文档 ID，当我更新一本书时，我需要使用操作中的新项目创建或更新该书的活动日志：

exports.update = function(req, res){  
    return Book.findById(req.params.id, function(err, book) {  
       var activity = new Activity({book_id: book.id});
       activity.actions.push({
           checked_in: req.body.checked_in,
       });

       Activity.update({ book_id: book.id}, activity.toObject(), { upsert: true }));

       book.checked_in = req.body.checked_in;
       return device.save(function(err) {
           return res.send(book);
       });
    });
};

我遇到的问题是没有任何东西被插入到 Activity 集合中。如果我使用 .save() 那么我只会在集合中得到很多重复项。

更新

我已经开始按照下面给出的建议重新工作，但我仍然没有任何运气。这是我现在拥有的：

module.exports = Activity = mongoose.model('Activity', new Schema({
  book_id: Schema.ObjectId,
  actions: [new Schema({
    checked_in: Boolean,
    last_user: String
  })]
},{ collection : 'Activity' }));

现在是更新代码：

exports.update = function(req, res){  
  // TODO: Check for undefined.
  return book.findById(req.params.id, function(err, book) {    
    if(!err) {
      // Update the book.
      book.checked_in = req.body.checked_in;
      book.last_user = req.body.last_user;    
      book.save();

      // If there's no associated activity for the book, create one. 
      // Otherwise update and push new activity to the actions array.
      Activity.findById(book._id, function (err, activity) {
        activity.actions.push({
          checked_in: req.body.checked_in,
          last_user: req.body.last_user
        })

        activity.save();      
      });
    }
  });
};

我想要最终得到的是每本书的文档，其中包含一系列签出/签入，每次有人签入或签出一本书时都会更新。即：

{
    book_id: "5058c5ddeeb0a3aa253cf9d4",
    actions: [
        { checked_in: true, last_user: 'ralph' },
        { checked_in: true, last_user: 'gonzo' },
        { checked_in: true, last_user: 'animal' }
    ]
}

最终我将在每个条目中都有一个时间戳。

【问题讨论】：

在您的代码更新后，您需要帮助解决哪些具体问题？
它不会创建新的活动，可能是因为我使用的是 findById。但如果我只使用 .save() ，我最终会得到很多相同书籍 ID 的重复文档。

标签： javascript node.js mongodb express mongoose

【解决方案1】：

有几个问题：

您正在尝试使用findById 查找图书的活动文档，并使用图书的 ID 而不是活动的 ID。
您没有处理图书活动文档尚不存在的情况。

试试这个：

Activity.findOne({book_id: book._id}, function (err, activity) {
  if (!activity) {
    // No Activity doc for the book yet, create one.
    activity = new Activity({book_id: book._id});
  }
  activity.actions.push({
    checked_in: req.body.checked_in,
    last_user: req.body.last_user
  });

  activity.save();
});

【讨论】：

这实际上很有意义（并且有效）。虽然我认为查询应该是 {book_id: book.id} 而不是 book._id 我收集的。我脑子里有个愚蠢的想法，Mongoose 会自动从相应的 ID 中找出问题。我很快就会用我所学到的一切来概括一下。谢谢:)
@Crungmungus book.id 是 Mongoose 默认添加的字符串化 alias 到 book._id。所以两者都可以，但 book.id 效率不高，因为 Mongoose 必须将其转换回 ObjectId 才能执行 findOne 查询。

【解决方案2】：

我看到了一些可以改进的地方......

Activity 模型中的book_id 字段应为Schema.ObjectId，而不是String。然后，您可以根据需要使用填充。
您没有在 exports.update 中进行任何错误检查。如果用户传入了无效的id，您将需要检查book 是否未定义，以及常见的if (err) return next(err)（这要求您的函数参数为res, res, next）。
当您在exports.update 中创建活动时，您希望使用book._id 而不是book.id
不需要所有return 语句
device 变量未在任何地方声明，我不确定您要保存什么...我想您的意思是 book 那里。

然后您可以只.save() 活动而不是执行Activity.update。

【讨论】：