Обработать временный файл в попытке с ресурсами

Question

Для моего приложения мне нужно написать метод, который принимает аргумент InputStream, записывает содержимое во временный файл, выполняет некоторые операции и, наконец, удаляет временные файлы.

Это то, что я до сих пор:

public void myMethod(InputStream in, String name) { 
    //... 
    Path path = Paths.get("./tmp/benchmarks/" + name + ".zip") 

    try { 
     Files.copy(in, path); 
     //operations... 
    } catch (IOException e) { 
     //error handling for copy... 
    } finally { 
     try { 
      Files.delete(path)); 
     } catch (IOException e) { 
      //error handling for delete... 
     } 
    } 
    //... 
} 

Это делает работу, но и выглядит очень некрасиво. Мне было интересно, есть ли способ использовать try-with-resources, чтобы обработать это более изящно. Возможно ли это как-то?

ОБНОВЛЕНИЕ: Я написал решение «на лету» за десять минут. Это выглядит так:

public class TemporaryFileHandler implements AutoCloseable { 

    private File file; 

    public TemporaryFileHandler(final InputStream in, final Path path) throws IOException { 
     Files.copy(in, path); 
     this.file = new File(path.toString()); 
    } 

    public File getFile() { return file; } 

    @Override 
    public void close() throws IOException { 
     Files.delete(file.toPath()); 
    } 
} 

Я уверен, что это не самое лучшее, но сейчас он работает. Если у кого-то есть предложения по улучшению этого в любом случае, предложения более чем приветствуются.

Answer 1

Try-with-resource просто вызывает метод close для классов, реализующих интерфейсы, реализующие java.lang.AutoCloseable. Ничто не мешает вам создать реализацию Файла, которая реализует AutoCloseable и удаляет себя, когда вызывается функция close().

Вы также можете позвонить в файл deleteOnExit(), чтобы удалить JVM при его завершении. Это подходит только в том случае, если вы в порядке, ожидая, пока JVM не будет завершена, чтобы удалить ваш временный файл. Вероятно, это не будет хорошей идеей для долгого JVM, как Java Webapp.

Answer 2

Files.createTempFile позволяет создать временный файл в временном каталоге по умолчанию JVM. Это не означает, что файл автоматически удаляется или помечен для удаления с использованием File.deleteOnExit(). Разработчик отвечает за управление жизненным циклом временных файлов.

Имя файла является вектором уязвимостей безопасности. Используйте только имя для отображения в пользовательском интерфейсе для проверки и обратной связи. Не используйте ненадежный пользовательский ввод для имен файлов.

Невозможно управлять жизненным циклом файла с помощью ресурсов try-with-ресурсами, используя java.io.File или java.nio.file.Path. InputStream управляется с помощью try-with-resources.

public void myMethod(InputStream in) { 
    Path path = null; 

    try (InputStream stream = in) { 
     path = Files.createTempFile(null, ".zip"); 
     Files.copy(stream, path); 
    } catch (IOException e) { 
     //error handling for copy... 
    } finally { 
     if (path != null) { 
      try { 
       Files.delete(path)); 
      } catch (IOException e) { 
       //error handling for delete... 
      } 
     } 
    } 
} 

Answer 3

Я думаю, с небольшим хелперов/обертку, как

public class AutoDeletingTempFile implements AutoCloseable { 

    private final Path file; 

    public AutoDeletingTempFile() throws IOException { 
     file = Files.createTempFile(null, null); 
    } 

    public Path getFile() { 
     return file; 
    } 

    @Override 
    public void close() throws IOException { 
     Files.deleteIfExists(file); 
    } 
} 

, который закрывается и удаляет файл он оборачивает вы получите хороший и короткий синтаксис:

public void myMethod(InputStream in, String name) { 
    try (AutoDeletingTempFile wrapper = new AutoDeletingTempFile()) { 
     //Files.copy(in, wrapper.getFile()); 
     //operations... 
    } catch (IOException e) { 
     //error handling for copy... 
     // + temp file creation 
    } 
} 

---

или аккуратный маленький Closable через lambdas

public void myMethod(InputStream in, Path existingFile, String name) { 
    try (Closeable closable =() -> Files.deleteIfExists(existingFile)) { 
     // ... 
    } catch (IOException e) { 
     // 
    } 
} 

Answer 4

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

Мое решение, которое использует вложенную попытку, не так элегантно, но должно гарантировать, что потоки будут закрыты заранее.

Перед

File out = // get file; 

try(
    FileOutputStream fos = new FileOutputStream(out); 
    ZipOutputStream zos = new ZipOutputStream(fos); 
){ 
    // Create ZIP file and deliver to client using HTTPServletResponse 
} 
finally{ 
    if (out != null){ 
     out.delete(); 
    } 
} 

После

File out = // get file; 

try{ 
    try(
     FileOutputStream fos = new FileOutputStream(out); 
     ZipOutputStream zos = new ZipOutputStream(fos); 
    ){ 
     // Create ZIP file and deliver to client using HTTPServletResponse 
    } 
} 
finally{ 
    if (out != null){ 
     out.delete(); 
    } 
} 

Answer 5

Вы могли бы сделать что-то подобное в Java 8:

Path path = Files.createTempFile("temp-", ".tmp"); 
try (Closeable onClose =() -> Files.delete(path)) { 
    ... 
} 

, но это действительно так же, как:

Path path = Files.createTempFile("temp-", ".tmp"); 
try { 
    ... 
} finally { 
    Files.delete(path); 
}