承诺 battlenet-api 库

Question

我正在尝试自定义承诺 battlenet-api 库，它具有以下函数签名：

function (args, callback)

其中callback的签名是：

function (err, body, res)

在这种特殊情况下，我需要在执行前向承诺添加速率限制器 (node-rate-limiter)，因此函数将如下所示：

function throttler(originalMethod) {
    // return a function
    return function throttled() {
        var args = [].slice.call(arguments);
        // Needed so that the original method can be called with the correct receiver
        var self = this;

        // which returns a promise
        return new Promise(function(resolve, reject) {
            // We call the originalMethod here because if it throws,
            // it will reject the returned promise with the thrown error
            rateLimiterByHour.removeTokens(1, function() {
                 rateLimiterBySecond.removeTokens(1, function() {
                     originalMethod.apply(self, args, function(err, body, res) {
                             if (err) { reject(err, body, res); }
                             else { resolve({err, body, res}); }
                         };
                     });
                 );
             });
        });
    };
};

Promise.promisifyAll(require('battlenet-api').wow, {
    promisifier: throttler
});

throttler 是

我完全不确定这是否可行，或者我是否应该采用其他方法。

我该如何适当地承诺？

Answer 1

那应该是工作。但是，reject 只接受一个参数，如果 err 是假的，您可能不想将其作为实现值的一部分传递：

if (err) reject(err); 
else resolve({body, res});

Answer 2

看起来应该可以。然而，一个问题是 originalMethod 回调必须是 args 数组的一部分才能被 Function#apply 传入。类似于：

var callback = function(err, body, res) {
                   if (err) { reject(err); }
                   else { resolve({body, res}); }
               }
...
originalMethod.apply(self, args.concat(callback));

承诺 battlenet-api 库

Promisifying battlenet-api library

javascript

node.js

promise

bluebird