Question

尝试在Node.js和MongoDB中查找唯一ID，方法是创建一个while循环，在MongoDB中查询现有ID，直到找到唯一值。如果ID已被使用，则数字会在末尾递增，直到Mongo不返回任何内容。

一切正常，但找到唯一ID时break;语句除外。 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);
    }

});

我做错了什么？

编辑：

如果有人需要，这是使用异步的修复代码：）

        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);
            }
        });

Answer 1

您的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不在循环中！我还以其他方式使事情中断，因为uniqueNumber和newUnique值不再在范围内，但这是一个不同的问题。 :)这里要看的重要一点是，函数在代码中引入了一个“硬”边界，很难根据语言的语法很难看到。这就是为什么这种回调编程风格如此难以实现的原因之一。

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

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

非法中断语句（Node.js）

1 个答案: