Esiste un modo migliore per scrivere l'intero contenuto di un file in un OutputStream?

Question

Quando ho voglia di scrivere l'intero contenuto di un file in un OutputStream, di solito allocare un buffer come byte[], poi fare un ciclo for ai dati read dal file di InputStream nel buffer e scrivere il contenuto del buffer nel OutputStream, fino a quando lo InputStream non ha più byte disponibili.

Questo mi sembra piuttosto maldestro. C'è un modo migliore per farlo?

Inoltre, non sono sempre sicuro della dimensione del buffer. Di solito, sto allocando 1024 byte, perché è semplicemente buono. C'è un modo migliore per determinare una dimensione del buffer ragionevole?

Nel mio caso corrente, voglio copiare l'intero contenuto di un file nel flusso di output che scrive il contenuto di una risposta HTTP. Quindi, questa non è una domanda su come copiare i file sul file system.

Answer 1

Per Java 1.7+ è possibile utilizzare Files.copy(Path, OutputStream), ad es.

HttpServletResponse response = // ... 
File toBeCopied = // ... 

try (OutputStream out = response.getOutputStream()) { 
    Path path = toBeCopied.toPath(); 
    Files.copy(path, out); 
    out.flush(); 
} catch (IOException e) { 
    // handle exception 
} 

nota, dal momento che si tratta di HttpServletResponse è è anche una buona idea per impostare intestazioni di risposta corretta. Aggiungere le seguenti righe prima di copiare i dati del file effettivi alla risposta:

String mimeType = URLConnection.guessContentTypeFromName(toBeCopied.getName()); 
String contentDisposition = String.format("attachment; filename=%s", toBeCopied.getName()); 
int fileSize = Long.valueOf(toBeCopied.length()).intValue(); 

response.setContentType(mimeType); 
response.setHeader("Content-Disposition", contentDisposition); 
response.setContentLength(fileSize); 

nota, la codifica del nome del file passato alla disposizione contenuto è importante, vedi this question.

Answer 2

Con commons-io si dispone di una soluzione one-line:

IOUtils.copy(yourFileInputStream, outputStream); 

Nota che avresti dovuto chiudere i flussi manualmente (o con IOUtils.closeQuitely(..))

Answer 3

Apache Commons-IO:

IOUtils.copy(fileInputStream,outputStream); 

JDK NIO

new FileInputStream(file).getChannel().transferTo(otherChannel);