TypeScript中FetchFromPortal返回Promise<void | T>而非预期对象的问题

嘿，我一眼就看到问题出在哪了——你的FetchFromPortal方法里的catch分支没有正确处理错误的传递，导致TypeScript推断出的返回类型变成了Promise<T | void>。

具体原因 #

当Promise链触发catch时，你只执行了console.log(error)，这个语句的返回值是undefined，所以此时整个Promise会被resolved为void（而不是保持rejected状态）。而正常流程下Promise会返回T类型，所以TypeScript就会把方法的返回类型推断为Promise<T | void>，这就是为什么你调用后拿到的result类型不符合预期。

修复方案 #

有两种常用的修复方式，你可以根据业务需求选择：

方案1：重新抛出错误（推荐）

如果错误需要向上层调用者传递，那就在catch块里重新抛出错误，这样Promise会保持rejected状态，不会被解析为void：

public FetchFromPortal<T>(xyz: string) {
  // ------ Other less relevant code -----------
  return fetch(url, options)
    .then((response) => this.CheckStatus(response))
    .then((response) => this.ParseJSON<T>(response))
    .then((response) => {
      console.log(response);
      return response;
    })
    .catch((error: Error) => {
      console.log(error);
      throw error; // 关键：重新抛出错误，维持Promise的rejected状态
    });
}

方案2：返回T类型的默认值

如果业务允许在出错时返回一个默认的T类型值，那可以在catch里返回符合类型的默认值，这样整个Promise的类型就会统一为Promise<T>：

public FetchFromPortal<T>(xyz: string) {
  // ------ Other less relevant code -----------
  return fetch(url, options)
    .then((response) => this.CheckStatus(response))
    .then((response) => this.ParseJSON<T>(response))
    .then((response) => {
      console.log(response);
      return response;
    })
    .catch((error: Error) => {
      console.log(error);
      return {} as T; // 根据你的业务场景替换成合适的默认值
    });
}

额外优化：显式指定返回类型 #

为了让TypeScript更早帮你发现这类问题，你可以给方法显式指定返回类型Promise<T>，这样如果你的代码不符合这个类型，TypeScript会直接报错提示：

public FetchFromPortal<T>(xyz: string): Promise<T> {
  // 方法体同上
}

至于你之前尝试在调用时加then处理没用的原因——因为原方法返回的Promise本身就存在void的可能，外层的then只能处理resolved的值，但无法改变原Promise的类型推断结果，必须从根源上修复catch分支的处理逻辑才行。

内容的提问来源于stack exchange，提问作者Dandy