javascript - Node 数组异步错误

Question

这可能是异步调用之外的错误，在这种情况下，无论如何，一双新的眼睛可能是好的。

我正在编写一个与 flickr api 交互的函数。给定一个用户，它会获取他们的 ID，使用他们的 ID 来获取该用户的集合(相册)列表，并根据该集合列表，获取每个集合中的照片列表。

这是调用问题函数的代码(仅用于上下文；据我所知，这一切都有效):

router.post('/getPhotosFromSets', function(req, res) {
var uname = req.body.Username;
getIdFromUsername(uname, function(id) {
    console.log("ID Received: " + id);
    getSetsFromId(id, function(setList) {
        console.log("setList received: " + JSON.stringify(setList));
        getURLsFromSets(setList, function(){
            console.log("\n\nSetlist received: "+JSON.stringify(setList));
            res.render('userPhotos', {'setList': setList});
        });
    });
});

我知道嵌套异步调用不是最佳实践，但只有 3 个，所以它是可以管理的，而且我不想学习 async.js，除非真的必须这样做。

我知道 getID... 和 getSets... 可以工作。问题出在 getURLsFromSets 中。基本上我想做的是，从 getSetsFromId 获取列表的列表。像这样:

[[Album Name, Album ID],[Album2 Name, Album 2 ID]]. 

对于每个列表，我想附加一个照片 URL 列表。所以它看起来像:

[[Album Name, Album ID, [Photo1 URL, Photo2 URL]],[Album2 Name, Album 2 ID, [Photo1 URL]]. 

这是 getURLSFromSets 的代码:

getURLsFromSets = function(setList, callback) {
console.log("setList argument: "+setList);
var activeThreads = setList.length;
var onComplete = function() {
    callback(setList);
};

for(var i = 0; i < setList.length; i++) {
    var setInfo = setList[i];

    // URL is query to grab photos from a set
    var url = 'https://api.flickr.com/services/rest/?';
    url += 'method=flickr.photosets.getPhotos&api_key=';
    url += 'MY_API_KEY'; // API Key
    url += '&photoset_id='+setInfo[1];
    url += '&format=json&nojsoncallback=1';
    console.log("Set info before request: " + setInfo);

    request(url, function(error, response, body) {
        console.log("Set info at request: " + setInfo);
        var photos = JSON.parse(body).photoset.photo;
        setInfo.push(stringifyUrl(photos));
        if (--activeThreads === 0) {
            onComplete();
        }
    });
}
};

setList，所采用的参数，是我上面谈到的列表的列表。因此它想要遍历列表的 setList 中的每个列表，使用列表中的信息来获取 URLlist，并将 URLlist 附加到列表中。抱歉，如果不清楚。但我想要的最终结果是一个列表列表，其中每个子列表包含相册信息和照片列表。

现在，这就是问题出现的地方。当我为只有一张专辑的用户执行此操作时，此方法非常有效。例如，当我输入我自己的 ID djbhindi 时，我得到:

Setlist received: [["Desktops","72157645413282948",["https://farm6.staticflickr.com/5233/14669675871_4882577de4_b.jpg","https://farm4.staticflickr.com/3861/14486214499_b492c8c6c3_b.jpg"...]]]

这是一个只有 1 个子列表的列表，该子列表代表 1 个名为“desktops”、id“721576...”的相册以及所需的照片 URL 列表。

但是，当我输入一个拥有超过 1 个专辑的用户时(我随机发现了一个名为“joshuamalik”的专辑)，这就搞砸了:

Setlist received: [["365","72157630946802318"],["Images.","72157629614042425",["https://farm8.staticflickr.com/7106/7730306384_e07f5cff20_b.jpg",...],["https://farm8.staticflickr.com/7063/6994331163_e41e9687db_b.jpg","https://farm8.staticflickr.com/7108/7019359683_28dc6dc2e6_b.jpg"]]]

这可能不清楚，但基本上这个人有 2 张专辑，一张有 2 张照片，一张有 14 张。该方法的作用是根据需要返回包含 2 个子列表的列表(1 代表每个专辑)，但是第一个相册的照片 URL 列表将添加到第二个相册的列表中，然后第二个相册的列表也会添加到其中。

所以我最终得到:

[[Album 1 info] [ Album 2 info, [Album 1 photos], [Album 2 photos]]]

而不是

[[相册1信息，[相册1照片]]，[相册2信息，[相册2照片]]]

我的理论是，getUrsFromSets 中的异步调用在调用完成时已将 setInfo 绑定(bind)到专辑 2 子列表，因此它会附加到错误的内容。

有什么想法吗？非常感谢你们阅读整篇文章。我知道这不是授权问题，否则我不会从其他人那里取回照片网址。这只是一个格式问题。

Answer 1

关闭问题。基本上 setInfo 成为对最后迭代的 setList 元素的引用。将 ajax 回调包装在一个以正确 setInfo 作为参数的立即函数中是修复它的一种方法。尝试进行这些更改。

request(url, function(error, response, body) {
    console.log("Set info at request: " + setInfo);
    var photos = JSON.parse(body).photoset.photo;
    setInfo.push(stringifyUrl(photos));
    if (--activeThreads === 0) {
        onComplete();
    }
});

变成了

request(url, 
  (function (setInfo) {
    return function(error, response, body) {
      console.log("Set info at request: " + setInfo);
      var photos = JSON.parse(body).photoset.photo;
      setInfo.push(stringifyUrl(photos));
      if (--activeThreads === 0) {
        onComplete();
      }
    }
  })(setInfo)
);