如何确保将正确的参数传递给循环中声明的函数调用？

Question

我是 node 新手，但我已经喜欢它了。唯一的问题是，异步功能正在杀死我。

我正在使用 google 包来获取我的网站在 google 中的排名：

for (var j=0;j<keywords.length;j++) {
    var keyword = keywords[j];

    google(keyword, function(err, next, links) {

    console.log('Searching for keyword "' + keyword + '" in google.' + google.tld + ' ('+ google.lang +')');
      if (err) console.error(err);

      for (var i = 0; i < links.length; ++i) {

        var rank = i+1;

        console.log(keyword + ' #'+ rank + ' - ' + links[i].link + ' | ' + links[i].title);
        //link.href is an alias for link.link
        //console.log(links[i].description + "\n");
      }
      console.log('\n');
    });

}

我的问题是控制台日志显示所有网站的关键字都相同，尽管我在关键字数组中定义了三个。

我错过了什么？

Answer 1

这里最直接的问题是j 和keyword 变量在回调被调用之前一直在变化。

一个简单的解决方案是在闭包中保护这个变量：

for (var j=0; j<keywords.length; j++) {
    (function(j){
        var keyword = keywords[j];
        ...
    })(j);
}

当您只使用简单数组时，您还可以通过forEach 使用闭包：

keywords.forEach(function(keyword, j){
     ...
});

要处理这种异步问题，您现在应该深入研究 Promise，它有助于以更清晰（且缩进更少）的方式构建您的代码。 An introduction.