2015-02-23 2 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 }) 
} 
}) 

Как вы можете видеть, я повторяю то же .then() и .catch() функцию после того, как каждый из моих $ запросов HTTP. Хо, я могу избежать этой репликации в соответствии со стандартом DRY?

+0

Я не вижу, что делают эти обратные вызовы ... – elclanrs

+1

Почему бы просто не поймать ошибку в контроллере (или где бы вы не пользовались службой 'userAPI')? – aarosil

+1

Кроме того, что более важно, почему вы не используете __ [$ resource] (https://docs.angularjs.org/api/ngResource/service/$resource) __ для этого ??? – aarosil

ответ

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) сократить код еще больше, чем при использовании затем/поймать.

+0

Это самое лучшее и самое лучшее, что вы можете сделать для такой темы async ...! –

+2

Чтобы сохранить ошибку в пути сбоя, 'error.data' следует отбрасывать, а не возвращать. –

+1

Или, может быть, это 'return $ q.reject (error.data);' с $ q. * confusing * –

Смежные вопросы