TSED框架中$onDestroy钩子重复调用问题解析与修复

问题背景

在TSED框架的使用过程中，开发者发现了一个关于服务销毁钩子$onDestroy的异常行为。该钩子被设计为在服务实例销毁时执行一次清理逻辑，但在实际应用中却出现了重复调用的情况。

问题现象

当开发者实现了一个消息处理服务(MessageHandlerService)时，该服务实现了OnDestroy接口并定义了$onDestroy方法。在应用程序关闭时，预期该方法应该只执行一次，但实际日志显示该方法被调用了两次：

destroying message handler
[INFO] disposing message handler
destroying message handler
Call hook $onDestroy

这种重复调用会导致资源清理逻辑被多次执行，可能引发各种问题，特别是对于需要精确控制的外部资源（如数据库连接、消息队列连接等）。

问题分析

经过TSED核心团队的调查，确认这是一个框架层面的问题。正常情况下，$onDestroy钩子应该确保每个单例服务只被调用一次。但在特定情况下，框架内部触发了两次销毁流程，导致钩子被重复执行。

临时解决方案

在问题修复前，开发者可以采用以下临时解决方案：

@Service()
export class MessageHandlerService implements OnDestroy {
  destroyed = false;
  
  async $onDestroy(): Promise<any> {
    if (this.destroyed) return;
    this.destroyed = true;
    // 清理逻辑
  }
}

通过添加一个标志位(destroyed)来确保清理逻辑只执行一次。虽然这种方法有效，但增加了额外的状态管理代码，不够优雅。

框架修复

TSED团队迅速响应，在版本8.7.4中修复了这个问题。修复确保：

1. $onDestroy钩子现在严格保证每个服务实例只调用一次
2. 销毁流程更加可靠和一致
3. 服务生命周期管理更加可预测

最佳实践

即使问题已经修复，在实现$onDestroy钩子时仍建议：

1. 确保清理逻辑是幂等的（可以安全地多次执行）
2. 对于关键资源，添加适当的错误处理
3. 记录重要的销毁操作以便调试

总结

TSED框架对生命周期钩子的持续改进体现了其对开发者体验的重视。$onDestroy钩子的正确行为对于构建可靠的应用程序至关重要，特别是在需要精确管理外部资源的场景下。开发者应及时更新到修复版本(8.7.4或更高)以获得稳定的销毁行为。

对于需要更精细控制销毁流程的高级场景，可以考虑结合使用框架提供的其他生命周期钩子和事件机制来构建更健壮的资源管理策略。