19

我正在编写一个将异步检索数据的服务（$http 或 $resource）。我可以通过返回一个最初为空但最终会被填充的数组来隐藏它是异步的事实：

.factory('NewsfeedService1', ['$http', function($http) {
   var posts = [];
   var server_queried = false;
   return {
      posts: function() {
         if(!server_queried) {
            $http.get('json1.txt').success(
              function(data) {
                server_queried = true;
                angular.copy(data, posts);
            });
         }
         return posts;
      }
   };
}])
.controller('Ctrl1', ['$scope','NewsfeedService1',
function($scope, NewsfeedService1) {
    $scope.posts = NewsfeedService1.posts();
}])

或者我可以通过返回一个承诺来暴露异步性：

.factory('NewsfeedService2', ['$http', function($http) {
  var posts = [];
  var server_queried = false;
  var promise;
  return {
     posts_async: function() {
       if(!promise || !server_queried) {
         promise = $http.get('json2.txt').then(
           function(response) {
              server_queried = true;
              posts = response.data;
              return posts;
         });
       }
       return promise;
     }
  };
}])

.controller('Ctrl2', ['$scope','NewsfeedService2',
function($scope, NewsfeedService2) {
  NewsfeedService2.posts_async().then(
    function(posts) {
      $scope.posts = posts;
  });
  // or take advantage of the fact that $q promises are
  // recognized by Angular's templating engine:
  // (note that Peter and Pawel's AngularJS book recommends against this, p. 100)
  $scope.posts2 = NewsfeedService2.posts_async();
}]);

（Plunker - 如果有人想玩弄上述两种实现。）

暴露异步性的一个潜在优势是我可以通过向方法添加错误处理程序来处理控制器中的错误then()。但是，我可能会在应用程序范围的拦截器中捕获和处理 $http 错误。

那么，什么时候应该暴露服务的异步性呢？

4

3 回答 3

9

我的猜测是你会在这道栅栏的两边找到人。就个人而言，我认为您应该始终公开库或函数的异步性（或者更准确地说：我认为您永远不应该隐藏库或函数的异步性）。主要原因是透明度；例如，这行得通吗？

app.controller('MyController', function(NewsfeedService) {
  $scope.posts = NewsfeedService.posts();
  doSomethingWithPosts($scope.posts); // <-- will this work?
});

如果您使用第一种方法（例如$resource），它不会，即使$scope.posts从技术上讲是一个数组。如果doSomethingWithPosts有自己的异步操作，您最终可能会遇到竞争条件。相反，无论如何您都必须使用异步代码：

app.controller('MyController', function(NewsfeedService) {
  $scope.posts = NewsfeedService.posts(function() {
    doSomethingWithPosts($scope.posts);
  });
});

（当然，您可以让回调接受posts作为参数，但我仍然认为它令人困惑且不标准。）

幸运的是，我们有 Promise，而 Promise 的目的就是代表操作的未来价值。此外，由于使用 Angular 的$q库创建的 Promise 可以绑定到视图，这并没有错：

app.controller('MyController', function(NewsfeedService) {
  $scope.posts = NewsfeedService.posts();
  // $scope.posts is a promise, but when it resolves
  // the AngularJS view will work as intended.
});

[更新：您不能再将 Promise 直接绑定到视图；您必须等待 promise 得到解决并手动分配范围属性。]

顺便说一句，Restangular，一个流行的替代品$resource，使用 Promise，AngularJS 自己的$resource将在 1.2 中支持它们（他们可能已经在最新的 1.1.x 中支持它们）。

于 2013-07-02T03:56:02.120 回答
5

我总是选择异步选项，因为我不喜欢隐藏底层框架的异步性质。

同步版本在使用时可能看起来更干净，但它会无意中导致开发人员没有意识到调用本质上是异步的并在调用后尝试访问数据的错误。

SO充满了问题，人们在$resource考虑它本质上是同步的，并期待得到回应时犯了这个错误。$resource也采用与选项 1 类似的方法，在调用完成后填充结果，但仍$resource公开成功和失败函数。

如果返回了 Promise，AngularJS 会尝试隐藏异步调用的复杂性，因此直接绑定到 Promise 感觉就像是在进行同步调用。

于 2013-07-02T03:59:06.020 回答
5

我说不，因为它使以这种方式构建的多个服务更难工作。使用 Promise，您可以$q.all()用来发出多个请求并在所有请求完成时做出响应，或者您可以通过传递 Promise 将操作链接在一起。

对于同步样式服务，没有直观的方法可以做到这一点。

于 2013-07-02T04:07:19.923 回答