总是返回承诺有用吗

Question

我正在使用 bluebird 围绕 http 服务设计一些 nodejs api 包装器。 这个包装器中的许多函数都是异步的，因此从这些实现中返回 Promise 很有意义。

我的同事已经在这个项目上工作了几天，有趣的模式正在出现，他还从同步实现的函数中返回承诺。

例子：

function parseArray(someArray){
    var result;
    // synchronous implementation
    return Promise.resolve(result);           
}

如果以后需要使实现异步，我可以看到这将如何有用，因为您不必重构调用站点。 我想所有方法都始终“异步”也很好，但我不确定这到底有多棒。

这是否被认为是一种不好的做法，我们有什么理由不应该这样做吗？

Answer 1

在同步方法中返回一个承诺是没有意义的。

Promises 提供了对并发性的抽象。当不涉及并发时，例如提供数组时。返回一个 Promise 会导致更糟糕的流控制，并且速度要慢得多。

这也传达了错误的信息。事实上，无缘无故地承诺事情是一种很常见的反模式。

有用的一种情况是方法可能 是异步的 - 例如：从缓存中获取某些内容或在缓存中不存在时发出请求：

function getData(id){
     if(cache.has(id) return Promise.cast(cache.get(id));
     return AsyncService.fetch(id).tap(cache.put);
}

Answer 2

如果我们可以忽略性能，那么只有在使用示例实现时才不好：

function parseArray(someArray) {
    var result;
    // synchronous implementation
    return Promise.resolve(result);           
}

这个函数很乱，因为它可以同步抛出但也返回一个promise。返回承诺的函数绝不能抛出 - 否则您将失去承诺的大部分好处，因为现在您必须同时使用 try-catch 和 .catch()。

正确的实现方式是用.method“注解”一个函数：

var parseArray = Promise.method(function() { 
    var result;
    //Promise.resolve(result) is unnecessary now
    return result;
});

现在保证函数永远不会同步抛出，并且可以以一致的方式使用。