如何简化深层嵌套的 Promise

Question

我遇到了一种情况，我需要在 then() 中的每个“else”子句处中断，这看起来并不比嵌套回调更好，这是一个登录过程：

User.findOne({
    username: username
}).exec()
    .then(user => {
        if (user) {
            return user.verifyPassAsync()
                .then(matched => {
                    if (matched) {
                        return User.getBriefProfile(username))
                            .then(emp => {
                                if (emp) {
                                    return saveToSession(emp);
                                } else {
                                    //return 
                                }})
                    } else {
                        //return ...
                    }})
        } else {
        // return false
        }
    })

有什么方法可以简化这个吗？

Answer 1

并非如此，因为您不是（仅）在这里嵌套承诺，而是条件。如果使用 async/await (ES7) 编写，您的函数将如下所示

var user = await User.findOne({username}).exec();
if (user) {
    var matched = await user.verifyPassAsync();
    if (matched) {
        var emp = await User.getBriefProfile(username);
        if (emp) {
            return saveToSession(emp);
        } else {
            // return …;
        }
    } else {
        // return …;
    }
} else {
    // return …;
}

如您所见，嵌套是您的程序所固有的。虽然它已经有点简化了（then 调用没有嵌套），您现在可以使用 Promise.coroutine 和 ES6 生成器来完成此操作。

您最好的选择可能是在每个else 中给throw 一个错误，然后用它来线性化链，最后.catching 它。当然，如果你在每个 else 块中都做同样的事情，那么你也可以写

var user = await User.findOne({username}).exec();
if (user)
    var matched = await user.verifyPassAsync();
if (matched)
    var emp = await User.getBriefProfile(username);
if (emp) {
    return saveToSession(emp);
else
    // return …;

这很容易转换回then 回调：

User.findOne({username: username}).exec()
.then(user =>
    user && user.verifyPassAsync()
).then(matched =>
    matched && User.getBriefProfile(username);
).then(emp =>
    emp ? saveToSession(emp) : …;
);