• In the Client MessagesThread class, in the while loop, it should be while (scanner.hasNextLine()) { • In the same class you are appending to a JTextArea from a background thread, something that you should not do. • In the Client MessagesThread class, implement Runnable rather than extend Thread (general recommendation) • Even better -- use a SwingWorker, so you can use publish/process to update the JTextArea on the Swing event thread. • For my money, I'd get the program up and running in a non-GUI version first before trying to put it into a GUI.

• Your spelling errors suggest that the code shouldn't even compile, what with clientss, and clents.??? Is this your real code? If we can't copy/paste your code and test it, we will have a tough time helping you. Edit I've looked at and run your new code. Do you see that you're trying to create a new Client in a catch block, one that is never called? Most importantly -- use a debugger when running your code to see what your code is doing or is not doing. Add println statements as well (more of them).

Multi-Client-Server-chat-application - Socket Programming - Multi-Client Server chat application using java swing & Android.

If you did this, you'd see that Client's constructor is never called, and would know to look at the code where you try to call it to see why. Rocket From The Crypt Live From Camp X-ray Rar here.