Angularjs $ q.all

Я реалізував $ q.all у angularjs, але не можу змусити код працювати. Ось мій код:

UploadService.uploadQuestion = function(questions){

        var promises = [];

        for(var i = 0 ; i < questions.length ; i++){

            var deffered  = $q.defer();
            var question  = questions[i]; 

            $http({

                url   : 'upload/question',
                method: 'POST',
                data  : question
            }).
            success(function(data){
                deffered.resolve(data);
            }).
            error(function(error){
                deffered.reject();
            });

            promises.push(deffered.promise);
        }

        return $q.all(promises);
    }

І ось мій контролер, який телефонує до служб:

uploadService.uploadQuestion(questions).then(function(datas){

   //the datas can not be retrieved although the server has responded    
}, 
function(errors){ 
   //errors can not be retrieved also

})

Я думаю, є певна проблема з налаштуванням $ q.all у моїй службі.

У JavaScript не є block-level scopesтільки function-level scopes:

Прочитайте цю статтю про розміщення та піднімання JavaScripta .

Подивіться, як я налагодив ваш код:

var deferred = $q.defer();
deferred.count = i;

console.log(deferred.count); // 0,1,2,3,4,5 --< all deferred objects

// some code

.success(function(data){
   console.log(deferred.count); // 5,5,5,5,5,5 --< only the last deferred object
   deferred.resolve(data);
})

• Коли ви пишете var deferred= $q.defer();всередині циклу for для циклу, він піднімається до верхньої частини функції, це означає, що javascript оголошує цю змінну в області функції поза межами for loop.
• З кожним циклом останнє відкладене переосмислює попереднє, не існує області блокового рівня для збереження посилання на цей об'єкт.
• Коли асинхронні зворотні виклики (успіх / помилка) викликаються, вони посилаються лише на останній відкладений об'єкт, і тільки він стає вирішеним, тому $ q.all ніколи не вирішується, оскільки він все ще чекає інших відкладених об'єктів.
• Вам потрібно створити анонімну функцію для кожного елемента, який ви повторите.
• Оскільки функції мають області застосування, посилання на відкладені об'єкти зберігаються closure scopeнавіть після виконання функцій.
• Як прокоментував #dfsq: Не потрібно вручну створювати новий відкладений об'єкт, оскільки $ http сам повертає обіцянку.

  ---

Рішення з angular.forEach:

Ось демонстраційний викрадник: http://plnkr.co/edit/NGMp4ycmaCqVOmgohN53?p=preview

UploadService.uploadQuestion = function(questions){

    var promises = [];

    angular.forEach(questions , function(question) {

        var promise = $http({
            url   : 'upload/question',
            method: 'POST',
            data  : question
        });

        promises.push(promise);

    });

    return $q.all(promises);
}

Мій улюблений спосіб - це використовувати Array#map:

Ось демонстраційний планк: http://plnkr.co/edit/KYeTWUyxJR4mlU77svw9?p=preview

UploadService.uploadQuestion = function(questions){

    var promises = questions.map(function(question) {

        return $http({
            url   : 'upload/question',
            method: 'POST',
            data  : question
        });

    });

    return $q.all(promises);
}

$ http - це теж обіцянка, ви можете зробити це простіше:

return $q.all(tasks.map(function(d){
        return $http.post('upload/tasks',d).then(someProcessCallback, onErrorCallback);
    }));

Проблема, здається, полягає в тому, що ви додаєте deffered.promiseте deffered, що саме тоді ви обіцяєте додавати:

Спробуйте змінити, щоб promises.push(deffered);ви не додавали розгорнуту обіцянку до масиву.

 UploadService.uploadQuestion = function(questions){

            var promises = [];

            for(var i = 0 ; i < questions.length ; i++){

                var deffered  = $q.defer();
                var question  = questions[i]; 

                $http({

                    url   : 'upload/question',
                    method: 'POST',
                    data  : question
                }).
                success(function(data){
                    deffered.resolve(data);
                }).
                error(function(error){
                    deffered.reject();
                });

                promises.push(deffered);
            }

            return $q.all(promises);
        }