1

Устал писать код

$scope.getTour = function () {
    $http({
        url: "/tour/tour/" + $routeParams.id,
        method: "POST"
    }).success(function (data) {
        $scope.formData = data;
    }).error(function (data, status, headers, config) {
        $scope.status = status;
    });
};

$scope.onChangePublish = function (newValue, oldValue, tour_id) {
    $scope.published = newValue ? 1 : 0;
    $http({
        url: "/tour/update-tour",
        method: "POST",
        data: {
            id: tour_id,
            published: $scope.published
        }
    }).success(function (data) {
        $scope.formData = data;
    }).error(function (data, status, headers, config) {
        $scope.status = status;
    });
};

прочитал, что это плохой тон, надо как то вынести запрос к серверу в отдельный сервис(factory) и в дальнейшем использовать одну функцию. Подскажите пример

4
  • где прочитали?в чем проблема с выносом?
    – Grundy
    14 мая 2016 в 16:39
  • прочитал на тостере, типа никогда не использовать сервисы http в контроллерах, а выносить все в отдельные сервисы(factory).
    – ruslik
    14 мая 2016 в 16:52
  • проблема в том, что не понимаю как это сделать
    – ruslik
    14 мая 2016 в 16:52
  • справка по сервисам
    – Grundy
    14 мая 2016 в 16:56

1 ответ 1

1

Да, это плохой тон.
Работа с данными должна происходить в сервисах.
Вот, например, сервис для добавления подписки на email:

(function () {
    'use strict';
    angular.module('common')
    .factory('newsletterService', newsletterService);

    function newsletterService($http, constants) {
        return {
            addNewsletter: addNewsletter
        };

        function addNewsletter(email) {
            var data = angular.toJson({
                email: email
            });

            return $http.post(constants.restUrl + '/newsletter/register', data);
        }
    }
})();

Желательно ссылку на rest указать единожды в константах, чтобы использовать потом в каждом сервисе.

(function () {
    'use strict';

    angular
        .module('common')
        .value('constants', {
            restUrl: 'http://rest.domain.ru
        });
});

Метод addNewsletter сервиса newsletterService возвращает $http.post, который соответственно возвращает promise. На этот promise нужно подписаться в контроллере.

(function () {
    'use strict';

    angular
        .module('common')
        .controller('newsletterCtrl', newsletterCtrl);

    function newsletterCtrl($scope,  newsletterService) {
        $scope.addNewsletter = addNewsletter;
        function addNewsletter() {
             newsletterService.addNewsletter($scope.email).then(function(){
                  alert('Подписка прошла успешна');
             });
        }
    }
});
2
  • нашел более изящный вариант - resource
    – ruslik
    8 июн 2016 в 14:25
  • @ruslik resourse не всегда удобен
    – Namig
    9 июн 2016 в 17:51

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими условиями использования и подтверждаете, что прочитали и поняли наши политику конфиденциальности и нормы поведения.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.