2015-02-23 129 views
1

我在我的角度应用一个简单的userAPI服务:

app.service('userAPI', function ($http) { 
this.create = function (user) { 
    return $http 
     .post("/api/user", { data: user }) 
     .then(function (promise) { return promise.data }) 
     .catch(function (error) { return error.data }) 
} 

this.read = function (user) { 
    return $http 
     .get("/api/user/" + user.id) 
     .then(function (promise) { return promise.data }) 
     .catch(function (error) { return error.data }) 
} 

this.update = function (user) { 
    return $http 
     .patch("/api/user/" + user.id, { data: user }) 
     .then(function (promise) { return promise.data }) 
     .catch(function (error) { return error.data }) 
} 

this.delete = function (user) { 
    return $http 
     .delete("/api/user/" + user.id) 
     .then(function (promise) { return promise.data }) 
     .catch(function (error) { return error.data }) 
} 
}) 

正如你所看到的,我每次我的$ HTTP请求后,重复相同的。那么()和.catch()函数。根据DRY原则,我可以避免这种重复吗?

+0

我不明白这些回调是做什么的...... – elclanrs 2015-02-23 05:01:20

+1

为什么不在控制器中捕获错误(或者使用'userAPI'服务的地方)? – aarosil 2015-02-23 05:06:08

+1

此外,更重要的是,为什么你不使用__ [$资源](https://docs.angularjs.org/api/ngResource/service/$resource)__为此? – aarosil 2015-02-23 06:13:29

回答

1

为什么不直接写一次函数并将它们应用于服务中的每个回调?

喜欢的东西:

app.service('userAPI', function ($http) { 
    var success = function (response) { return response.data; }, 
     error = function (error) { return error.data; }; 

    this.create = function (user) { 
     return $http 
      .post("/api/user", { data: user }) 
      .then(success, error); 
    } 
    this.read = function (user) { 
     return $http 
     .get("/api/user/" + user.id) 
     .then(success, error); 
    }; 
    this.update = function (user) { 
     return $http 
     .patch("/api/user/" + user.id, { data: user }) 
     .then(success, error); 
    }; 
    this.delete = function (user) { 
     return $http 
     .delete("/api/user/" + user.id) 
     .then(success, error); 
    }; 
}); 

另外请注意,您可以使用then(successcallback, errorcallback, notifycallback),以进一步缩短你的代码比使用再/ catch语句。

+0

这是最好的和最可以做这种异步的东西......! – 2015-02-23 05:18:02

+2

为了保持错误路径上的错误,应该抛出'error.data',而不是返回。 – 2015-02-23 12:33:58

+1

或者可能是'返回$ q.reject(error.data);'带$ q。 *令人困惑* – 2015-02-23 12:38:04