Promiseifying battlenet-api library

Question

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

function (args, callback)

callback的签名是：

function (err, body, res)

在这种特殊情况下，我需要在执行之前在promise中添加一个速率限制器（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

似乎它应该工作。然而，一个问题是originalMethod回调必须是args数组的一部分才能被Function#apply传入。类似的东西：

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

Answer 2

这应该是关于工作的。但是，reject只会引用一个参数，如果err是假的，您可能不想将其作为履行价值的一部分传递：

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