2017-04-17 58 views
1

问题:数据INFO_X有时随机成为空。NodeJS变量碰撞?与请求异步/同步

问题,

是否可变覆盖彼此INFO_1 INFO_2 INFO_3因为运行的NodeJS快速不像PHP它遵循一个序列/步步。

我在执行请求之前检查了NULLS,但在执行第二个请求之前调试显示它不是NULL,随机,随机变量在提交第二个请求时将变为空。

我也检查了我的源码绝对不是返回null。

在发送第二个请求之前变量被覆盖或者是什么?请指教。

var request = require('request'); 

var urls = [ 'URL', 
      'URL', 
      'URL']; 

urls.forEach(processUrl); 

function processUrl(url) { 

    request(url, function (error, response, body) { 
    if (!error) { 
    var obj = JSON.parse(body); 
    for (var i = 0, len = obj['products'].length; i < len; ++i) { 
     var data = obj['products'][i]; 
     var INFO_1 = data.INFO_1 
     var INFO_2 = data.INFO_2 
     var INFO_3 = data.INFO_3 

     request("URL/POSTINFO?INFO_1="+INFO_1+"&INFO_2="+INFO_2+"&INFO_3="+INFO_3+"&seller_id=", function(error, response, body) { 
      console.log(body); 
     }); 

    } 
    } 
}); 

} 

回答

2

是的,就是这样,因为请求函数是异步的。我不会调用nodejs比PHP更快,它只是运行异步request方法,而PHP通常是同步的。

您可以通过承诺解决问题,例如, Promise.all([])并提供请求功能的阵列(在这里看到:https://developer.mozilla.org/en/docs/Web/JavaScript/Reference/Global_Objects/Promise/all),或在你的情况,因为你是在request功能使用回调使用这个库https://github.com/caolan/async#forEach

,你最好的选择是使用async在所提供的上面的链接。下面是一个例子代码:

function request(url, cb) { 
     setTimeout(() => { 
     cb(url + ' accessed at ' + new Date()); 
     }, 2000); 

    } 

    var urls = ['URL1', 'URL2', 'URL3']; 

    async.each(urls, (item) => { 
     console.log(item); 
     request(item, (value) => { 
     request(value, (newValue)=>{ 
      console.log(newValue); 
     }); 
     }); 
    }, (err) => { 
     console.log(err); 
    }); 

这里有一个工作示例:https://plnkr.co/edit/Q5RAvKdaLxV9cUT4GP4w?p=preview

+0

请求功能是异步,的NodeJS心不是异步,因此我需要使用承诺运行要求这是一个异步功能时,使我的NodeJS代码异步?我说得对吗? – CodeGuru

+0

就是这样的。 '.forEach'基本上是调用异步函数,我会提供一个简单的例子,我会在几分钟内更新我的答案。 – uglycode

+0

好的,我提供了一个基本的简单例子。请让我知道这对你有没有用。 – uglycode