从 Promise 中解决值的哪种方法是正确的？

Question

我对 JavaScript 中的所有这些异步内容仍然很陌生，并且在处理一些 Promise 时遇到了一些令人困惑的情况。这是我当前的代码：

exists(filename, targetDir){
    const filepath = path.join(targetDir || this.getTargetDir(), filename);
    // plug in any one method from below
}

当我查看其他人的代码时，我看到他们解析这样的值（插入上面的代码）：

// method 1
return request(this.getUrl(filepath))
    .then(res => {
        return Promise.resolve(res.statusCode === 200);
    })
    .catch(() => {
        return Promise.resolve(false);
    });

// method 2
return request(this.getUrl(filepath))
    .then(res => {
        Promise.resolve(res.statusCode === 200);
    })
    .catch(() => {
        Promise.resolve(false);
    });

// method 3
return request(this.getUrl(filepath))
    .then(res => {
        return res.statusCode === 200;
    })
    .catch(() => {
        return false;
    });

// method 4
return new Promise((resolve, reject) => {
    request(this.getUrl(filepath))
        .then(res => {
            resolve(res.statusCode === 200);
        }
        .catch(() => {
            resolve(false);
        };
});

在这种情况下哪些是正确的？哪些是不正确的？这取决于场景吗？推荐其中哪一种？一个很好的解释将不胜感激，谢谢！

澄清： exists 是一个类方法，它返回一个解析为布尔值的 Promise，其中true 表示 URL 存在，false 表示它不存在.

澄清 #2： 如果发生错误，exists 应解析为 false。

Answer 1

• 方法 1 正确但不必要的复杂。

• 方法 2 完全错误。
  您创建了新的 Promise，但它们没有填充到任何地方，因此它们在函数结束时被释放。
  相当于：

  return request(this.getUrl(filepath)).catch(err => undefined);
  

• 方法 3 是最好的方法。

• 方法 4 也可以解析为 corect 值，但它是一种反模式。
  What is the explicit promise construction antipattern and how do I avoid it?

Answer 2

方法3最好。

下一个.then() 接收当前.then() 返回的内容。如果这个返回值恰好是一个promise，那么下一个.then()会在promise解决后执行。

使用方法3，您将能够将其他.then()s 附加到当前链：

exists('test.txt', '/')
  .then(doesExist => console.log(doesExist ? 'The file exists.' : 'The file doesn\'t exist.'));

方法 1 和方法 4 也可以做到这一点，但方法 1 不必要地将值包装在将立即解析的承诺中。

方法 4 不必要地将整个请求包装在一个 Promise 中。这称为explicit-construction anti-pattern。

方法 2 不允许 Promise 链接。您不能使用包裹在 Promise.resolve 中的值，因为 .then() 和 .catch() 都隐式返回 undefined。