TypeScript报错TS2345：Dialog中返回Promise.all()类型不匹配求助

这个错误的核心原因很明确：你的whenClosed回调函数存在代码路径返回undefined的情况，而TypeScript严格要求回调的返回值必须匹配any[] | PromiseLike<any[]>类型，undefined显然不符合这个要求。

具体问题分析 #

当用户点击对话框的取消按钮时，response.wasCancelled为true，此时你的回调函数没有任何return语句，TypeScript会默认这个分支返回undefined。但whenClosed期望回调要么直接返回数组，要么返回一个解析为数组的Promise，undefined完全不在允许的类型范围内，所以触发了TS2345错误。

另外，原代码还有一个小细节可以优化：你在map里用了async，但回调本身没有标记为async——虽然这不会直接报错，但会让代码逻辑的可读性变差，建议统一处理。

修正后的代码 #

private async deleteSelectedLuminaires(): Promise<void[]> {
  // 把返回类型从any[]改为void[]更严谨，因为removeLuminaire返回Promise<void>
  return this.dialogService.open({ viewModel: DeleteLuminaire, model: 'Cancel or Ok', lock: false })
    .whenClosed(async (response) => {
      if (!response.wasCancelled) {
        const requests = this.selectedSerials
          .map(serial => this.luminaireApi.removeLuminaire({ serial }));
        // 返回Promise.all的结果，它会解析为void[]
        return Promise.all(requests);
      } else {
        // 取消操作时返回空数组，满足类型要求
        return [];
      }
    });
}

关键改动说明 #

1. 处理取消分支的返回值：在用户取消时返回一个空数组[]，确保所有代码路径都返回符合要求的类型（数组或Promise解析为数组）。
2. 优化返回类型：将Promise<any[]>改为Promise<void[]>，因为removeLuminaire返回Promise<void>，Promise.all会将所有void结果收集为void[]，这样类型更精确，避免使用模糊的any。
3. 标记回调为async：让回调函数明确是异步函数，虽然这里直接返回Promise.all的结果也能工作，但标记async能让代码逻辑更清晰，符合异步代码的书写习惯。

如果你坚持要保留Promise<any[]>的返回类型，只需要把返回的空数组改成[] as any[]即可，但还是推荐使用更精确的void[]。

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