web-dev-qa-db-ja.com

promiseが拒否された場合のAngularJSサービスの再試行

次のように、コントローラー内の非同期サービスからデータを取得しています。

myApp.controller('myController', ['$scope', 'AsyncService',
function($scope, AsyncService) {
    $scope.getData = function(query) {
        return AsyncService.query(query).then(function(response) {
            // Got success response, return promise
            return response;
        }, function(reason) {
            // Got error, query again in one second
            // ???
        });
    }
}]);

私の質問：

  1. Promiseを返さずにサービスからエラーが発生したときに、サービスを再度クエリする方法。
  2. 私のサービスでこれを行う方が良いでしょうか？

ありがとう！

javascriptangularjsangularjs-service
15
Angelin

コントローラではなく、サービス自体でリクエストを再試行できます。

そう、 AsyncService.queryは次のようになります：

AsyncService.query = function() {
  var counter = 0
  var queryResults = $q.defer()

  function doQuery() {
    $http({method: 'GET', url: 'https://example.com'})
      .success(function(body) {
        queryResults.resolve(body)
      })
      .error(function() {
        if (counter < 3) {
          doQuery()
          counter++ 
        }
      })
  }

  return queryResults.promise
}

そして、コントローラーのエラー関数を取り除くことができます。

myApp.controller('myController', ['$scope', 'AsyncService',
  function($scope, AsyncService) {
    $scope.getData = function(query) {
      return AsyncService.query(query).then(function(response) {
        // Got success response
        return response;
      });
    }
  }
]);
21
M.K. Safi

これは実際に機能します：

angular.module('retry_request', ['ng'])
  .factory('RetryRequest', ['$http', '$q', function($http, $q) {
    return function(path) {
      var MAX_REQUESTS = 3,
          counter = 1,
          results = $q.defer();

      var request = function() {
        $http({method: 'GET', url: path})
          .success(function(response) {
            results.resolve(response)
          })
          .error(function() {
            if (counter < MAX_REQUESTS) {
              request();
              counter++;
            } else {
              results.reject("Could not load after multiple tries");
            }
          });
      };

      request();

      return results.promise;
    }
  }]);

次に、それを使用する単なる例：

RetryRequest('/api/token').then(function(token) {
  // ... do something
});

モジュールを宣言するときにそれを要求する必要があります：

angular.module('App', ['retry_request']);

そしてあなたのコントローラーでは：

app.controller('Controller', function($scope, RetryRequest) {
  ...
});

誰かがリクエストを再試行するために何らかのバックオフまたはランダムなタイミングでそれを改善したい場合、それはさらに良いでしょう。いつかそのようなものがAngular Coreにあることを願っています

9
Dorian

再帰を使用しない指数バックオフの実装を作成しました（ネストされたスタックフレームが作成されますよね？）実装方法には複数のタイマーを使用するコストがかかり、make_single_xhr_callのすべてのスタックフレームが常に作成されます（成功した後でも） 、失敗後だけではなく）。それが価値があるかどうかはわかりませんが（特に平均的なケースが成功した場合）、それは思考の糧です。

呼び出し間の競合状態が心配でしたが、javascriptがシングルスレッドで、コンテキストスイッチがない場合（1つの$ http.successが別の$ http.successによって中断され、2回実行される可能性があります）、ここで問題ありません。正しい？

また、私はangularjsと最新のjavascriptに非常に慣れていないので、規則も少し汚いかもしれません。どう考えているか教えてください。

var app = angular.module("angular", []);

app.controller("Controller", ["$scope", "$http", "$timeout",
    function($scope, $http, $timeout) {

  /**
   * Tries to make XmlHttpRequest call a few times with exponential backoff.
   * 
   * The way this works is by setting a timeout for all the possible calls
   * to make_single_xhr_call instantly (because $http is asynchronous) and
   * make_single_xhr_call checks the global state ($scope.xhr_completed) to
   * make sure another request was not already successful.
   *
   * With sleeptime = 0, inc = 1000, the calls will be performed around:
   * t = 0
   * t = 1000 (+1 second)
   * t = 3000 (+2 seconds)
   * t = 7000 (+4 seconds)
   * t = 15000 (+8 seconds)
   */
  $scope.repeatedly_xhr_call_until_success = function() {
    var url = "/url/to/data";
    $scope.xhr_completed = false
    var sleeptime = 0;
    var inc = 1000;
    for (var i = 0, n = 5 ; i < n ; ++i) {
      $timeout(function() {$scope.make_single_xhr_call(url);}, sleeptime);
      sleeptime += inc;
      inc = (inc << 1); // multiply inc by 2
    }
  };

  /**
   * Try to make a single XmlHttpRequest and do something with the data.
   */
  $scope.make_single_xhr_call = function(url) {
    console.log("Making XHR Request to " + url);

    // avoid making the call if it has already been successful
    if ($scope.xhr_completed) return;
    $http.get(url)
      .success(function(data, status, headers) {
        // this would be later (after the server responded)-- maybe another
        // one of the calls has already completed.
        if ($scope.xhr_completed) return;
        $scope.xhr_completed = true;
        console.log("XHR was successful");
        // do something with XHR data
      })
      .error(function(data, status, headers) {
        console.log("XHR failed.");
      });
  };

}]);
2
mathewguest

この記事に続いて AngularJSでの約束、漫画として説明

応答が5XXカテゴリに該当する場合にのみ再試行する必要があります

私はhttpと呼ばれるサービスを作成しました。これは、すべてのhttp構成を次のように渡すことで呼び出すことができます。

 var params = {
  method: 'GET',
  url: URL,
  data: data
 }

次に、次のようにサービスメソッドを呼び出します。

   <yourDefinedAngularFactory>.http(params, function(err, response) {});

http: function(config, callback) {
  function request() {
    var counter = 0;
    var queryResults = $q.defer();

    function doQuery(config) {
      $http(config).success(function(response) {
        queryResults.resolve(response);
      }).error(function(response) {
        if (response && response.status >= 500 && counter < 3) {
          counter++;
          console.log('retrying .....' + counter);
          setTimeout(function() {
            doQuery(config);
          }, 3000 * counter);
        } else {
          queryResults.reject(response);
        }
      });
    }
    doQuery(config);
    return queryResults.promise;
  }
  request(config).then(function(response) {
    if (response) {
      callback(response.errors, response.data);
    } else {
      callback({}, {});
    }
  }, function(response) {
    if (response) {
      callback(response.errors, response.data);
    } else {
      callback({}, {});
    }
  });
}
2
Ratan Kumar

私はこれをたくさんやることになったので、この問題に対処するのに役立つライブラリを書きました:)

https://www.npmjs.com/package/reattempt-promise-function

この例では、次のようなことができます

myApp.controller('myController', ['$scope', 'AsyncService',
function($scope, AsyncService) {
    var dogsQuery = { family: canine };
    $scope.online = true;
    $scope.getDogs = function() {
        return reattempt(AsyncService.query(dogsQuery)).then(function(dogs) {
            $scope.online = true;
            $scope.dogs = dogs;
        }).catch(function() {
            $scope.online = false;
        });
    }
}]);
1
beckyconning