javascript - Node.js console.log() 给出了不可预测的答案

Question

这是一个搜索测试问题并匹配用户给出并存储在数据库中的答案的功能。 Console.log() 显示所有 6 个问题，但顺序是随机的。每次迭代的 num 值为 6。如果我执行 console.log(num) 而没有从数据库中找到任何内容，那么我会正确显示值 1,2,3,4,5,6。

function(req,res){
    var arr = [2,1,3,4,1,4],score=0, num=0;
    Test.find({name:req.params.testnum}).
    populate({
        path: 'question',
        model: 'Testques'
    }).
    exec(function(err,test){
        console.log(test)
        if(err){
            res.status(500).send('DB error');
        }
        else{
            arr.forEach(myFunction)
            function myFunction(value){
                num+=1;
                Testques.find({Serialnum:num},function(err,ques){
                    console.log(num);
                    if(err){
                        console.log(err);
                    }
                    else{
                        console.log(ques);
                        console.log(ques[0].answer);
                        if(ques[0].answer == value){
                            score=score+4;
                            console.log(score);
                        }
                    }
                })
            }
        }
    })
}

Answer 1

我同意CRice不得不说一下。在回调的 else 内部，您尝试运行同步 forEach 循环，但您正在内部运行异步代码块(即: Testques.find )它不会按照您希望的方式工作。

一个优雅的解决方案可能是， promise 您的 Mongoose 调用(使用可用的 promise 实用程序)，然后，一旦 promise ，要么使用 Promise.all 来解析 Testques.find这些排队的 promise 的数组 您可以将其插入其中。

否则，您也可以按照以下方式进行操作:将 forEach 参数内的函数移动到该方法范围之外的另一个方法，然后使用递归的基本知识来实现​​您想要的目的。它应该类似于:

function cbIterator($index, arr, num, score) {
    if ($index < arr.length) {
        const value = arr[$index];
        num++;
        Testques.find({ Serialnum: num }, function (err, ques) {
            console.log(num);
            if (err) {
                console.log(err);
            }
            else {
                console.log(ques);
                console.log(ques[0].answer);
                if (ques[0].answer === value) {
                    score = score + 4;
                    console.log(score);
                }
                cbIterator($index + 1, arr, num, score);
            }
        });
    }
    return;
}

function myTestFunction(req, res) {
    // A bit of ES6. Feel free to replace the const / let with var if you are not comfortable with those
    const arr = [2, 1, 3, 4, 1, 4];
    let score = 0, num = 0;
    Test.find({ name: req.params.testnum })
        .populate({
            path: 'question',
            model: 'Testques'
        }).exec(function (error, test) {
            if (error) {
                res.status(500).send('DB Error');
            } else {
                let $index = 0;
                cbIterator($index, arr, num, score);
            }
        });
}