$ qで約束関数を返す配列の連鎖

Question

私は関数の配列を持っています。各関数はajax呼び出しからpromiseを返します。

var promises = []; 
if (form.$valid) { 
    Object.keys($scope.Model.Data.FormFiles).forEach(function (key) { 
    var file = $scope.Model.Data.FormFiles[key]; 

    function uploadFile(){ 
     var deferred = $q.defer(); 
     var upload = Upload.upload({ 
      url: "/api/ir/funnelApi/UploadFile", 
      data: { file: file } 
     }); 

     upload.then(function (response) {       
       // do something 
       deferred.resolve(response.statusText); 
      }, function (error) { 
       deferred.reject(error.data); 
      }, function (evt) { 

     });    
     return deferred.promise; 
    }  

    promises.push(uploadFile); 
    }); 
} 

私がやっていることは、すべてのファイルが正常にアップロードされていれば何かをすることです。

$q.all(promises).then(function (responses) {      
    // do something 
}, function (errors) { 
    // if any of the file upload fails, it should come here 
}); 

しかし、問題は、Ajaxでを解雇しないと$q.allは常に関数配列を成功に移動しありません。

私は何をしていますか？

Answer 1

あなたが変更してみてください機能を呼び出すと返された約束

を押していない....配列に機能の参照を推進している。

promises.push(uploadFile()); 

---

へ

promises.push(uploadFile); 

を

新しい約束を作成する$q.defer();is also an antipatternUpload.upload()がすでに約束を返すと、代わりにそれを返すことができます