无法设置未定义的属性'XY'

Question

我有以下代码:

var favourites = JSON.parse(localStorage.getItem("favourites"));
Service.all().then(function (multiple) {
        var array = [];            
        for (var i = 0; i < multiple.length; i++) {
           for (var j = 0; j < favourites.length; j++) {
              if (favourites[j].checked === true) {
                 if (multiple[i].Name === favourites[j].name) {
                    Service.getAllBySomething(multiple[i].Id).then(function (resources) {
                       var arrayOfSomething = [];
                       for (var k = 0; k < resources.length; k++) {
                                                     arrayOfSomething.push(resources[k].ResourceCategoryId);
                       }                           
                       arrayOfSomething = arrayOfSomething .filter(function (elem, pos, arr) {
                          return arr.indexOf(elem) == pos;
                       });
                       multiple[i].existingProperty= arrayOfSomething;
                    });                        
                    array.push(multiple[i]);
                 }
              }                  
           }
        }            
        $scope.vendors = array;
     });

我的问题是，它说每次'不能设置属性existingProperty of undefined'。我不知道为什么multiple[I]在这行没有定义:

multiple[i].existingProperty= arrayOfSomething;

属性存在，我确定。它是被定义的，它是一个空数组。我想把这个空数组替换成循环中的made数组。错在哪里?我如何用我的数组填充existingProperty ?

Answer 1

Service.getAllBySomething是异步的吗?因为在这种情况下，回调函数运行时，i(仅在闭包中捕获)已被移动到数组的末尾。

使用一个额外的闭包在调度异步调用时捕获i的值，如下所示:

Service.getAllBySomething(multiple[i].Id).then(function(i){
     return function (resources) {
                   var arrayOfSomething = [];
                   for (var k = 0; k < resources.length; k++) {
                                                 arrayOfSomething.push(resources[k].ResourceCategoryId);
                   }                           
                   arrayOfSomething = arrayOfSomething .filter(function (elem, pos, arr) {
                      return arr.indexOf(elem) == pos;
                   });
                   multiple[i].existingProperty= arrayOfSomething;
                };
}() );    

注意，新函数接收i作为参数并返回您以前使用的函数。我立即调用(IIFE)来返回要作为回调传递的函数。在该函数内部，i的值将与调度此异步调用时的值相同，因为它在用作参数时被复制。