为什么我的诺言没有得到解决？

Question

我有两个功能-一个下载文件的辅助功能，如下

var downloadHelper = function(url, saveDir) {
    var deferred = Q.defer();

    setTimeout(function() {
        deferred.resolve("success");    
    }, 2000);


    return deferred.promise;
}

现在，我有了要并行下载的文件列表。 我对该函数的逻辑如下：

var downloadAll = function() {
    var fileDownloadList = []
    for(var key in config.files) {

        var deferred = Q.defer();
        var saveLocation = __base + config.localDir
        downloadHelper(
            config.files[key], 
            saveLocation
        ).then(function() {
            deferred.resolve("downloaded: " + fileUrl);
        }).catch(function(err) {
            deferred.reject(err);
        });

        fileDownloadList.push(deferred.promise);
    }

    Q.all(fileDownloadList).done(function() {
        console.log("All downloaded");
    },function(err) {
        console.log(err);
    });

    setTimeout(function() {
        console.log(fileDownloadList);
    }, 10000);
}

完成永远不会被召唤！

出于调试目的，我添加了一个setTimeout，它将在10秒后调用，并且我看到的是2个文件中的第2个承诺已解决，第一个仍处于待处理状态。

有任何想法吗？

提前致谢

Answer 1

一种使代码正常工作的方法

for(var key in config.files) {
    (function() {
        var deferred = Q.defer();
        var saveLocation = __base + config.localDir
        downloadHelper(
            config.files[key], 
            saveLocation
        ).then(function() {
            deferred.resolve("downloaded: " + fileUrl);
        }).catch(function(err) {
           deferred.reject(err);
        });
        fileDownloadList.push(deferred.promise);
    }());
}

但是由于downloadhelper返回了一个承诺，因此无需创建另一个

for (var key in config.files) {
    var saveLocation = __base + config.localDir
    fileDownloadList.push(downloadHelper(
        config.files[key],
        saveLocation
    ).then(function () {
        return("downloaded: " + fileUrl);
    }));
} 

你会看到我被删除

.catch(function(err) {
    deferred.reject(err);
})

这是多余的，就像根本没有抓住一样