繰延チェーン、Node.jsのとq、すべてのチェーンの部品のための1つのErrorHandlerのは

Question

私はこのようなデザインを持っています。私は拒絶が起こった後にもう何も実行されないことを望んだが、最後の.fail。

私の「CHECKDATAは」この（いやそのチェーンにおける唯一の同期一部）のようになります。deferred.rejectが呼び出された場合でも、

var checkData = function(args){ 
    var deferred = q.defer(); 
    var fields = args[0]; 
    var files = args[1]; 
    var file = args[1].uploadedFile0; 
    var secondFile = args[1].uploadedFile1;  
    if(file.type !== "application/javascript"){ 
     return deferred.reject("please only upload javascript-files"); 
    } 
    (...) 
    return deferred.resolve([fields, file]); 
} 

、私は渡された引数なしで「findCommand」で終わります。私はこの鎖に間違ったことを理解していますか？

また、質問：このデザインは良いパターンですか？または、他の/より良いベストプラクティスがありますか？

Answer 1

checkData()が約束チェーンの先頭にあった場合、それは戻って、あなたのown answerのように書かれる必要があるだろう明白な約束。

ミッドチェーンでは、そのように書かれたときにも機能しますが、ミッドチェインでは、書いたものについてより良いオプションが得られます。投げる代わりの代わりに約束包まれたデータのデータを返す拒否約束、

を返す

• ：

  具体的には、次の方法で約束を作成して返すために必要性を回避することができます。

ので、約束のチェーンを開始するために他の場所で使用されることはありませんcheckData()を提供し、それはこのように書くことができます。

var checkData = function(args) { 
    var fields = args[0], 
     files = args[1], 
     file = args[1].uploadedFile0, 
     secondFile = args[1].uploadedFile1;  
    if(file.type !== "application/javascript") { 
     throw new Error("please only upload javascript-files"); // always throw an Error object, thus emulating a natural Error. This error will percolate down to the `.fail(onUploadFailed.bind(this, res))` handler. 
    } 
    (...) 
    return [fields, file]; // the next step in the chain will assimilate a promise or data. 
} 

Answer 2

それは文法問題だった彼のコメントにキースによってadvicedとして、解決策：

var checkData = function(args){ 
    var deferred = q.defer(); 
    var fields = args[0]; 
    var files = args[1]; 
    var file = args[1].uploadedFile0; 
    var secondFile = args[1].uploadedFile1;  
    if(file.type !== "application/javascript"){ 
     deferred.reject("please only upload javascript-files"); 
    } 
    else{ 
     deferred.resolve([fields, file]); 
    }   
    return deferred.promise; 
}