$ q.all - 解決が早すぎますか？

Question

私はいくつかのサーバーの呼び出しを行う機能を持っている：

function doCalls(options) { 
    var deferred = $q.defer; 
    myService.doCallOne(options).then(function(response){ 
     myService.doCallTwo().then(function() { 
      deferred.resolve(); 
     }); 
    }); 
    return deferred.promise; 
} 

は、私は別のオプションの配列を持っていると私は$q.allに渡す約束の配列を作成したいので、私はこれを実行します。

var promiseArray = []; 
_.each(optionArray, function(options) { 
    promiseArray.push(doCalls(options)); 
}); 

を

は、その後、私はそれらを解決するのを待つしよう：

$q.all(promiseArray).then(function() { 
    doNextPart(); 
}); 

問題があり、doNextPart()は、任意の前に呼び出さなっているようです私の約束が解決する。私はここで間違って何かをしていますか？

Answer 1

をお忘れの場合は、$q.deferと発声し、実際には延期を作成しませんでした。 doCallsファンクションコールはundefined、deferred.resolve()コールは例外をスローされました（約束により飲み込まれました）、そして$q.allがすぐにundefinedの配列で返されました。

あなたは

var deferred = $q.defer(); 
//      ^^ 

にコードを変更することができますが、本当にあなたはdeferred antipatternを避ける必要があります！ただ、

function doCalls(options) { 
    return myService.doCallOne(options).then(function(response){ 
     return myService.doCallTwo(); 
    }).then(function(secondResponse) { 
     return undefined; // you might want to omit this 
    }); 
}