Devo chiudere il flusso di output del servlet?

Question

Eventuali duplicati:
Should one call .close() on HttpServletResponse.getOutputStream()/.getWriter()?

sono io responsabile per la chiusura del HttpServletResponse.getOutputStream() (o il getWriter() o anche l'InputStream) o dovrei lasciare al contenitore?

protected void doGet(HttpServletRequest request, HttpServletResponse response) 
    throws ServletException, IOException { 
    OutputStream o = response.getOutputStream(); 
    ... 
    o.close(); //yes/no ? 
} 

Answer 1

In effetti non è necessario.

Regola del pollice: se non lo hai creato/aperto da solo utilizzando new SomeOutputStream(), non è necessario chiuderlo da solo. Se fosse per esempio uno new FileOutputStream("c:/foo.txt"), allora ovviamente hai bisogno di chiuderlo da solo.

I motivi per cui alcune persone lo fanno ancora sono solo per assicurare che non verrà scritto nient'altro nel corpo della risposta. Se ciò dovesse accadere, ciò causerebbe un IllegalStateException nei registri dell'appserver, ma ciò non influirebbe sul client, quindi il client otterrà comunque la risposta corretta. Questo è anche un debug più facile per individuare i potenziali problemi nella catena richiesta-risposta che non si vedrebbero a prima vista. Ad esempio, qualcos'altro aggiunge altri dati al corpo della risposta da qualche parte più in basso nella catena.

Un'altra ragione che si vede tra i principianti è che volevano solo impedire che più dati siano scritti nel corpo della risposta. Lo si vede spesso quando JSP ha un ruolo non corretto nella risposta. Ignorano semplicemente i IllegalStateException nei registri. Inutile dire che questo particolare scopo è cattivo.

Answer 2

No non è necessario chiuderlo. Se lo fai fondamentalmente finisci la risposta al cliente. Dopo aver chiuso lo stream non è possibile inviare altro al client fino alla richiesta successiva. Non hai aperto lo stream, quindi non devi chiuderlo.