Question

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?

Was it helpful?

Solution

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

OTHER TIPS

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).

Licensed under: CC-BY-SA with attribution
Not affiliated with StackOverflow
scroll top