Правильный способ закрыть вложенные потоки и записи в Java [дубликат]

при закрытии цепных потоков, вам нужно только закрыть самый внешний поток. Любые ошибки будут распространяться по цепочке и быть пойманы.

смотрите Потоки Ввода-Вывода Java для сведения.

чтобы решить эту проблему

однако, если flush () по какой-то причине выбрасывает исключение времени выполнения, то out.close () никогда не будет вызван.

Это неправильно. После того, как вы поймаете и проигнорируете это исключение, выполнение будет забрать обратно после блока catch и out.close() оператор будет выполнен.

ваш коллега делает хороший момент о Runtimeисключения. Если вам абсолютно необходимо, чтобы поток был закрыт, вы всегда можете попытаться закрыть каждый из них индивидуально, снаружи, останавливаясь на первом исключении.

в эпоху Java 7,try-with-resources это, конечно, путь. Как упоминалось в нескольких предыдущих ответах, запрос закрытия распространяется от самого внешнего потока к самому внутреннему потоку. Таким образом, единственное закрытие-это все, что требуется.

try (ObjectInputStream ois = new ObjectInputStream(new FileInputStream(f))) {
  // do something with ois
}

однако есть проблема с этим шаблоном. Try-with-resources не знает о внутреннем FileInputStream, поэтому если конструктор ObjectInputStream создает исключение, FileInputStream никогда не закрывается (пока сборщик мусора не доберется до него). Решение...

try (FileInputStream fis = new FileInputStream(f); ObjectInputStream ois = new ObjectInputStream(fis)) {
  // do something with ois
}

это не так элегантно, но более надежно. Так ли это на самом деле проблема будет зависеть от того, какие исключения могут быть брошены во время строительства внешний объект(ы). ObjectInputStream может бросить IOException, который вполне может быть обработан приложением без завершения. Многие потоковые классы только выбрасывают непроверенные исключения, что вполне может привести к завершению работы приложения.

рекомендуется использовать Apache Commons для обработки объектов, связанных с IO.

на finally предложение use IOUtils

IOUtils.closeQuietly(bWriter); IOUtils.closeQuietly (oWritter);

фрагмент кода ниже.

BufferedWriter bWriter = null;
OutputStreamWriter oWritter = null;

try {
  oWritter  = new OutputStreamWriter( httpConnection.getOutputStream(), "utf-8" );
  bWriter = new BufferedWriter( oWritter );
  bWriter.write( xml );
}
finally {
  IOUtils.closeQuietly(bWriter);
  IOUtils.closeQuietly(oWritter);
}

коллега поднимает интересный вопрос, и есть основания спорить в любом случае.

лично я бы игнорировал RuntimeException, потому что непроверенное исключение означает ошибку в программе. Если программа неверна, исправьте ее. Вы не можете "обрабатывать" плохую программу во время выполнения.

это удивительно неудобный вопрос. (Даже если предположить acquire; try { use; } finally { release; } код правильный.)

если конструкция декоратора терпит неудачу, то вы не будете закрывать основной поток. Поэтому вам нужно явно закрыть базовый поток, будь то в finally after use или, более diifcult после успешной передачи ресурса декоратору).

если исключение вызывает сбой выполнения, вы действительно хотите сбросить?

некоторые декораторы на самом деле имеют ресурсы сами. Текущая реализация ВС ZipInputStream например, выделена память кучи не Java.

было заявлено, что (IIRC) две трети ресурсов, используемых в библиотеке Java, реализованы явно неверным образом.

пока BufferedOutputStream закрывается даже на IOException С flush,BufferedWriter правильно закрывается.

мой совет: закройте ресурсы как можно быстрее, и не позволяйте им портить другим кодом. Ото, вы можете потратить слишком много времени на этот вопрос - если OutOfMemoryError бросается приятно вести себя красиво, но другие аспекты вашей программы, вероятно, имеют более высокий приоритет, и код библиотеки, вероятно, сломан в этой ситуации в любом случае. Но я всегда писал:

final FileOutputStream rawOut = new FileOutputStream(file);
try {
    OutputStream out = new BufferedOutputStream(rawOut);
    ... write stuff out ...
    out.flush();
} finally {
    rawOut.close();
}

(Смотри: нет улова!)

и, возможно, использовать выполнение вокруг идиомы.

Java SE 7 try-with-resources Кажется, не упоминается. Это устраняет необходимость явно делать закрытие полностью, и мне очень нравится эта идея.

к сожалению, для разработки Android этот сладкий становится доступным только с помощью Android Studio (я думаю) и таргетинг Kitkat и выше.

Также вам не нужно закрывать все вложенные потоки

проверить это http://ckarthik17.blogspot.com/2011/02/closing-nested-streams.html

Я использую, чтобы закрыть потоки, как это, без вложенности попробуйте-поймайте в конце концов блоки

public class StreamTest {

public static void main(String[] args) {

    FileOutputStream fos = null;
    BufferedOutputStream bos = null;
    ObjectOutputStream oos = null;

    try {
        fos = new FileOutputStream(new File("..."));
        bos = new BufferedOutputStream(fos);
        oos = new ObjectOutputStream(bos);
    }
    catch (Exception e) {
    }
    finally {
        Stream.close(oos,bos,fos);
    }
  }   
}

class Stream {

public static void close(AutoCloseable... array) {
    for (AutoCloseable c : array) {
        try {c.close();}
        catch (IOException e) {}
        catch (Exception e) {}
    }
  } 
}

JavaDocs Солнца включают RuntimeExceptions в их документации, как показано InputStream read (byte [], int, int) метод; документировано как бросание NullPointerException и IndexOutOfBoundsException.

FilterOutputStream в flush () это только документально, как бросать исключение IOException, таким образом, он фактически не бросайте RuntimeExceptions. любой, который может быть брошен, скорее всего, будет завернут в IIOException.

Она все еще может бросить Error, но вы не так много можете сделать с ними; Sun рекомендует вам не пытаться их поймать.