由于异步生成器中的非并行等待承诺而导致速度下降

时间:2014-06-12 20:44:21

标签: javascript async-await promise generator bluebird

我正在使用生成器和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毫秒。

3 个答案:

答案 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.propsPromise.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 assignmentsobject 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]
    };
});