如果我不调用 UdpClient.Close() 方法有什么缺点？

Question

我有以下代码块，它影响了我的程序效率。这里的问题是，如果目标主机存在，一切正常。但如果它确实存在，执行起来需要很长时间。最后，我发现“udp.Close()”占据了大部分的执行时间。如果我不调用close方法，效率还是不错的。

如果我不调用 close 方法，谁能帮我告诉我有什么缺点？非常感谢。

{ // This is my code block, I need to execute it many many times.
string ipAddress = Dns.GetHostAddresses("joe-pc").FirstOrDefault().ToString();
UdpClient udp = new UdpClient(ipAddress, Port);
udp.Send(msg, msg.Length);
udp.Close();
udp = null;
}

Answer 1

缺点是你会有资源泄漏。你可能很幸运，垃圾收集经常发生，以至于它没有在你的程序中展示自己，但为什么要冒险呢？来自Close的文档：

关闭会禁用底层Socket并释放与UdpClient关联的所有托管和非托管资源。

注意，它涉及非托管资源。这些将仅由运行一些代码的UdpClient 发布 - 它要么在 Close/Dispose 中执行，要么必须在其 Finalize 方法中执行 - 没有其他的导致它们被释放（假设程序保持运行）。

您也许可以隐藏Close 操作的成本，方法是使用Task.Run 让它在另一个线程上运行 - 但您必须权衡这样做的成本.

---

或者，更具体的说——你说你需要这个方法运行很多次。如果不在这里清理您的资源，您将增加后续调用完全失败的机会，因为它无法获取所需的资源（它们都被绑定在现有的、非Closed UdpClient 个实例）。

---

而且，正如我的评论中所指出的，以下行毫无意义：

udp = null;

此类代码曾经在 COM 时代的 VB 中使用，但它在 .NET 世界中没有位置。

Answer 2

Close() 禁用底层 Socket 并释放与 UdpClient 关联的所有托管和非托管资源。 如果你没有关闭，那么它不会禁用和释放资源，比如你的端口和 IP 地址

Answer 3

您是否可以使用BeginSend 而不是使用Send，然后在您尝试关闭时在回调中处理异常（如果这确实是问题）？

Answer 4

随着 dns 记录的更改，您可能使用不同的 ip 地址来定位 joe-pc，但您为每次发送重复使用相同的 UdpClient。完成后记得关闭（）它。

使用

//
// Summary:
//     Sends a UDP datagram to a specified port on a specified remote host.
//
// Parameters:
//   dgram:
//     An array of type System.Byte that specifies the UDP datagram that you intend
//     to send represented as an array of bytes.
//
//   bytes:
//     The number of bytes in the datagram.
//
//   hostname:
//     The name of the remote host to which you intend to send the datagram.
//
//   port:
//     The remote port number with which you intend to communicate.
//
// Returns:
//     The number of bytes sent.
//
public int Send(byte[] dgram, int bytes, string hostname, int port);

并跳过 dns 查找。

Answer 5

我知道这是一段时间，但我今天偶然发现了这个问题并想补充：

udp=null;

恕我直言，这不是毫无意义的。在某些情况下，您可以在内存分析器中看到，如果您不将其设置为 null，您的实例仍然可用。