Считается ли плохой практикой запрашивать данные с сервера в конструкторе службы в Angular?

Question

У меня есть угловая служба, которая отвечает за представление коллекции сотрудников (которая обычно хранится в базе данных). Я слышал, что это плохой запах кода для выполнения запроса $http.get внутри конструктора класса службы. Мне сказали, что я должен вводить зависимость, которая позволяет вам собирать соответствующие данные, но не то, что я делаю, используя $http, используя инъекцию зависимостей?

Так это плохая форма, просто сделать var employees = $http.get('employees') с сервера?

Большое спасибо!

Answer 1

Услуга - это ваша модель, все вызовы на ваш сервер должны быть централизованы в сервисах.

Если я начать с вашего примера, вы должны иметь «EmployeeService» И в нем функцию так: (не забывайте вводить $^и $ HTTP услуг)

me = this; 
me.cacheEmployee = null; 
me.getEmployees = function(){ 
    var deferred = $q.defer(); 
    if(me.cacheEmployee != null){ 
     deferred.resolve(me.cacheEmployee); 
    }else{ 
     $http.get('urlToEmployee').success(function(result){ 
      me.cacheEmployee = result; 
      deferred.resolve(me.cacheEmployee); 
     }); 
    } 

    return deferred.promise; 
}; 
me.forceLoadEmployee = function(){ // after each CreateUpdateDelete operations 
    var deferred = $q.defer(); 
    $http.get('urlToEmployee').success(function(result){ 
     me.cacheEmployee = result; 
     deferred.resolve(me.cacheEmployee); 
    }); 
    return deferred.promise; 
}; 

И в контроллер страницы, которую вы хотите отобразить ваши сотрудники (не забудьте придать EmployeeService)

EmployeeService.getEmployees().then(function(result){ 
    $scope.employees = result; 
}); 

используется асинхронный вызов, вы не можете думать в процедурный способ решить вашу проблему.

Основной документ об обещании: https://docs.angularjs.org/api/ng/service/ $ д

Я надеюсь, что мой ответ будет соответствовать вам.

Имейте славный день!

Редактировать: теперь у вас есть список сотрудников в кеше, вызов будет запущен на сервер только в первый раз.