服务控制器中的成功和错误函数

Question

我在服务中有以下代码，我正在从控制器调用 fetchData 函数。

服务

app.service("geturl", function($http) {
    urllist = [];
    geturl.fetchData = function() {
        var data = [];
        for (i = 0; i < urllist.length; i++) {
          (function(index) {
            return $http.get(geturl.urllist[index], {
              timeout: 8000
            })
            .then(function(response) {
              data[index] = response.data;
            });
          }(i);
          return data;
        });
    };
});

$http.get的成功和错误函数我想写在控制器中，因为UI中需要它，我该怎么做呢？

Answer 1

..我想把$http.get的成功和错误函数写在 控制器..

通常，.then() 函数有两个函数参数。第一个参数是成功处理程序，第二个参数是错误处理程序。

$http.get(url,options).then(function (response){
    //success handler function
    },function(error){
  //error handler function     
  })

或者，您可以分别指定.success 和.error 函数。

$http.get(url,options).success(function (response){
    //success handler function
    }).error(function(error){
  //error handler function     
  })

更新： 从您的代码来看，您似乎打算从您的服务geturl 返回一些东西并在那里提供回调。这不是应该如何做的。你应该从你的服务中返回一个承诺。

   ... 
   app.service("geturl",function($http){
     ...
     getData:function(){
        return $http.get(url,options);
     }
     ...               
   });
   ...

并在您使用服务的模块中处理成功/错误回调

geturl.getData().success(function(){
//handle success
}).error(function(){
//handle error
})

如果您需要发出多个 http 请求，永远不要使用 for 循环。请记住，一切都是异步的，并且您不能保证在创建新请求之前从前一个请求中获得响应。在这种情况下，您应该使用$q service。有关更多详细信息，请参阅@pankajparkar 的答案

Answer 2

好像你想在所有 ajax 完成后返回一个数据，你可以通过使用 $q.all() 来实现这一点

服务

app.service("geturl", function($http, $q) {
    this.urllist = [];
    this.fetchData = function() {
        var data = [], promises = [];
        for (i = 0; i < urllist.length; i++) {
          (function(index) {
            var promise = $http.get(geturl.urllist[index], {
              timeout: 8000
            })
            .then(function(response) {
              data[index] = response.data;
            });
            promises.push(promise); //creating promise array
          }(i);
        };
        return $q.all(promises).then(function(resp){ 
            return data; //returning data after all promises completed
        });
    };
});