错误：服务 angularjs 中发生 [$injector:undef]

Question

使用服务和 http 时出现错误 Error: [$injector:undef]。在更改针对此特定错误建议的所有更改后，我找不到为什么会出现这种情况。请检查以下代码

mainApp.controller('deviceDataController',["$scope","mainService", function ($scope,mainService) {
                console.log(mainService);
                mainService.deviceDataVar().success(function (data) {
                   // $scope.deviceDetails = mainService.devices;
                   console.log(data);
                });
            }]);
            
                
            mainApp.factory("mainService",["$http", function ($http) {

                angular.forEach(deviceIds, function(value, key) {
                    var timezone = user_response.timezone;
                    var DataUrl = LAX_API + "device_info.php?deviceid=" + value + "&limit=288&timezone=" + timezone;

                    return {
                        deviceDataVar: function () {
                           return $http.get(DataUrl).success(function (response) {
                                devices = response.data;
                                return devices;
                            }).error(function (data, status, headers, config) {
                                // log error
                                console.log(data)
                            });;
                        }
                    }

                });
            }]);

请帮我解决我的问题

谢谢！！

Answer 1

您的工厂声明无效。

1. 你的工厂应该只返回一个方法
2. 创建一个返回 $http 承诺的方法
3. 聚合 $q 中的所有 Promise，等待所有 Promise 返回响应
4. 汇总所有响应
5. 在 promise 中返回它们 - 你不能返回值，因为 AJAX 调用是异步的。

你的工厂应该是这样的：

      mainApp.factory("mainService", function ($http, $q) {
                /* generate single link for deviceID */
                function getDevice(value){
                   var timezone = user_response.timezone;
                   var dataUrl= LAX_API + "device_info.php?deviceid=" + value + "&limit=288&timezone=" + timezone; 
                   return $http.get(dataUrl);
                 }

                function deviceDataVar(){
                   // map method will transform list of deviceIds into a list of $http promises
                   var allRequests = deviceIds.map(getDevice);
                   // this promise will wait for all calls to end, then return a list of their results
                   $q.all(allRequests).then(function(arrayOfResults) { 
                   // here you will have all responses, you just need to agregate them into single collection.
                   // secondly you will need to wrap them into a promise and return
                   return ...
                 });

               }

                  /* Return only public methods, at the end */
                return {
                    deviceDataVar: deviceDataVar
                }
        });

（我是在匆忙中写的，所以可能会有一些错误:)，但这个概念是正确的）

有用的链接：

1. 聚合承诺：angular -- accessing data of multiple http calls - how to resolve the promises
2. Array.prototype.map：https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/map
3. 如何向工厂承诺：How can I pass back a promise from a service in AngularJS?

更新：

要通过调用小步骤使其工作，您应该：

1. 模拟工厂方法，让它返回一个硬编码的值。通过承诺将其归还。
2. 将硬编码值替换为选定设备的单个（硬编码）URL。
3. 对单个 $http 调用使用聚合 ($q.all)。
4. 将单个 $http 替换为由 deviceIds 列表组成的数组。