使用AngularJS解决服务/工厂与控制器中的承诺

时间:2015-07-20 16:22:29

标签: javascript angularjs angular-promise angular-services

所以我已经玩过承诺在服务和控制器中解决的承诺。我更喜欢在服务中解决它,所以我可以重用变量而不必多次解决它。

我遇到的问题是它有效,但它的返回速度非常慢。所以我觉得我在这里做错了什么。我的ng-options填充大约需要5或6秒。哪个更好?我如何改进我的代码,使其运行得更快?

已解决服务:

resortModule.factory('locaService',['$http', '$rootScope', function ($http, $rootScope){
    locaService.getLocations=
        function() {
            return $http.get('/api/destinations').then(
                function(result){
                    locaService.locations= result.data;
                    return locaService.locations;
                }
            );
        return locaService.locations;
    };
resortModule.controller('queryController',['$scope', 'locaService', function($scope, locaService) {
    $scope.getLocations= locaService.getLocations().then(function(result){
       $scope.locations= result;
    });
}]);

已在Controller中解决:

resortModule.factory('locaService',['$http', '$rootScope', function ($http, $rootScope){
locaService.getLocations=
    function() {
        locaService.locations= $http.get('/api/destinations');
        //stores variable for later use
        return locaService.locations;
    };
}]);
resortModule.controller('queryController',['$scope', 'locaService',          
    function($scope, locaService) {
       locaService.getLocations()
       .then(
            function(locations) // $http returned a successful result
            {$scope.locations = locations;} //set locations to returned data
       ,function(err){console.log(err)});
}]);

HTML:

<select ng-click="selectCheck(); hideStyle={display:'none'}" name="destination" ng-style="validStyle" ng-change="getResorts(userLocation); redirect(userLocation)" class="g-input" id="location" ng-model="userLocation">
    <option value=''>Select Location</option> 
    <option value='/destinations'>All</option>
    <option value="{{loca.id}}" ng-repeat="loca in locations | orderBy: 'name'">{{loca.name}}</option>
</select>

1 个答案:

答案 0 :(得分:4)

在角度方面,服务是单身,因此您的应用中只有一个实例。这允许您解析数据一次(在您的服务中),存储它,然后在后续调用中返回已经解析的数据。这样您就无法多次解析数据,也可以在服务和控制器之间分离逻辑。

更新 - 缓存承诺,感谢yvesmancera查找错误

resortModule.factory('locaService', ['$http', '$rootScope', function ($http, $rootScope) {
    var locationsPromise = null;

    locaService.getLocations =
        function() {
            if (locationsPromise == null) {
                locationsPromise = $http.get('/api/destinations').then(
                  function(result) {
                      return result.data;
                  }
                );
            }

            return locationsPromise;
        };

    ...
}

resortModule.controller('queryController',['$scope', 'locaService', function($scope, locaService) {
    $scope.getLocations= locaService.getLocations().then(function(result) {
        $scope.locations= result;
    });
}]);

就加快数据加载速度而言,我发现您的javascript没有任何问题。它可能只是你的api电话占用了大量的时间。如果您发布与HTML相关的代码,我们可以检查出来,看看是否有任何因素可能会减慢它。