任务执行完成或出错后是否需手动释放？TCP客户端任务处理疑问

你的代码示例 #

TcpClient client = tcpListener.AcceptTcpClient();
Task.Factory.StartNew(() => Read_it(client, tcpPort)).ContinueWith(x =>
    {
       if (x.Exception != null)
       {
           x.Dispose();                            
       }
    }, TaskContinuationOptions.OnlyOnFaulted | TaskContinuationOptions.OnlyOnRanToCompletion)
.ContinueWith(x =>
    {
        x.Dispose();                        
    });

疑问解答 #

1. 手动释放Task是否有必要？

完全没必要，原因如下：

• Task实现IDisposable仅仅是为了清理阻塞等待操作（比如Task.Wait()、Task.Result）时内部创建的等待句柄。如果你的代码里没有对这个Task执行任何阻塞等待操作，它内部不会持有需要手动释放的非托管资源。
• TcpClient和NetworkStream的资源清理属于业务逻辑范畴，只要你在Read_it方法里已经正确释放了这些资源，它们的生命周期和Task本身没有关联，不需要为了它们去手动释放Task。

2. 当前代码的合理性与GC相关问题

• 你当前的代码逻辑既冗余又不合理：
  • 第一个ContinueWith指定了OnlyOnFaulted | OnlyOnRanToCompletion，意味着无论原任务成功还是失败都会执行，但你只在任务出错时才调用Dispose()，逻辑矛盾；
  • 嵌套的第二个ContinueWith去释放第一个延续任务，属于完全多余的操作。
• 已完成的Task会被GC自动回收：只要没有其他代码持有对这个Task的引用，GC会在正常的回收周期里清理它，不需要手动干预。
• 绝对不要定期手动调用GC。手动触发GC会破坏GC的自动优化机制，导致应用性能出现不必要的波动，你的TCP客户端场景完全不需要这种操作。

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