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

时间:2018-07-12 20:41:38

标签: javascript node.js mongodb

此功能可以搜索测试问题并匹配用户给出的答案并存储在数据库中。 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);
                        }
                    }
                })
            }
        }
    })
}

1 个答案:

答案 0 :(得分:1)

我同意CRice对此的看法。在回调的else中,您尝试运行一个同步forEach循环,但在其中运行一个异步代码块(即:Testques.find),这种方式不起作用您希望它能正常工作。

一个优雅的解决方案是,使您的猫鼬呼叫(使用可用的实用工具)传播,然后一旦实现,就可以使用Promise.all来解决Testques.find的这些排队承诺的数组可能会推入其中。

否则,您还可以按照以下步骤进行操作:将函数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);
            }
        });
}