[英]node.js: apply a list of async functions to each item of a list of lists
我有一个人的名单,每个人都包含一个图像URL列表。
我需要下载每个URL,然后为该图像计算一个“签名”,确认它是唯一的,然后将其保存到文件系统中。
我当前的方法是:两个嵌套的async.each
调用,和一个async.waterfall
以检查和保存图像(为简化起见,省略了错误处理):var async = require('async'); var request = require('request');
var syncPersonsImages = function(persons, images, callback) {
async.each(
persons,
function(person, callbackPerson) {
async.each(
person.imageUrls,
function(imageUrl, callbackImage) {
download(imageUrl, function(err, image) {
downloadPost(callbackImage);
});
},
function(err) {
callbackPerson();
}
);
},
function(err) {
callback(null, persons);
}
);
function download() {
request(
{ uri: image.url },
function(err) {
callback(err, image);
},
function(contents, res) {
image.contents = res.contents;
callback(null, image);
}
);
}
function downloadPost(image, callback) {
async.waterfall(
[
getSignatureFromImage,
findSimilarSignatureImage,
saveImage,
],
function (err, image) {
callback(image);
}
);
}
function getSignatureFromImage(image, callback) {
image.signature = crypto.createHash('md5').update(image.url).digest('hex');
callback(null, image);
}
function findSimilarSignatureImage(image, callback) {
if (existsAlready(image.signature)) { // this function is not shown but it's behaviour is clear...
image.isNew = true;
}
callback(null);
}
function save(image, callback) {
if (image.isNew) {
img.save(function(err) {
console.log('image', image.url, 'saved');
callback(null, image);
}
} else {
callback(null, null);
}
}
};
问题是syncPersonsImages
方法永远不会终止...
有什么建议吗?
我的方法至少在理论上是正确的吗?
总的来说,我认为这种方法是正确的。 有两点。 我认为您不必要地包装了一些回调。 例如,这:
function(person, callbackPerson) {
async.each(
person.imageUrls,
function(imageUrl, callbackImage) {
download(imageUrl, function(err, image) {
downloadPost(callbackImage);
});
},
function(err) {
callbackPerson();
}
);
},
function(err) {
callback(null, persons);
}
);
可以更改为:
function(person, callbackPerson) {
async.each(
person.imageUrls,
function(imageUrl, callbackImage) {
download(imageUrl, function(err, image) {
downloadPost(callbackImage);
});
}, callbackPerson);
}, callback);
...取决于您是否希望async.each调用在第一次出现错误时保释。
另外,您定义了不带参数的下载方法。
声明:本站的技术帖子网页,遵循CC BY-SA 4.0协议,如果您需要转载,请注明本站网址或者原文地址。任何问题请咨询:yoyou2525@163.com.