Есть ли чистый способ предоставить объекты объектов Java с уничтожением?

Question

Представьте, что MyOpenedFile - это что-то, обертывающее файл с открытыми потоками. Теперь предположим, этот код:

// method in an Util class 
static void safeClose(MyOpenedFile f) { 
    if (f != null) { 
    try { 
     f.close(); 
    } catch(IOException ex) { /* add logging or console output */ } 
    } 
} 

Фактический метод вопрос:

void doSomeFileOperation(...) throws IOException, ... { 
    MyOpenedFile f1 = null; 
    MyOpenedFile f2 = null; 
    try { 

     /* method's "business logic" code beings */ 
     f1 = new MyOpenedFile(...); 
     //do stuff 
     f2 = new MyOpenedFile(...); 
     // do stuff 
     f1.close(); f1 = null; 
     // do stuff with f1 closed 
     f2.close(); f2 = null; 
     // do stuff with f2 closed 
     /* method's "business logic" code ends */ 

    } finally { 
     Util.safeClose(f1); f1 = null; 
     Util.safeClose(f2); f2 = null; 
    } 
} 

Сейчас это довольно неаккуратно и особенно подвержены ошибкам (некоторый код в конце концов блок может быть очень трудно получить вызов в блоке тесты, например). Например, в C++ деструктор позаботится о том, чтобы очистить (либо получить вызов с помощью деструктора указателя области действия, либо напрямую), и код будет намного чище.

Итак, есть лучше/красивее/уборщик способ обернуть выше части бизнес-логики кода, так что любые исключения получить размножают, но оба файла f1 и f2 закрываются (или по крайней мере близко предпринимается на обоих, даже если он не получится)?

Также ответы, указывающие на любые библиотеки с открытым исходным кодом, такие как Apache Commons, обеспечивая приятные обертки.

Answer 1

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

Ресурс вам нужно закрыть это FileInputStream или BufferedReader, и вы можете закрыть это неявно с ARM в Java 7

try(BufferedReader br = new BufferedReader(new FileReader(file))) { 

} 

Это закроет br whent он блокирует выходы.

http://www.oracle.com/technetwork/articles/java/trywithresources-401775.html

Answer 2

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

Другой бит, что образец у вас есть действительно много лишних, ненужных шагов

void doSomeFileOperation(...) throws IOException, ... { 
    File f1 = null; 
    File f2 = null; 
    try { 
     f1 = new File(...); 
     f2 = new File(...); 
     // callback to another class/method that does the real work 
    } finally { 
     Util.safeClose(f1); 
     Util.safeClose(f2); 
    } 
} 

Вам не нужно устанавливать экземпляры файлов в нуль. Если вы попытаетесь использовать их, вы получите исключение.

Я не удивляюсь, какой File объект, который вы используете. Стандартный класс File в java не имеет метода close().

Answer 3

Посмотрите на The try-with-resources Statement, который закроет ресурсы после завершения блока try.

Класс File, который вы используете, кажется, не java.io.File, потому что он не имеет никакого метода close(). В этом случае убедитесь, что ваш собственный класс File реализует Closeable, чтобы он работал с ARM.

try (FileInputStream f1 = new FileInputStream("test1.txt"); 
    FileInputStream f2 = new FileInputStream("test2.txt")) { 
    // Some code 
} catch (FileNotFoundException e) { 
    e.printStackTrace(); 
} catch (IOException e) { 
    e.printStackTrace(); 
} 

Answer 4

Вам не нужно закрывать файлы (которые являются представлением файлов в файловой системе), как упомянуто здесь:

Do I need to close files I perform File.getName() on?

Я предполагаю, что вы просите больше о файловых потоках/читателях?

В этом случае Java 7 имеет хорошую новую особенность: http://www.vineetmanohar.com/2011/03/java-7-try-with-auto-closable-resources/

Если вы работаете на более старую версию Java я просто держать его просто с этим:

void doSomeFileOperation(...) throws IOException, ... { 
    FileInputStream f1 = null; 
    FileInputStream f2 = null; 
    try { 

    // do stuff 

    } finally { 
    Util.safeClose(f1); 
    Util.safeClose(f2); 
    } 
}