Question

我正在处理ReST API流程的脚本。现在我需要一个函数，它会在几秒钟内尽可能多地重试请求API。

所以我写了一些Promise抽象，我做了类似下面的事情：

$(function () {
  var $el = $('#test');
  function api (message) {
    return $.ajax('/echo/json/', {
      method: 'POST',
      data: { json: JSON.stringify({ message: message }), delay: 2000 },
      timeout: 1000
    });
  }
  // Process to keep retrying an API request as many as possible in 3 seconds
  retried(_.bind(api, undefined, 'Hello, world.'), 3000)
  .then(function (dat) {
    $el.text(dat.message);
  }, function (err) {
    if (err instanceof Error) $el.css('color', 'red');
    $el.text(err.message);
  });
});

我为此做的一些功能如下：

// Promise wrapper
var promisify = function (func) {
  var funcPartial = function () {
    var funcArgs = _.toArray(arguments);
    var dfr = new $.Deferred(), promiseArgs = [ dfr.resolve, dfr.reject ];
    var timeoutId = setTimeout(function () {
      clearTimeout(timeoutId);
      func.apply(undefined, _.union(promiseArgs, funcArgs));
    }, 1);
    return dfr.promise();
  };
  return funcPartial;
};

// Promise abstraction for recursive call
retried = promisify(function (resolve, reject, done, duration, start) {
  if (!_.isNumber(start)) start = +(new Date());
  return done()
  .then(resolve, function (err) {
    var stop = +(new Date());
    if (duration <= stop - start) {
      reject(err);
    } else {
      return retried(done, duration, start);
    }
  });
});

该过程以正确的状态完成，但retried函数不会返回Promise链。

我真的很累。有人能指出我错误的观点来纠正上面的实现吗？

这是整个演示脚本。

DEMO

谢谢。

解决

感谢下面的@BenjaminGruenbaum，我刚刚注意到我根本不需要promisify来制作retried功能。这是完全羞耻的问题，但再次感谢所有回答这个问题的人。

这是修订后的retried函数，根本不需要promisify ...

var retried = function (done, duration, start) {
  if (!_.isNumber(start)) start = +(new Date());
  return done()
  .then(function (dat) {
    return dat;
  }, function (err) {
    var stop = +(new Date());
    if (duration > stop - start) return retried(done, duration, start);
    return err;
  });
};

我更新了演示，现在XD

工作正常

DEMO（已修订）

Answer 1

您的重试逻辑非常复杂。你是在宣传＆＃34;具有解决和拒绝功能并执行explicit construction的功能。您将如何在同步代码中实现重试？

do {
   var failed = false;
   try { 
       var val = fn();
   } catch(e){
       failed = true;
   }
} while(failed);

但我们无法做到这一点，因为我们无法使用循环，但我们可以使用递归代替：

function retry(fn){
    try {
       return fn(); // call the function
    } catch(e){
       return retry(fn); // retry it again
    }
}

现在，添加承诺只意味着失败不是通过尝试/捕获而是通过承诺解决/缺乏：

function retry(fn) {
    return fn().then(null, function(e){ 
        return retry(fn); // on failure handler, retry again.
    });
}

此实现的好处是不依赖于使用的promise库（它将使用相同类型的fn返回）。

作为旁注，请不要忽略错误。甚至网络错误 - 至少将它们记录在某处：）

使用jQuery Deferred这个递归Promise函数有什么问题

1 个答案: