承诺拒绝可能未处理的错误:

时间:2014-05-23 21:54:27

标签: javascript node.js promise catch-block hapijs

我有一个使用数组执行某些操作的函数。 当数组为空时我想拒绝它。

作为一个例子

myArrayFunction(){
        return new Promise(function (resolve, reject) {
           var a = new Array();
           //some operation with a
           if(a.length > 0){
               resolve(a);
           }else{
               reject('Not found');
           }           
        };
}

当拒绝操作发生时,我收到以下错误。 可能未处理错误:未找到

但是,当调用myArrayFunction()时,我有以下捕获。

handlers.getArray = function (request, reply) {
    myArrayFunction().then(
        function (a) {
            reply(a);
        }).catch(reply(hapi.error.notFound('No array')));
};

拒绝承诺,接受拒绝并回应客户的正确方法是什么?

谢谢。

1 个答案:

答案 0 :(得分:18)

.catch将函数作为参数,但是,您传递的是其他内容。当你没有传递一个函数来捕获时,它将默默无法做任何事情。愚蠢,但这是ES6承诺的。

由于.catch没有做任何事情,拒绝变得无法处理并报告给您。


修复是将函数传递给.catch

handlers.getArray = function (request, reply) {
    myArrayFunction().then(function (a) {
        reply(a);
    }).catch(function(e) {
        reply(hapi.error.notFound('No array')));
    });
};

因为您正在使用catch all,所以错误不一定是No array错误。我建议你这样做:

function myArrayFunction() {
    // new Promise anti-pattern here but the answer is too long already...
    return new Promise(function (resolve, reject) {
            var a = new Array();
            //some operation with a
            if (a.length > 0) {
                resolve(a);
            } else {
                reject(hapi.error.notFound('No array'));
            }
        };
    }
}

function NotFoundError(e) {
    return e.statusCode === 404;
}

handlers.getArray = function (request, reply) {
    myArrayFunction().then(function (a) {
        reply(a);
    }).catch(NotFoundError, function(e) {
        reply(e);
    });
};

可以进一步缩短为:

handlers.getArray = function (request, reply) {
    myArrayFunction().then(reply).catch(NotFoundError, reply);
};

另请注意:

之间的区别
// Calls the method catch, with the function reply as an argument
.catch(reply)

// Calls the function reply, then passes the result of calling reply
// to the method .catch, NOT what you wanted.
.catch(reply(...))