Чи потрібно мені закрити () і FileReader, і BufferedReader?

Я читаю локальний файл за допомогою BufferedReader, обгорнутий навколо FileReader:

BufferedReader reader = new BufferedReader(new FileReader(fileName));
// read the file
// (error handling snipped)
reader.close();

Чи потрібно мені , а також, чи буде обгортка ручки , що? Я бачив код, де люди роблять щось подібне:close()FileReader

FileReader fReader = new FileReader(fileName);
BufferedReader bReader = new BufferedReader(fReader);
// read the file
// (error handling snipped)
bReader.close();
fReader.close();

Цей метод викликається сервлетом, і я хотів би переконатися, що я не залишаю жодної ручки відкритою.

немає.

BufferedReader.close()

закриває потік відповідно до javadoc для BufferedReader та InputStreamReader

так само, як

FileReader.close()

робить.

Як вказували інші, потрібно лише закрити зовнішню обгортку.

BufferedReader reader = new BufferedReader(new FileReader(fileName));

Є дуже малий шанс, що це може витікати ручку файлу, якщо BufferedReaderконструктор кине виняток (наприклад OutOfMemoryError). Якщо ваша програма перебуває в такому стані, то, наскільки обережна потреба в очищенні, може залежати від того, наскільки важливим є те, що ви не позбавляєте ОС ресурсів, які вона може захотіти виділити іншим програмам.

Інтерфейс, що закривається, може використовуватися, якщо конструктор обгортки, ймовірно, не працює в Java 5 або 6:

Reader reader = new FileReader(fileName);
Closeable resource = reader;
try {
  BufferedReader buffered = new BufferedReader(reader);
  resource = buffered;
  // TODO: input
} finally {
  resource.close();
}

Код Java 7 повинен використовувати шаблон пробного використання ресурсів :

try (Reader reader = new FileReader(fileName);
    BufferedReader buffered = new BufferedReader(reader)) {
  // TODO: input
}

За даними джерела BufferedReader, у цьому випадку bReader.close викликає fReader.close технічно, що вам не доведеться дзвонити останньому.

Вихідний код BufferedReader показує, що базовий елемент закритий, коли ви закриваєте BufferedReader.

Перевіривши вихідний код, я виявив, що для прикладу:

FileReader fReader = new FileReader(fileName);
BufferedReader bReader = new BufferedReader(fReader);

метод close () для об'єкта BufferedReader буде викликати абстрактний метод close () класу Reader, який в кінцевому підсумку викликає реалізований метод у класі InputStreamReader , який закриває об'єкт InputStream .

Отже, достатньо лише bReader.close ().

Починаючи з Java 7, ви можете скористатися заявою про використання ресурсів

try (BufferedReader br = new BufferedReader(new FileReader(path))) {
    return br.readLine();
}

Оскільки BufferedReaderекземпляр оголошений у операторі спробу використання ресурсу, він буде закритий незалежно від того, виконати випробування нормально або різко. Тож вам не потрібно закривати це самостійно у finallyвиписці. (Це також стосується вкладених заяв про ресурси)

Це рекомендований спосіб роботи з ресурсами, детальну інформацію див. У документації

Вам потрібно лише закрити bufferedReader, тобто reader.close (), і він буде добре працювати.

Я спізнююсь, але:

BufferReader.java:

public BufferedReader(Reader in) {
  this(in, defaultCharBufferSize);
}

(...)

public void close() throws IOException {
    synchronized (lock) {
        if (in == null)
            return;
        try {
            in.close();
        } finally {
            in = null;
            cb = null;
        }
    }
}

Вам не потрібно закривати загорнутий читач / письменник.

Якщо ви подивилися на документи ( Reader.close(), Writer.close()), ви побачите, що в Reader.close()ньому написано:

Закриває потік і вивільняє будь-які системні ресурси, пов'язані з ним.

Що просто говорить, що "вивільняє будь-які системні ресурси, пов'язані з ним". Незважаючи на те, що це не підтверджує .. це дає вам поштовх, щоб почати дивитися глибше. і якщо ви переходите до Writer.close()нього, то тільки заявляєте, що він закривається сам.

У таких випадках ми звертаємось до OpenJDK, щоб переглянути вихідний код.

На BufferedWriter Line 265 ви побачите out.close(). Так що це не закриває себе .. Це щось інше. Якщо ви шукаєте в класі виникнення " out", ви помітите, що в конструкторі в рядку 87, що outпише, клас загортається там, де він викликає інший конструктор, а потім призначає outпараметр своєму власномуout змінної.

Отже .. А як щодо інших? Ви можете побачити подібний код на BufferedReader Line 514 , BufferedInputStream Line 468 та InputStreamReader Line 199 . Інших я не знаю, але цього має бути достатньо, щоб припустити, що вони так роблять.