Как правильно преобразовать контроллер и службу из AngularJS в использование ng-ресурса

Недавно я начал изучать ng-ресурс. Мне интересно, если я нахожусь на правильном пути, когда речь заходит о преобразовании моего обычного приложения AngularJS в приложение, использующее ng-ресурс.

Это мой старый контроллер home.js (без использования ng-ресурса):

angular.module("HomeApp", ["BaseApp"])
    .controller("MainCtrl", ["$http", "$window", "BaseService", function($http, $window, BaseService) {

        var self = this;

        BaseService.fetch.posts(function() {
            self.posts = BaseService.posts;
            self.cerrorMessages = BaseService.cerrorMessages;
        });

        self.like = function(id, postType) {
            BaseService.like(id, postType, function() {
                self.cerrorMessages = BaseService.cerrorMessages;
            });
        };
    }]);

И это мой старый base.js (BaseService):

angular.module("BaseApp", [])
    .factory("BaseService", ["$http", "$window", function($http, $window) {
        var self = this;
        self.posts = [];
        self.cerrorMessages = [];
        self.accessErrors = function(data) {
             self.cerrorMessages = [];
             for (prop in data) {
                 if (data.hasOwnProperty(prop)){
                     self.cerrorMessages.push(data[prop]);
                 }
             }
         };

        self.fetch = {
             posts: function(callback) {
                 $http.get('/posts/')
                 .then(function(response) {
                     self.posts = response.data;
                     callback();
                 }, function(response) {
                     self.accessErrors(response.data);
                     callback();
                 });
             }
         };

        self.like = function(id, postType, callback) {
            $http.post("/" + postType + "/" + id + "/like/")
            .then(function(response) {
                angular.forEach(self.posts, function(post, index, obj) {
                    if (post.id == id) {
                        post.usersVoted.push('voted');
                        post.voted=true;
                    };
                });
            }, function(response) {
                 self.accessErrors(response.data);
                 callback();
            });
        };

        return self;
    }]);

Мне сказали, что я могу извлечь выгоду из использования ng-ресурсов, так как у меня есть бэкэнд RESTful. Я попытался преобразовать код выше, чтобы начать использовать ng-ресурс, и это то, что у меня есть сейчас. Это мой resources.js:

angular.module('PostsResources', ['ngResource'])

.factory('PostsFactory', function($resource) {
  return $resource('/posts/:postId', { postId:'@id' });
});

.factory('PostLikeFactory', function($resource) {
  return $resource('/posts/:postId/like', { postId:'@id' });
});

И это мой контроллер (home.js):

angular.module("HomePageApp", ["BaseApp", "PostsApp"])
    .controller("MainCtrl", ["$http", "$window", "BaseService", "PostsResource", "PostLikeResource", function($http, $window, BaseService, PostsResource, PostLikeResource) {

var self = this;

function loadPosts() {
    self.posts = PostsFactory.query(function(data) {},
        function(error) {
          BaseService.accessErrors(error.data);
          self.cerrorMessages = BaseService.cerrorMessages;
        }
    );
};

self.like = function likePost() {
    PostLikeFactory.save(function(data) {
        angular.forEach(self.posts, function(post, index, obj) {
            if (post.id == id) {
                post.usersVoted.push('voted');
                post.voted=true;
            };
        });
    }, function(error) {
        BaseService.accessErrors(error.data);
        self.cerrorMessages = BaseService.cerrorMessages;
    }
)};

И это мой BaseService (base.js):

angular.module("BaseApp", [])
    .factory("BaseService", ["$http", "$window", function($http, $window) {
        var self = this;
        self.posts = [];
        self.cerrorMessages = [];
        self.accessErrors = function(data) {
             self.cerrorMessages = [];
             for (prop in data) {
                 if (data.hasOwnProperty(prop)){
                     self.cerrorMessages.push(data[prop]);
                 }
             }
         };
        return self;
    }]);

Я нахожусь на правильном пути конвертации обычного приложения AngularJS в использование ng-ресурса? Мне кажется, что количество требуемого кода выглядит одинаково. Спасибо заранее!

Ответ 1

Я согласен с @Mark в том, что у вас, вероятно, недостаточно операций в вашем API, чтобы полностью использовать $resource, поэтому преимущества не так велики. Если вы хотите использовать $resource, я бы установил его так:

angular.module('PostsResources', ['ngResource'])
   .factory('Post', function($resource) {
      return $resource('/posts/:postId', { postId:'@id' }, {
         like: {
            url: '/posts/:postId/like',
            method: 'POST',
            params: { postId:'@id' }
         }
      })
   });

Если вы доберетесь до того, что используете более одного действия на конечную точку, у меня будет ресурс Post и PostLike, но в вашем случае это не обязательно.

В вашем контроллере вы сможете работать со своими сообщениями следующим образом:

self.posts = Post.query(null, BaseService.errorHandler(self));

self.like = function(post) {
   post.$like(function() {
      angular.forEach(self.posts, function(post, index, obj) {
            if (post.id == id) {
                post.usersVoted.push('voted');
                post.voted=true;
            };
        });
   }, BaseService.errorHandler(self));
};

Метод errorHandler, который вы можете добавить в свой BaseService, чтобы избавиться от некоторого дублирования. Он будет выглядеть примерно так:

self.errorHandler = function(viewModel) {
   return function(error) {
      BaseService.accessErrors(error.data);
      viewModel.cerrorMessages = BaseService.cerrorMessages;
   };
};

Ответ 2

Как правило, я использую только ng-ресурс, когда у меня есть все веб-приложения CRUD. Пример - это приложение типа CRM, в котором вам нужно создавать клиентов, обновлять клиентов, удалять клиентов и показывать список клиентов. Имеет смысл использовать ng-resource, поскольку он "обрабатывает" HTTP-методы get, post, put и delete.

Однако в вашем примере кажется, что единственные действия, которые у вас есть, - это "Как" и "Получение списка сообщений". Если вы используете ТОЛЬКО, чтобы использовать один из методов HTTP, для каждого типа ресурса, я бы предложил просто придерживаться старого способа использования BaseService.

Только мои 2 цента по этому вопросу.