非法 break 语句 (Node.js)

Question

尝试在 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);
            }
        });

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。