遞延鏈接，Node.js和受Q，所有鏈部分一個單一的ErrorHandler

Question

我有這樣的設計：

var postCommand = function(req, res){  
    parseForm(req) 
    .then(checkData) 
    .then(findCommand) 
    .then(initializeCommand) 
    .then(saveToDb) 
    .then(onUploadSucceeded.bind(this, res)) 
    .fail(onUploadFailed.bind(this, res)); 
} 

中的那麼的有可能拒絕該方法。我希望在發生拒絕事件之後不再執行任何操作，但最後一次.fail。

我「checkData」看起來是這樣的（是啊，它的唯一的同步鏈中的一部分）：

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

即使deferred.reject被調用時，我在「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

正如他的評論勸由Keith這是一個語法問題，解決方法：

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