Почему PrintStream.close () в итоге вызывается дважды? - PullRequest
Новая
       15

Почему PrintStream.close () в итоге вызывается дважды?

7 голосов
/ 12 мая 2009

К моему удивлению, следующий код дважды выводит «Close». При запуске через отладчик, кажется, MyPrintStream.close() вызывает super.close(), что в итоге вызывает MyPrintStream.close() снова. 

    
import java.io.*;

public class PrintTest
{
    static class MyPrintStream extends PrintStream
    {
        MyPrintStream(OutputStream os)
        {
            super(os);
        }

        @Override
        public void close()
        {
            System.out.println("Close");
            super.close();
        }
    }

    public static void main(String[] args) throws IOException
    {
        PrintStream ps = new MyPrintStream(new FileOutputStream(File.createTempFile("temp", "file")));
        ps.println("Hello");
        ps.close();
    }
}

Почему это происходит? Разве я не должен расширять PrintStream?

Ответы [ 2 ]

11 голосов
/ 12 мая 2009

Если вы посмотрите на свой код в отладчике и установите точку останова в методе close(), он обнаружит следы стека , который вызывает ваш close() метод:

  1. ваш основной метод
  2. sun.nio.cs.StreamEncoder $ CharsetSE.implClose () line 431

полная трассировка стека для последнего выглядит так: 

PrintTest$MyPrintStream.close() line: 20    
sun.nio.cs.StreamEncoder$CharsetSE.implClose() line: 431 [local variables unavailable]  
sun.nio.cs.StreamEncoder$CharsetSE(sun.nio.cs.StreamEncoder).close() line: 160 [local variables unavailable]    
java.io.OutputStreamWriter.close() line: 222 [local variables unavailable]  
java.io.BufferedWriter.close() line: 250 [local variables unavailable]  
PrintTest$MyPrintStream(java.io.PrintStream).close() line: 307  
PrintTest$MyPrintStream.close() line: 20    
PrintTest.main(java.lang.String[]) line: 27

К сожалению, я не могу сказать , почему StreamEncoder перезвонил бы в ваш PrintStream, так как моя IDE не имеет вложения источника для sun.nio.cs.StreamEncoder :( Это JDK 6 Кстати, если это имеет значение.

Кстати, , если вы задаете этот вопрос, потому что вы заметили, что пользовательский код в вашем методе close() выполняется дважды, вы действительно должны проверить, если this.closing. PrintStream.close() устанавливает значение true (и состояние комментариев класса /* To avoid recursive closing */).

1 голос
/ 13 мая 2009

Посмотрите на источник PrintStream.

Он имеет две ссылки на базовый Writer textOut и charOut, одну символьную базу и одну текстовую (что бы это ни значило). Кроме того, он наследует третью ссылку на байтовый OutputStream, который называется out. 

/**
 * Track both the text- and character-output streams, so that their buffers
 * can be flushed without flushing the entire stream.
 */
private BufferedWriter textOut;
private OutputStreamWriter charOut;

В методе close() он закрывает их все (textOut в основном совпадает с charOut). 

 private boolean closing = false; /* To avoid recursive closing */

/**
 * Close the stream.  This is done by flushing the stream and then closing
 * the underlying output stream.
 *
 * @see        java.io.OutputStream#close()
 */
public void close() {
synchronized (this) {
    if (! closing) {
    closing = true;
    try {
        textOut.close();
        out.close();
    }
    catch (IOException x) {
        trouble = true;
    }
    textOut = null;
    charOut = null;
    out = null;
    }
}
}

Теперь интересная часть состоит в том, что charOut содержит (обернутый) ссылку на сам PrintStream (обратите внимание на init(new OutputStreamWriter(this)) в конструкторе) 

private void init(OutputStreamWriter osw) {
   this.charOut = osw;
   this.textOut = new BufferedWriter(osw);
}

/**
 * Create a new print stream.
 *
 * @param  out        The output stream to which values and objects will be
 *                    printed
 * @param  autoFlush  A boolean; if true, the output buffer will be flushed
 *                    whenever a byte array is written, one of the
 *                    <code>println</code> methods is invoked, or a newline
 *                    character or byte (<code>'\n'</code>) is written
 *
 * @see java.io.PrintWriter#PrintWriter(java.io.OutputStream, boolean)
 */
public PrintStream(OutputStream out, boolean autoFlush) {
this(autoFlush, out);
init(new OutputStreamWriter(this));
}

Итак, вызов close() вызовет charOut.close(), что, в свою очередь, снова вызывает исходный close(), поэтому у нас есть закрывающий флаг, чтобы прервать бесконечную рекурсию.

...