Thread: Need help finding Fast Way to send a message to multiple sockets in a list object?

I think you should start to pinpoint where your delay is coming from. If you say you have a visible delay, then find out where it is. Print to console and observe where it's lagging. I'm really at a loss how to help you if you didn't even try the simplest debugging technique for three straight days.

Originally Posted by shivam1992

It waits for a socket to be accepted without the .accepted() call blocking it, i lowered the time for it to literally 1 and i was getting the delay still, i use the poll as a non blocking way so accept doesn't block the entire program

If you specify a nonzero value for the first parameter of poll, it isn't nonblocking anymore - it blocks for a length of time up to the time you specified in Poll().

Lastly, there's not much reason to loop over a socket list and calling Poll() - use Socket.Select() instead, which is much more efficient at polling a set of sockets.

Originally Posted by shivam1992

DO you think i should recieve all messages then send them all out at once? or is that less efficient?

Much less efficient unless you expect a huge, huge volume of traffic. Think about it like this - if the main loop is working properly, the time from a message coming in until the message is dispatched to all other clients should be on the order of milliseconds. How likely is it for two clients to both send a message within milliseconds of each other versus how likely is it that they won't?

I think your problem is likely to be the blocking code you have. Of the snippets you've posted, you've demonstrated a 100 second long block in your main loop, plus a n-second long block in receiving (where n is the number of connections). There could well be blocks in checkForDisconnections() as well.

If you're going to do this in the current design (check for disconnections, messages, and accept connections in a single loop), every poll() everywhere in the code should have a timeout of 0 so it's nonblocking. Any block anywhere in the code has the potential to stop your entire main loop until that block times out.

There are some other designs you could consider too:

1. To split the three main functions into separate threads, and block indefinitely on each thread until it has something to do. You'd need to handle thread synchronization on your list of sockets, so there's some overhead in terms of locking, but from a multitasking operating system point of view, blocking until an event is better than polling.

2. To use event-driven programming via use of the BeginAccept(), BeginReceive(), etc. methods, and use this to notify you when you need to do something. It still needs to be done in a thread-safe manner, but the framework would handle most of the details, except for thread safety on shared data such as the socket list.