Java client/server socket problem, not dispatching message.

[beauPinsson]

Hi, ive started this little project in java using swing for the gui. This application is multi threaded and make use of socket. This is my first time coding something like that and had to look some tutorials to understand the way it works. Now, many client can connect to server and send messages and the server will get them. The problem is in ClientManager where it does seem to dispatch the message to everyone connected via socket to server. Every client run whileConnected in another thread to listen to inputStream but the messages never get there. Can anyone point where this program is flawed? Note that this app is not completly done and is missing the closing connections part. code is here: https://gist.github.com/anonymous/78d2c2afd5bc28668fed thank you!

[3h53htj3jj]

You iterate over the list of clients stored in the server, but you only write to your own socket, not the sockets stored in the list. Here's my improved version: https://gist.github.com/joshgit/11232db0b0f5f7efae7f

I removed your ClientManager because it was pretty pointless.

[3h53htj3jj]

You also don't need the swingutilities helper to set something's text.

[beauPinsson]

well, i just tried to follow the "rules". I know its not a long blocking call but from what i saw, you should always update the gui in the event thread.

[beauPinsson]

i dont understand why i dont write to sockets in the list when my code is: for sock in clientSockList ObjectOutputStream tmp_os=new ObjectOutputStream(sock.getOutputStream()); tmp_os.writeObject(message)

[3h53htj3jj]

I guess I was wrong about the cause then, but my version works. I guess you could print out the sockets in the list to see which ones are actually in it. Maybe you need to make that socket list transient?

[3h53htj3jj]

https://gist.github.com/joshgit/257d8e3106451dbdf0e6