9

尝试在 Node.js 和 MongoDB 中找到唯一 ID，方法是创建一个 while 循环，在 MongoDB 中查询现有 ID，直到找到唯一值。如果 ID 已经在使用中，最后会增加一个数字，直到 Mongo 什么也不返回。

一切正常，除了break;找到唯一 ID 时的声明。Node.js 返回：SyntaxError: Illegal break statement

编码：

db.collection('landmarks').findOne({'id':uniqueIDer}, function(err, data){
    //if ID exists already
    if (data.id){

        var uniqueNumber = 1;

         while (1) {

            var uniqueNum_string = uniqueNumber.toString(); 
            var newUnique = data.id + uniqueNum_string;
            db.collection('landmarks').findOne({'id':newUnique}, function(err, data){

                if (data.id){
                    uniqueNumber++;
                }

                else {
                    saveLandmark(newUnique);
                    break;
                }
            });
        }
    }

    else {
        saveLandmark(uniqueIDer);
    }

});

我究竟做错了什么？

编辑：

如果有人需要，这是使用 async 的固定代码:)

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

            if (data){
                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,'world':worldVal}, 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);
                });
            }
            else {
                saveLandmark(uniqueIDer);
            }
        });
4

1 回答 1

18

您的break语句不在循环体内。相反，它位于函数体内，即findOne回调。为了更清楚地看到这一点，临时使用命名函数作为回调处理程序会很有帮助：

var cb = function(err, data){
    if (data.id){
        uniqueNumber++;
    }
    else {
        saveLandmark(newUnique);
        break;  // not inside a loop!
    }
};

db.collection('landmarks').findOne({'id':uniqueIDer}, function(err, data){
    //if ID exists already
    if (data.id){
        var uniqueNumber = 1;
        while (1) {
            var uniqueNum_string = uniqueNumber.toString(); 
            var newUnique = data.id + uniqueNum_string;
            db.collection('landmarks').findOne({'id':newUnique}, cb);
        }
    }
    else {
        saveLandmark(uniqueIDer);
    }
});

现在很清楚，break回调函数体中的 in 不在循环中！我还以其他方式破坏了事情，因为uniqueNumberandnewUnique值不再在范围内，但这是一个不同的问题。:) 这里要看到的重要一点是，函数在代码中引入了“硬”边界，纯粹基于语言的语法很难看到。这就是为什么这种回调编程风格很难做到正确的原因之一。

事实上，做到这一点比您最初对代码的尝试所暗示的要困难得多。当您重复调用findOne和分析结果（异步）时，您需要有一种方法通过可能的任意回调层向上传递成功信号。

您可以通过使用优秀的async库获得一些帮助，例如https://github.com/caolan/async#whilst。

于 2013-08-02T18:36:30.017 回答