一样
我有两个函数 - 用于下载文件的辅助函数,如下所示
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 个文件中,第二个承诺已解决,第一个承诺仍处于挂起状态。
有什么想法吗?
提前致谢
使代码正常工作的一种方法
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);
})
这是多余的,这和根本没有捕获