我正在使用生成器和Bluebird编写代码,我有以下内容:
var async = Promise.coroutine;
function Client(request){
this.request = request;
}
Client.prototype.fetchCommentData = async(function* (user){
var country = yield countryService.countryFor(user.ip);
var data = yield api.getCommentDataFor(user.id);
var notBanned = yield authServer.authenticate(user.id);
if (!notBanned) throw new AuthenticationError(user.id);
return {
country: country,
comments: data,
notBanned: true
};
});
然而,这有点慢,我觉得我的应用程序等待I / O太多而且它不是并行的。如何提高应用程序的性能?
countryFor
的总响应时间为800 getCommentDataFor
+ 400 + authenticate
的总响应时间为600,因此总共需要1800毫秒。
答案 0 :(得分:17)
您花费了太多时间等待来自不同来源的I / O.
在正常的承诺代码中,您可以使用Promise.all
,但是 - 人们倾向于编写等待生成器请求的代码。您的代码执行以下操作:
<-client service->
countryFor..
''--..
''--..
''--.. country server sends response
..--''
..--''
..--''
getCommentDataFor
''--..
''--..
''--..
''--.. comment service returns response
..--''
..--''
..--''
authenticate
''--..
''--..
''--.. authentication service returns
..--''
..--''
..--''
Generator done.
相反,应该这样做:
<-client service->
countryFor..
commentsFor..''--..
authenticate..''--..''--..
''--..''--..''--.. country server sends response
''--..--''.. comment service returns response
..--''..--''.. authentication service returns response
..--''..--''..
..--''..--''..--''
..--''..--''
..--''
Generator done
简单地说,所有的I / O应该在这里并行完成。
要解决此问题,我会使用Promise.props
。 Promise.props
获取一个对象并等待其所有属性解析(如果它们是promises)。
记住 - 生成器和承诺混合和匹配真的,你只需产生承诺:
Client.prototype.fetchCommentData = async(function* (user){
var country = countryService.countryFor(user.ip);
var data = api.getCommentDataFor(user.id);
var notBanned = authServer.authenticate(user.id).then(function(val){
if(!val) throw new AuthenticationError(user.id);
});
return Promise.props({ // wait for all promises to resolve
country : country,
comments : data,
notBanned: notBanned
});
});
这是人们第一次使用发电机时常犯的错误。
ascii艺术无耻地从Q-Connection获取Kris Kowal
答案 1 :(得分:11)
正如Promise.coroutine
的Bluebird文档中提到的那样,您需要注意不要在系列中yield
。
var county = yield countryService.countryFor(user.ip); var data = yield api.getCommentDataFor(user.id); var notBanned = yield authServer.authenticate(user.id);
此代码有3个yield
表达式,每个表达式都会停止执行,直到确定特定的承诺。代码将连续创建并执行每个异步任务。
要等待多个并行任务,您应该yield
承诺数组。这将等到所有这些都结算,然后返回结果值数组。使用ES6解构分配可以得到简洁的代码:
Client.prototype.fetchCommentData = async(function* (user){
var [county, data, notBanned] = yield [
// a single yield only: ^^^^^
countryService.countryFor(user.ip),
api.getCommentDataFor(user.id),
authServer.authenticate(user.id)
];
if (!notBanned)
throw new AuthenticationError(user.id);
return {
country: country,
comments: data,
notBanned: true
};
});
答案 2 :(得分:4)
Benjamin Gruenbaum的回答是正确的,但它完全失去了生成器方面,当你试图并行运行多个东西时,这往往会发生一些变化。但是,您可以使用yield
关键字使这项工作正常。我还使用了一些额外的ES6功能,例如destructuring assignments和object initializer shorthand:
Client.prototype.fetchCommentData = async(function* (user){
var country = countryService.countryFor(user.ip);
var data = api.getCommentDataFor(user.id);
var notBanned = authServer.authenticate(user.id).then(function(val){
if(!val) throw new AuthenticationError(user.id);
});
// after each async operation finishes, reassign the actual values to the variables
[country, data, notBanned] = yield Promise.all([country, data, notBanned]);
return { country, data, notBanned };
});
如果您不想使用这些额外的ES6功能:
Client.prototype.fetchCommentData = async(function* (user){
var country = countryService.countryFor(user.ip);
var data = api.getCommentDataFor(user.id);
var notBanned = authServer.authenticate(user.id).then(function(val){
if(!val) throw new AuthenticationError(user.id);
});
var values = yield Promise.all([country, data, notBanned]);
return {
country: values[0],
data: values[1],
notBanned: values[2]
};
});