2015-02-23 130 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