Java - странное поведение в PrintStream с пользовательским OutputStream

Question

Я пытаюсь написать программу, которая перенаправляет System.out на JTextArea (это не должно быть JTextArea), но когда я вызываю System.out.println (" ! Test ") выход в текстовую область, как так:

\n 
st! 
\n 

код для моего OutputStream:

package gui; 

import java.awt.*; 
import java.io.*; 
import javax.swing.text.*; 

public class LogOutputStream extends OutputStream 
{ 
    public void write(final int b) throws IOException 
    { 
     EventQueue.invokeLater(new Runnable() 
     { 
      public void run() 
      { 
       write0(b); 
      } 
     }); 
    } 

    public void write(final byte[] b, final int off, final int len) 
    { 
     EventQueue.invokeLater(new Runnable() 
     { 
      public void run() 
      { 
       write0(b, off, len); 
      } 
     }); 
    } 

    public void write(final byte[] b) 
    { 
     EventQueue.invokeLater(new Runnable() 
     { 
      public void run() 
      { 
       write0(b); 
      } 
     }); 
    } 

    private void write0(int b) 
    { 
     Document doc = FernflowerGUI.frame.textArea.getDocument(); 
     try 
     { 
      doc.insertString(doc.getLength(), String.valueOf((char)b), null); 
     } 
     catch(BadLocationException impossible) 
     { 

     } 
    } 

    private void write0(byte[] b, int off, int len) 
    { 
     Document doc = FernflowerGUI.frame.textArea.getDocument(); 
     try 
     { 
      doc.insertString(doc.getLength(), new String(b, off, len), null); 
     } 
     catch(BadLocationException impossible) 
     { 

     } 
    } 

    private void write0(byte[] b) 
    { 
     write0(b, 0, b.length); 
    } 
} 

код, который создает PrintStream:

PrintStream ps = new PrintStream(new LogOutputStream(), true); 

Может ли кто-нибудь сказать мне, что происходит на Земле?

Answer 1

Ваш код не является потокобезопасным, в основном.

Вы принимаете синхронный вызов, принимающий байтовый массив, - и затем вы используете этот массив байтов позже и считаете, что он все равно будет иметь тот же контент. Что делать, если вызывающий абонент write() перезаписывает данные в массиве байтов сразу после возврата метода? К тому времени, когда вы его используете, у вас не будет правильных данных.

Я бы извлечь String из массива байтов в вашем write вызова, а затем использовать этот String в вызове write0.

(я также лично использовать Writer, а не OutputStream. - в основном вы хотите иметь дело с текстовыми данными, а не двоичные данные)