延迟链接，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));  
}

then 中的方法可能会被拒绝。我希望在发生拒绝后不再执行任何操作，但最后一个 .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() 位于promise 链的开头，那么它需要像own answer 那样编写，返回一个明确的promise。

在中链中它也可以这样编写，但是中链为您提供了更好的选择来编写什么。

更具体地说，您可以通过以下方式避免创建和返回承诺：

• 抛出而不是返回被拒绝的承诺，
• 返回数据而不是承诺包装的数据。

因此，提供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;
}