我有一个问题需要在我的代码中解决,这与缓存API结果有关。我有一些像这样的async.map:
for(var user of allUsers) {
var requestList = fill(cache, locations, apiUrl, cacheMap);
async.map(requestList, function(obj, callback) {
// ...
}, function(err, results) {
// PUT RESULTS INTO A CACHE
});
}
函数fill
只是在缓存中查看locations
中的位置是否存在,而不是为我的API创建请求URL。
然而我意识到缓存在我的方法中根本不会用得多,因为我的代码将调度async.map并立即开始下一个循环迭代fill
,这意味着缓存赢了在每次用户迭代时都要同步。
我如何确保用户的每次迭代都有来自上一个用户的缓存的更新版本?我需要非常智能地使用我有限的API调用,所以如果有重复的请求我想要请求一次,那么在以后的请求中从缓存中提取该结果。
我现在唯一的做法就是同步请求而不是async.map,但我知道这违反了node.js的设计。
for(var user of allUsers) {
var requestList = fill(cache, locations, apiUrl, cacheMap);
// sync map instead
requestList.map(function(obj) {
var res = sync-request(obj)
// put request result into cache
});
// cont...
}
答案 0 :(得分:1)
使用Promises代理和缓存API调用。使用Promises批处理API请求和缓存结果非常简单。以下小代码模块在Promise中包含现有的expensiveAPI
调用,并将解析后的结果缓存60秒。
// Existing module to call the expensive API
// using the standard callback pattern
var expensiveApi = require("./expensiveApi");
// Using bluebird promise library
// See http://bluebirdjs.com/docs/api-reference.html
var Promise = require("bluebird");
// Promisify the existing callback
expensiveApi = Promise.promisify(expensiveApi);
// Calls to the API will now return a Promise
// A cache for the Promises
var cache = {};
module.exports = function expensiveApiWithPromises(item) {
// Check whether a cached Promise already exists
if (cache[item]) {
// Return it to the caller
return cache[item];
}
// Promise is not in the cache
cache[item] = expensiveApi(item)
.then(function(result) {
// Promise has resolved
// set the result to expire
setTimeout(function() {
delete cache[item];
}, 60 * 1000); // 60 seconds expiry
// Return the result
return result;
})
.catch(function(err) {
// Promise rejected with an error
// Reset the cache item
delete cache[item];
// Propagate the error
throw err;
});
// Return the newly created cached Promise
return cache[item];
}
答案 1 :(得分:1)
您可以使用async.eachSeries迭代allUsers
。这将按顺序逐步执行并保持异步。
async.eachSeries(allUsers, (user, done) => {
const requestList = fill(cache, locations, apiUrl, cacheMap);
async.map(requestList, (obj, callback) => {
// ..
callback(null, requestResult);
}, (err, results) => {
// PUT RESULTS INTO A CACHE
done(null, results);
});
}, cb);