Зачем возвращать обещание и данные при использовании $http в службе Angular?

Данные, возвращенные в .then, доступны следующему связанному обработчику .then, который вы в конечном итоге будете использовать для получения данных.

.success просто передает исходное обещание $http.get. Возврат данных из .success ничего не делает.

Итак, если у вас есть:

 function getStuff() {
    return $http.get('/api/stuff')
        .success(function(data) {
            // do something with data. returning doesn't do anything
        })
        .error(function(data) {
            console.error(data);
        });
};

в контроллере вы бы сделали:

getStuff().then(function(response){
  $scope.data = response.data; // this is the data available from `$http.get`
}

Учебник, на который вы ссылаетесь, не возвращает данные в обработчике успеха, он возвращает result.data в обработчике then. Затем возвращается то, что генерирует новое связанное обещание. Это удаляет данные HTTP-запроса, которые обычно присутствуют в .then.

Зачем беспокоиться?

Причина работать с .then вместо .success — проверка на будущее. .success не является частью обещаний, он специфичен для $http. Если вы когда-нибудь решите позже получить данные из другого асинхронного источника (веб-сокеты, веб-воркеры), ваш код сломается, когда возвращаемое вами обещание больше не имеет обработчика успеха.

Заставив его всегда возвращать .then, вы убедитесь, что ваши сервисы достаточно абстрагированы.

Обычно вы делаете это, если вам нужно выполнить некоторую обработку перед отправкой данных в контроллер.

Но в данном случае согласен так как нет обработки данных, нет необходимости в разрешении промиса в сервисе.

В этом случае я бы просто переписал сервисный метод следующим образом:

function getStuff() {
   return $http.get('/api/stuff');
}

Если мне нужно выполнить некоторую обработку перед возвратом данных в контроллер, я обычно использую $q, как показано ниже.

function getStuff() {
    var defer = $q.defer();
    $http.get('/api/stuff')
        .then(function(data) {
            // some processing
            defer.resolve(processedData);
        }, function(error) {
            defer.reject(error);   
        });
        return defer.promise;
}