Question

我的Sailsjs应用程序中有以下模型，它们具有多对多关系：

event.js：

attributes: {
 title : { type: 'string', required: true },
 description : { type: 'string', required: true },
 location : { type: 'string', required: true },
 maxMembers : { type: 'integer', required: true },
 currentMembers : { collection: 'user', via: 'eventsAttending', dominant: true },
 creator : { model: 'user', required: true },
 invitations : { collection: 'invitation', via: 'eventID' },
 tags : { collection: 'tag', via: 'taggedEvents', dominant: true },
 lat : { type: 'float' },
 lon : { type: 'float' },
},

tags.js：

attributes: {
 tagName : { type: 'string', unique: true, required: true },
 taggedEvents : { collection: 'event', via: 'tags' },
},

根据文档，这种关系看起来是正确的。我在tag.js中有以下方法，它接受一个标记字符串数组和一个事件id，并且应该添加或删除传入的标记：

modifyTags: function (tags, eventId) {
var tagRecords = [];

_.forEach(tags, function(tag) {
    Tag.findOrCreate({tagName: tag}, {tagName: tag}, function (error, result) {
        tagRecords.push({id: result.id})
    })
})

Event.findOneById(eventId).populate('tags').exec(function(error, event){
    console.log(event)
    var currentTags = event.tags;
    console.log(currentTags)
    delete currentTags.add;
    delete currentTags.remove;

    if (currentTags.length > 0) {
      currentTags = _.pluck(currentTags, 'id');
    }

    var modifiedTags = _.pluck(tagRecords, 'id');
    var tagsToAdd = _.difference(modifiedTags, currentTags);
    var tagsToRemove = _.difference(currentTags, modifiedTags);
    console.log('current', currentTags)
    console.log('remove', tagsToRemove)
    console.log('add', tagsToAdd)

    if (tagsToAdd.length > 0) {
      _.forEach(tagsToAdd, function (tag) {
        event.tags.add(tag);
      })
        event.save(console.log)
    }

    if (tagsToRemove.length > 0) { 
      _.forEach(tagsToRemove, function (tagId) {
         event.tags.remove(tagId)
      })
       event.save()
    }      
})
}

这是从事件模型中调用方法的方法：

afterCreate: function(record, next) {
  Tag.modifyTags(tags, record.id)
  next();
}

当我发布到event / create时，我得到了这个结果：http://pastebin.com/PMiqBbfR。

看起来方法调用本身就是循环的，而不仅仅是tagsToAdd或tagsToRemove数组。更令人困惑的是，最后，在事件的最后一个日志中，看起来事件具有正确的标签。然后，当我发布到event / 1时，tags数组为空。我也尝试在每个.add()之后立即保存，但仍然会得到类似的结果。

理想情况下，我想循环使用tagsToAdd和tagsToRemove数组，在模型的集合中修改它们的ID，然后在模型上调用.save()一次。

我花了很多时间尝试调试这个，所以任何帮助都会非常感激！

Answer 1

您的实施存在一些问题，但主要问题是您正在将某些方法 - 即.save()和.findOrCreate 视为同步方法，当时它们（像所有Waterline方法一样）异步，需要回调。因此，您可以有效地并行运行一堆代码，而不是在返回之前等待它完成。

另外，既然你想要做的就是用这个新列表替换当前的事件标签，你提出的方法有点过度设计 - 你不要我需要使用event.tags.add和event.tags.remove。您可以使用普通的update。

所以你可以将modifyTags方法重写为：

modifyTags: function (tags, eventId, mainCb) {

  // Asynchronously transform the `tags` array into an array of Tag records
  async.map(tags, function(tag, cb) {
    // For each tag, find or create a new record.
    // Since the async.map `cb` argument expects a function with 
    // the standard (error, result) node signature, this will add
    // the new (or existing) Tag instance to the resulting array.
    // If an error occurs, async.map will exit early and call the
    // "done()" function below
    Tag.findOrCreate({tagName: tag}, {tagName: tag}, cb);
  }, function done (err, tagRecords) {
    if (err) {return mainCb(err);}
    // Update the event with the new tags
    Event.update({id: eventId}, {tags: tagRecords}).exec(mainCb);
  });

}

请参阅async.map here的完整文档。

如果您希望使用.add和.remove来坚持您的实施，您仍然希望使用async.map，并在done中执行其余逻辑方法。您不需要两个.save电话;只需先运行所有.add和.remove代码，然后执行一次.save(mainCb)即可完成此操作。

我不知道您要通过删除.add中的.remove和currentTags方法（这是对event.tags的直接引用）来实现的目标，但它不起作用，以后会引起混淆！

将多个记录添加到模型的集合Sailsjs中

1 个答案: