Как правильно закрывать вложенные стримы в Java?

Question

[Dik_Nik]

У меня есть такой код:

try (final FSDataInputStream fileStream = getFileInputStream() {
     Table table = csvUtils.getData(tableInfo, new BufferedReader(new InputStreamReader(fileStream)));
}

Подскажите пожалуйста, потечет ли этот код, если я InputStream закрываю, а BufferedReader нет?

Answer 1

[Сергей Горностаев]

Почему бы не сделать так?

try (BufferedReader reader = new BufferedReader(new InputStreamReader(getFileInputStream()))) {
    Table table = csvUtils.getData(tableInfo, reader);
}

Comments

[Dik_Nik]

Тут проблема в том, что код не совсем полный. Вот точнее:

try (final FSDataInputStream fileStream = getFileInputStream(tableInfo.getHdfsPath())) {
     if ("csv".equals(tableInfo.getMode())) {
          Table table = csvUtils.getData(tableInfo, new BufferedReader(new InputStreamReader(fileStream)));
           return mapper.writeValueAsString(table);
     }

     return CharStreams.toString(new InputStreamReader(fileStream));
}

[Сергей Горностаев]

Dik_Nik, точно не потечёт. При закрытии BufferedReader будут закрыты и освобождены все используемые им ресурсы, и так по цепочке до исходного стрима. В документации об этом недвусмысленно написано.

[Исрапил Ахмедов]

Dik_Nik, возможно не в тему, но лучше вместо строки использовать enum.

[Dik_Nik]

Сергей Горностаев, да, ваш вариант точно не потечет, я сначала не заметил блока try в вашем варианте.
Но я не могу открывать сразу BufferedReader.
Вопрос, что будет с ним, если я по цепочке ниже закрою, а его нет?

[Dik_Nik]

Исрапил Ахмедов, извините, немного не понял. В каком месте вы предлагаете заменить на enum?

[Сергей Горностаев]

Dik_Nik, тогда можно сделать так

try (FSDataInputStream fileStream = getFileInputStream(tableInfo.getHdfsPath());
     BufferedReader br = new BufferedReader(new InputStreamReader(fileStream))) {
    if ("csv".equals(tableInfo.getMode())) {
          Table table = csvUtils.getData(tableInfo, br);
...

[Dik_Nik]

Сергей Горностаев, я знаю)
Но вопроса это не снимает с:

[Сергей Горностаев]

Dik_Nik, ничего сильно страшного не произойдёт, если BufferedReader не закрыть, так как у него маленькая область видимости и он станет доступен сборщику мусора сразу после выхода из блока try. Но явное освобождение ресурсов - правила хорошего тона.

[Dik_Nik]

Сергей Горностаев, спасибо!) Так и думал)

[Исрапил Ахмедов]

Dik_Nik, "CSV". Насколько я понял, там несколько типов документа. В таком случае лучше использовать enum.

[Dik_Nik]

Исрапил Ахмедов, к сожалению данное поле не задокументировано и я не знаю, какие в нем могут быть значения.
Но мне в принципе это и не важно, т.к я использую его только в одном месте и только csv, как вы могли увидеть.