1

我正在尝试遍历数组并将数组中的项目添加到 mongodb 数据库（如果它不存在）。在循环内部，如果数组中的下一个元素已经在数据库中，我会尝试查询我的数据库。但是在回调函数中，console.log 总是返回数组的最后一个元素。

for(var j = 0 ; j < req.body.array.length; j++ ){
    var currenttag = req.body.array[j];
    Tag.findOne({tagname : currenttag},
      function(err,tag){
        if(tag){
          console.log("tag exists");
        }
        else{
            console.log(currenttag);//returns the last tag in the array for array.length times
            var newtag  = new Tag({tagname : currenttag});
            newtag.save();
        }
      });
  }
4

1 回答 1

3

我认为该错误与 Javascript 中如何处理闭包有关。有关这意味着什么的更多详细信息，请在此处查看我的答案。

在您的情况下，事件处理程序正在获取对 的引用currentTag，该引用设置为循环的最后一个元素。

要解决此问题，您可以尝试执行以下操作：

var createTagIfMissing = function (currentTag){

Tag.findOne({tagname : currenttag},
      function(err,tag){
        if(tag){
          console.log("tag exists");
        }
        else{
            console.log(currenttag);
            var newtag  = new Tag({tagname : currenttag});
            newtag.save();
        }
      });
}

for(var j = 0 ; j < req.body.array.length; j++ ){
  createTagIfMissing(req.body.array[j]);
}

您可以进一步推动这一点，并将此方法定义为您的标签静态之一：

//In your db.js file, before mongoose.model('Tag',Tag);
Tag.statics.createTagIfMissing = function(tag){
    this.findOne //...
}

然后，在您的路由器代码中：

for(var j = 0 ; j < req.body.array.length; j++ ){
  Tag.createTagIfMissing(req.body.array[j]);
}
于 2013-06-26T19:06:57.163 回答