是否需手动调用unsubscribe()取消NgbModal的@Output() EventEmitter订阅？

首先明确结论：Angular和NgBootstrap不会自动帮你取消这类订阅，如果不主动处理，确实存在内存泄漏的风险。

分析你的两种实现 #

1. 第一种takeWhile的实现
   这种方式逻辑上是想通过opened变量控制订阅生命周期，但存在明显隐患：模态框关闭后，someOutputEvent这个EventEmitter大概率不会再触发任何事件，takeWhile的条件也就没机会被检查，订阅会一直处于活跃状态——如果父组件是长期存在的路由组件，就会造成内存泄漏。所以这种方式无法可靠防止泄漏。

2. 第二种手动unsubscribe的实现
   这是更可靠的方案：无论模态框是通过result.then()的成功回调（点击确认）还是失败回调（点击关闭/点击背景）关闭，都主动调用mySubscription.unsubscribe()，直接切断订阅，彻底避免泄漏。

更简洁的优化写法 #

可以利用RxJS的takeUntil操作符，结合模态关闭的信号简化代码，不用手动管理订阅变量：

import { from } from 'rxjs';
import { takeUntil, catchError, ignoreElements } from 'rxjs/operators';

const modalRef = this.modalService.open(SomeModalComponent);

// 将模态关闭的Promise转为Observable，覆盖所有关闭场景
const modalClosed$ = from(modalRef.result).pipe(
  catchError(() => from(Promise.resolve())), // 处理点击背景关闭等错误场景
  ignoreElements() // 只关心完成信号，无需处理模态返回结果
);

// 用takeUntil自动取消订阅
modalRef.componentInstance.someOutputEvent.pipe(
  takeUntil(modalClosed$)
).subscribe(_ => {
  // 处理事件逻辑
});

关键提醒 #

NgBootstrap的模态框关闭后，组件实例并不会立即被Angular销毁，订阅会一直绑定在实例上。只要订阅存在，组件实例就无法被垃圾回收，最终导致内存泄漏。因此必须主动取消订阅，手动unsubscribe或用RxJS操作符自动管理都是可行方案，后者代码更简洁易维护。

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