如何处理for循环中的回调(Node.JS)

时间:2015-03-30 09:51:45

标签: node.js mongodb mongoose

我正在尝试使用NodeJS编写代码,我从外部API中获取数据,然后使用Mongoose在MongoDB中填充它们。在这之间,我将检查Mongo中是否已存在该特定内容。以下是我的代码。

router.route('/report') // the REST api address
  .post(function(req, res) // calling a POST 
  {
    console.log('calling report API');
    var object = "report/" + reportID; // related to the API
    var parameters = '&limit=100' // related to the API
    var url = link + object + apiKey + parameters; // related to the API

    var data = "";
    https.get(url, function callback(response)
    {
      response.setEncoding("utf8");
      response.on("data", function(chunk)
      {
        data += chunk.toString() + ""; 
      });

      response.on("end", function()
      {
        var jsonData = JSON.parse(data);
        var array = jsonData['results']; // data is return in array of objects. accessing only a particular array
        var length = array.length;
        console.log(length);

        for (var i = 0; i < length; i++) 
        {
          var report = new Report(array.pop()); // Report is the schema model defined. 
          console.log('^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^');
          console.log(i);
          console.log('*****************************');
          console.log(report);
          console.log('*****************************');
          // console.log(report['id']);

          /*report.save(function(err)
          {
            if(err)
              res.send(err);
          });*/

          Report.find({id:report['id']}).count(function(err, count) // checks if the id of that specific data already exists in Mongo
          {
            console.log(count);
            console.log('*****************************');
            if (count == 0) // if the count = 0, meaning not exist, then only save
            {
              report.save(function(err)
              {
                console.log('saved');
                if(err)
                  res.send(err);
              });
            }
          });
        };
        res.json({
                    message: 'Grabbed Report'
                  }); 
      });
      response.on("error", console.error);
    });
  })

我的问题是,由于NodeJS回调是并行的,因此不会按顺序调用它。我的最终结果将是这样的:

  1. 调用报告API
  2. console.log(length)= 100
  3. ^^^^^^^^^^^^^^^^^^^^^^^^
  4. console.log(i)=以0开头
  5. *********
  6. console.log(report)=将存储在Mongo中的数据
  7. *********
  8. 数字3 - 7重复100次,因为长度等于100
  9. console.log(count)= 0或1
  10. 数字9重复100次
  11. console.log(&#39;已保存&#39;)
  12. 11号重复100次
  13. 最后,只有100个数据中的最后一个存储在Mongo
  14. 我需要的是处理这些回调的某种技术或方法,这些回调是一个接一个地执行而不是顺序跟随循环。我很确定这是问题,因为我的其他REST API都在工作。

    我已经研究过异步方法,承诺,递归函数以及其他一些我真正理解如何解决这个问题的方法。我真的希望有人可以对这件事情有所了解。

    如果我在提出问题的方式上犯了任何错误,请随意纠正我。这是我在StackOverflow中发布的第一个问题。

2 个答案:

答案 0 :(得分:5)

此问题称为"callback hell"。 还有许多其他方法,例如使用您可以找到的PromiseAsync库。

我对本地人async ES7会带来更多的兴奋, 您现在可以使用转发器库Babel实际开始使用它。

但到目前为止,我发现的最简单方法如下: 您取出长回调函数并在外部定义它们。

router.route('/report') // the REST api address
    .post(calling_a_POST)

function calling_a_POST(req, res) {
    ...
    var data = "";
    https.get(url, function callback(response) {
        ...
        response.on("end", response_on_end_callback); // --> take out
        response.on("error", console.error);
    });
}

function response_on_end_callback() {                 // <-- define here
    ...
    for (var i = 0; i < length; i++) {
        var report = new Report(array.pop());
        ...
        Report.find({ id: report['id'] })
              .count(Report_find_count_callback);     // --> take out
    };
    res.json({
        message: 'Grabbed Report'
    });
}

function Report_find_count_callback(err, count) {     // <-- define here
    ...
    if (count == 0) {
        report.save(function(err) {                   // !! report is undefined here
            console.log('saved');
            if (err)
                res.send(err);                        // !! res is undefined here
        });
    }
}

需要注意的是,您无法访问曾经是回调的所有变量, 因为你已将它们带出了范围。

这可以通过&#34;依赖注入来解决&#34;包装类型以传递所需的变量。

router.route('/report') // the REST api address
    .post(calling_a_POST)

function calling_a_POST(req, res) {
    ...
    var data = "";
    https.get(url, function callback(response) {
        ...
        response.on("end", function(err, data){       // take these arguments
            response_on_end(err, data, res);          // plus the needed variables
        });
        response.on("error", console.error);
    });
}

function response_on_end(err, data, res) {  // and pass them to function defined outside
    ...
    for (var i = 0; i < length; i++) {
        var report = new Report(array.pop());
        ...
        Report.find({ id: report['id'] })
            .count(function(err, count){
                Report_find_count(err, count, report, res);  // same here
            });
    };
    res.json({                                        // res is now available
        message: 'Grabbed Report'
    });
}

function Report_find_count(err, count, report, res) {        // same here
    ...
    if (count == 0) {
        report.save(function(err) {                   // report is now available
            console.log('saved');
            if (err)
                res.send(err);                        // res is now available
        });
    }
}

  

当我执行response_on_end函数时,我收到undefined:1 unexpected token u错误。   我非常确定它与这一行有关:var jsonData = JSON.parse(data)   我的response_on_end如下:var jsonData = JSON.parse(data); // problem here

我意识到我在这里犯了一个错误:

function calling_a_POST(req, res) {
    ...
    var data = "";
    https.get(url, function callback(response) {
        ...
        //sponse.on("end", function(err, data){
        response.on("end", function(err){ // data shouldn't be here
            response_on_end(err, data, res);
        });
        response.on("error", console.error);
    });
}

我可以预见的另一个问题,实际上可能不会出现在这里,但仍然会更好地谈论。 data变量,因为它是一个与对象不同的原始类型的字符串,它是通过值&#34;传递的。 More info

最好将变量包装在一个对象中并传递该对象,因为javascript中的对象始终通过引用&#34;传递。

function calling_a_POST(req, res) {
    ...
    // var data = ""; // 
    var data_wrapper = {};
    data_wrapper.data = {};                                // wrap it in an object
    https.get(url, function callback(response) {
        ...
        response.on("data", function(chunk){
            data_wrapper.data += chunk.toString() + "";   // use the dot notation to reference
        });
        response.on("end", function(err){ 
            response_on_end(err, data_wrapper, res);      // and pass that object
        });
        response.on("error", console.error);
    });
}

function response_on_end_callback(err, data_wrapper, res) {
    var data = data_wrapper.data;                         // later redefine the variable
    ...
    for (var i = 0; i < length; i++) {
        var report = new Report(array.pop());
        ...

答案 1 :(得分:0)

您可以使用async库来控制执行流程。还有用于处理数组的迭代器。