¿Debo cerrar FileInputStream?

Question

Estoy trabajando como aprendiz en Test Automation. Estoy trabajando con la creación de código Junit con Eclipse y ejecuto usando Eclipse. En eso estoy recuperando los datos de la hoja de Excel usando la función FileInputStream.

FileInputStream fi=new FileInputStream("c:\\search.xls"); 
Workbook w=Workbook.getWorkbook(fi); 
Sheet s=w.getSheet(0); 

¿Es necesario cerrar la función Inputstream? Si es así, guíanos con algunas codificaciones.

Answer 1

Sí, necesita close la intensidad de entrada si desea liberar los recursos de su sistema.

FileInputStream.close() es lo que necesita.

Answer 2

FileInputStream fi=null; 
try { 
    fi=new FileInputStream("c:\\search.xls"); 
    Workbook w=Workbook.getWorkbook(fi); 
    Sheet s=w.getSheet(0); 
} finally { 
    if (fi!=null) { 
     fi.close(); 
    } 
} 

Answer 3

Debe cerrar() o finalizar su programa.

Sin embargo, puede que tenga problemas confusos si no cierra el archivo como

• veces la prueba se ejecutan de forma individual o un grupo de prueba se ejecutan en el mismo proceso. (De modo que podría tener una prueba que funciona de una forma pero no de la otra)
• no puede cambiar el nombre ni eliminar un archivo abierto.

Es una buena práctica cerrar siempre los recursos que haya terminado con ellos, sin embargo, veo las pruebas unitarias como secuencias de comandos que no siempre tienen que seguir las mejores prácticas.

Answer 4

Basic CompSci 101 nos dicen que nos aseguremos de cerrar los recursos que abrimos, en Java o en cualquier idioma. Entonces sí, debes cerrarlos. El mal juju está destinado a suceder cuando no lo haces.

Además, debe aprender (y tener la inclinación) a utilizar Javadocs. Mire el Javadoc para FileInputStream y Closeable. Las respuestas están ahí.

Answer 5

Es siempre una buena idea para cerrar los recursos que utiliza, PERO:

Si utiliza recursos Un en recursos B, es sensato cerrar B en lugar de Un si tiene un método para eso.

En su caso, se utiliza en FileInputStreamWorkbook, así que es mejor para cerrar Workbook y se basan en Workbok que cerrará FileInputStream.

En este caso en particular, en realidad, Workbookwill closeFileInputStream al final del método getWorkbook() pero aún así es una buena idea para closeWorkbook para poder ser basura recogida.

Answer 6

Sí! siempre debe liberar los recursos una vez después de que haya terminado con ellos. Java tiene un poderoso mecanismo para la recolección de basura (tenga en cuenta que es diferente a la gestión de recursos/fugas). Entonces, ¿un recolector de basura no puede determinar si necesita el recurso en el futuro o no? No liberar los recursos puede causar problemas como: Denegación de servicios, bajo rendimiento.

Como ya se respondió pero otra manera es menos esfuerzo try with resources

try (FileInputStream fi = new FileInputStream("c:\\search.xls")) { 

     //do something with fi. 
     //fi.getChannel() ; 

    } catch(IOException e) { 
     // exception handling. 
    } finally { 
    // some statements for finally. 
    } 

Ahora usted no necesita fi.close método call() de forma explícita.

Answer 7

Recientemente, cuando traté de refactorizar mi código, tuve que mover la creación del libro a otro método, y FileInputStream se creó en ese método. Ese método crea un FileInputStream y devuelve un libro de trabajo. Pero FileInputStream no es visible desde el método principal; Entonces, ¿cómo cerraré mi FileInputStream al final del método principal? La respuesta es que no tiene que cerrar FileInputStream, sino que simplemente cierra el libro, que cierra internamente FileInputStream. En resumen, es incorrecto decir que debe cerrar FileInputStream sin importar qué.