Почему ввод из нескольких сокетов не обновляет пользовательский интерфейс правильно? Код

Question

Notebook: http://pastebin.com/hNgjr4QY, код сервера: http://pastebin.com/J5hnz2mu

Я в основном interesested в блокнот части кода:

private class ClientHandler implements Runnable 
{ 
    public void run() 
    { 
     String message = null; 

     try { 
      bufferedReader = new BufferedReader(new InputStreamReader(socket.getInputStream())); 

      while ((message = bufferedReader.readLine()) != null) 
      { 
       if (!message.equals("")) 
       { 
        textArea.setText(message); 
        textArea.setCaretPosition(textArea.getDocument().getLength()); 
       } 
      } 
     } catch (IOException ex) { 
      ex.printStackTrace(); 
     } 
    } 

и

public void run() 
    { 
     String message = null; 

     try { 
      BufferedReader bufferedReader = new BufferedReader(new InputStreamReader(socket.getInputStream())); 

      while ((message = bufferedReader.readLine()) != null) 
      { 
       for (ComputerConnections ccc: comCon) 
       { 
        ccc.printWriter.println(message); 
        ccc.printWriter.flush(); 
       } 
      } 
      bufferedReader.close(); 
     } catch (IOException ex) { 
      ex.printStackTrace(); 
     } 
    } 

хорошо, пункт программы, чтобы позволить так как многие пользователи подключаются, и после того, как они все могут писать в ОДНОМ блокноте, и программа должна обновить его соответствующим образом, ну и кажется, что он ведет себя странно. Сначала он прекрасно обновляется, но затем он просто внезапно останавливается и работает каждые несколько ключевых нажатий!

Answer 1

Я подозреваю, что это проблема пользовательского интерфейса, а не проблема сокета, которую вы описываете (хотя есть и другие проблемы с этим кодом). Это было бы легко понять, просто поместите некоторые отладочные строки System.out.println() вместо записи пользовательского интерфейса.

Если это так, то вы должны использовать нить EDT так вносить изменения в графический интерфейс, так что единственный поток, который вносит изменения в ГПИ является то, что один и только что один, чтобы избежать помех, как и

SwingUtilities.invokeLater( 
     new Runnable() { 
      public void run() { 
       textArea.setText(message); 
      } 
     } 
    ); 

Пользовательским интерфейсам значительно сложно сделать многопоточность, поэтому наилучшим способом этого является очередь обновлений, которые обрабатываются одним потоком пользовательского интерфейса (в случае Javas, EDT - Event Dispatch Thread).

Answer 2

Я бы использовал SwingWorker для такого вида обработки.

Позвольте всем сокетам publish данных, а затем обновите GUI в методе process.