1

根据上一个问题Illegal break statement (Node.js)的建议，我实现async.whilst()了，但它没有多次迭代。

我试图通过在 ID 末尾增加一个数字并查询 Mongo 以查看该 ID 是否存在来找到一个唯一 ID。如果不存在，则找到唯一 ID。它只循环一次，而不是直到找到一个唯一的。怎么了？

编码：

 var uniqueNumber = 1;
 var newUnique;

 async.whilst(
    function () { 

       var uniqueNum_string = uniqueNumber.toString(); 
       newUnique = data.id + uniqueNum_string;

       db.collection('landmarks').findOne({'id':newUnique}, function(err, data){

           if (data){
              console.log('entry found!');
              return;
           }

           else {
              console.log('entry not found!');

           }
        });

  },
  function (callback) {

     uniqueNumber++;

   },
   function (err) {

      saveLandmark(newUnique);
   }
);
4

2 回答 2

6

我实际上找不到适合这个任务的好的异步函数，所以我使用 async.forever() 一起破解了一些东西。该函数将继续运行，直到您回调“错误”，这是您想要做的。

var uniqueNumber = 1;
var newUnique;

async.forever(function (next) {
  var uniqueNum_string = uniqueNumber.toString(); 
  newUnique = data.id + uniqueNum_string;

  db.collection('landmarks').findOne({'id':newUnique}, function(err, data){
    if (data){
      console.log('entry found!');
      uniqueNumber++;
      next();
    }

    else {
      console.log('entry not found!');
      next('unique!'); // This is where the looping is stopped
    }
  });
},
function () {
  saveLandmark(newUnique);
});

关于您要解决的问题，在我看来，您想插入一个具有唯一 ID 的新文档。如果是这种情况并且你会经常这样做，我会说这是一种非常低效的方法。如果您在数据库中有一千个文档，您甚至会在接近唯一 ID 之前对数据库执行一千个完全无意义的请求。

更好的方法是从集合中获取按 id 降序排序的第一个文档（例如，最高的 id）。然后将该 id 加一并尝试插入，直到它不被拒绝。因为即使您找到一个唯一的 id，当您保存文档时，可能已经从另一个客户端或另一个实例进行了另一个插入（在负载平衡的情况下）。在您的情况下，这可能是也可能不是问题，我对您的应用程序了解不够，我只是认为您应该意识到您当前的方法和我的答案的缺点。

于 2013-08-02T21:00:12.483 回答
1

像这样的东西？我没有测试它，因为我不知道你使用的是什么数据库模块，但逻辑应该是显而易见的。

function searchNubmersForResults(firstNumber, callback) { 
    //place other variables here, and you can collect the results within the closure, and send them as arguments to your callback
   function testNumber(uniqueNumber) {
        var uniqueNum_string = uniqueNumber.toString(); 
        newUnique = data.id + uniqueNum_string;

        db.collection('landmarks').findOne({'id':newUnique}, function(err, data){

            if (data){
                console.log('entry found!');
                callback(data);//We're done, use the callback on the data
            } else {
                console.log('entry not found!');
                testNumber(uniqueNumber++);//Launch the next test
            }
        });
    }

    testNumber(firstNumber);//Laucn the first test
}

searchNubmersForResults(0, function(data) {
    console.log('You have data now: ' + data);
});
于 2013-08-02T20:40:40.030 回答