Angularjs $ q.all

Zaimplementowałem $ q.all w angularjs, ale nie mogę sprawić, by kod działał. Oto mój kod:

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);
    }

A oto mój kontroler, który wywołuje usługi:

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

})

Myślę, że wystąpił problem z konfiguracją $ q.all w mojej usłudze.

W javascript nie ma block-level scopestylkofunction-level scopes :

Przeczytaj ten artykuł o javaScript Scoping and Hoisting .

Zobacz, jak debugowałem Twój kod:

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);
})

• Kiedy piszesz var deferred= $q.defer();wewnątrz pętli for, jest ona podnoszona na górę funkcji, oznacza to, że javascript deklaruje tę zmienną w zakresie funkcji poza for loop.
• Z każdą pętlą ostatnia odroczona przesłania poprzednią, nie ma zasięgu na poziomie bloku, w którym można by zapisać odwołanie do tego obiektu.
• Gdy wywoływane są asynchroniczne wywołania zwrotne (powodzenie / błąd), odwołują się one tylko do ostatniego odroczonego obiektu i tylko on zostanie rozwiązany, więc $ q.all nigdy nie jest rozwiązywane, ponieważ nadal czeka na inne odroczone obiekty.
• To, czego potrzebujesz, to utworzenie anonimowej funkcji dla każdego iterowanego elementu.
• Ponieważ funkcje mają zakresy, odniesienia do obiektów odroczonych są zachowywane w pliku closure scope nawet po wykonaniu funkcji.
• Jak skomentował #dfsq: Nie ma potrzeby ręcznego tworzenia nowego odroczonego obiektu, ponieważ sam $ http zwraca obietnicę.

  ---

Rozwiązanie z angular.forEach:

Oto demo plunker: 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);
}

Moim ulubionym sposobem jest użycie Array#map :

Oto demo plunker: 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 to również obietnica, możesz to uprościć:

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

Wydaje się, że problem polega na tym, że dodajesz deffered.promisekiedydeffered polega na sam jest obietnicą, którą powinieneś dodać:

Spróbuj zmienić na, promises.push(deffered);aby nie dodawać rozpakowanej obietnicy do tablicy.

 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);
        }