Question

我正在使用自定义间隔函数来确保函数执行先完成，直到下次运行。一般情况下，我会执行以下操作：检查redis数据库并检查作业是否由sidekiq执行。如果它被执行了，我向数据库发出请求，如果它被写入它就获取信息，如果在第10次没有数据后我运行这个函数10次，我解决了undefined。我想知道我现在的解决方案能否以某种方式得到改善。

 const interval = (func, wait, times) => {
    const interv = function(w, t){
      return () => {
        if (typeof t === 'undefined' || t-- > 0) {
          setTimeout(interv, w);
          try {
            func.call(null);
          }
          catch (e) {
            t = 0;
            throw e.toString();
          }
        }
      };
    }(wait, times);
    setTimeout(interv, wait);
  };

    let intervalCount = 0;
    interval(() => {
      intervalCount++;
      redisClient.lrange('queue:default', 0, -1, (err, results) => {
        const job = results.find((element) => { return JSON.parse(element).jid === jobId; });
        if (job === undefined) {
          checkDatabase(personId).then((result) => {
            if (result) {
              resolve(checkDatabase(personId));
            } else if (intervalCount >= 10) {
              resolve(undefined);
            }
          });
        }
      });
    }, 1500, 10);

Answer 1

嗯，你使用的是现代NodeJS并且你提到你使用了promises和bluebird，所以让我们使用一个使用生成器和蓝鸟的现代解决方案。

你根本不应该经常使用promise构造函数。您可以宣传redis API，让您的生活更轻松：

Promise.promisifyAll(require("redis")); // now redis is promisified


var pollAndResolve = Promise.coroutine(function* pollAndResolve () {
    for(var i = 0; i < 10; i++) {
       yield Promise.delay(1500); // wait 1500 ms
       yield redisClient.lrangeAsync('queue:default', 0, -1); // since we promisifed
       const job = results.find((element) => JSON.parse(element).jid === jobId);
       if (result) return yield checkDatabase(personId);
    }
});

间隔执行后的javascript回调

1 个答案: