Ручное закрытие внутри try-with-resource

Question

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

Из-за отсутствия документации неясно, закрывает ли этот метод поток после его завершения, поэтому одним из возможных решений может быть обернуть вызов в try-with-resource, просто чтобы быть включенным безопасная сторона.

К сожалению, спецификация Java делает (насколько я могу судить) отсутствие упоминания о том, что произойдет, если ресурс закрыт вручную внутри try-with-resource. Кто-нибудь знает?

Answer 1

Это полностью зависит от реализации самого ресурса. Оператор try-with-resource является «просто» синтаксическим сахаром (но так сладким) для вызова close() в блоке finally (и сохранения исключений и т. Д.).

До тех пор, пока поток поддерживает close(), который вызывается дважды, - что я ожидаю от большинства реализаций, и требуется контракт InputStream - это будет абсолютно нормально.

Обратите внимание, что вы были бы в той же ситуации со знакомой упаковкой одного ресурса в другом, например.

try (InputStream input = new FileInputStream(...)) { 
    try (Reader reader = new InputStreamReader(input, ...)) { 
     ... 
    } 
} 

Или с одной примерки с-ресурсов заявление:

try (InputStream input = new FileInputStream(...); 
    Reader reader = new InputStreamReader(input, ...)) { 
    ... 
} 

В обоих случаях будет два finally блоки, так что первый reader.close() называется, то input.close() - но reader.close() закроет input так или иначе.

Answer 2

Спецификация говорит все, что может: если resource.close() выдает исключение, это исключение будет выбрано из конструкции.

Спецификация не может знать, будет ли какой-либо конкретный метод close выдавать исключение, конечно. Чтобы узнать это, вы должны проверить свой ресурс.

Answer 3

Вы можете попытаться закрыть его в описании finally.

InputStream stream = null; 
try { 
    stream = new InputStream(); 
    yourMethod(stream); 
} catch (...) { 

} finally { 
    try { 
    stream.close() 
    } catch (IOException ioe) { 
    // can throw IOException while closing closed stream 
    } 
} 

Answer 4

Метод close() из Closeable (и, таким образом, из InputStream) требуется для идемпотентным:

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

Поэтому безопасно закрыть InputStream более одного раза. Интерфейс

Однако, более общий AutoCloseable не требует его метод close() в идемпотентную, поэтому это может быть небезопасно делать то же самое для других ресурсов, чем Closeable:

Обратите внимание, что в отличие от близкого метода закрывающиеся, это метод close не должен быть идемпотентным. Другими словами, вызов этого закрытого метода более одного раза может иметь некоторый видимый побочный эффект, в отличие от Closeable.закрытие, которое не должно иметь эффекта, если вызвано более одного раза. Однако разработчикам этого интерфейса настоятельно рекомендуется сделать их близкие методы идемпотентными.