use .Net UdpClient in a multithreaded environment

2020-03-26 07:48发布

I have an instance of a class (lets call it A) which serves some threads, this instance only sends UDP packets via the UdpClient class. It initialize the the UdpClient in its constructor and only serves to send the packets.

It looks something like:

public class A{

private UdpClient m_Client;
public class A(string host, int port){

    m_Client = new UdpClient(host, port);
}

public void Send(string dataToSend){

 var data= encoding.GetBytes(dataToSend);
 client.BeginSend(data, data.Length, null, null);
}

}

My questions is:

I know that UdpClient isn't thread-safe (according to MSDN documentation), what is the best way to support multithreaded without using locking mechanism?

  1. On each send create new instance of the UdpClient? (just use some local UdpClient var). performance?

  2. Use ThreadLocal for the UdpClient? but what about the disposing of the UdpClient in this situation?

  3. Any other solution?

2条回答
Luminary・发光体
2楼-- · 2020-03-26 08:25

I think the best way here would be to create a new UdpClient every time the method is called. This way, you can be sure the code is safe. The performance isn't likely to be a problem, and if profiling shows that it is, only then you should start solving that.

Also, you shouldn't forget to call EndSend() after each BeginSend() (ideally in the callback).

查看更多
兄弟一词,经得起流年.
3楼-- · 2020-03-26 08:32

Eventually I believe that my current implementation should work with no issues (as long as Microsoft won't change the UdpClient class implementation).

For whom it might be interesting: http://social.msdn.microsoft.com/Forums/en-US/csharpgeneral/thread/cbdd9818-00f0-499f-a935-d8e555899d64

查看更多
登录 后发表回答