Java: hacer referencia nulo después de cerrar la secuencia

Question

¿Es una buena práctica establecer las referencias de flujo a nulo después de cerrarlas? ¿Esto liberaría recursos de alguna manera?

Ejemplo:

BufferedReader input= new BufferedReader(new FileReader("myfile.txt")); 

// code 

input.close(); 
input = null; 

// possible more code 

Answer 1

No, es una mala práctica. OMI, incluso debería considerar hacer la variable final.

El manejo de recursos se debe manejar de la manera estándar acquire(); try { use(); } finally { release(); }. En este caso:

final Reader rawIn = new FileReader("myfile.txt"); // Character encoding?? 
try { 
    BufferedReader in = new BufferedReader(rawIn); 

    // code 

} finally { 
    rawIn.close(); 
} 

En realidad, este código selecciona la codificación de caracteres que está configurada por defecto. Sugiero ser explícito con un conjunto de caracteres codificados o con Parameterise from Above.

final InputStream rawIn = new FileInputStream("myfile.txt"); 
try { 
    BufferedReader in = new BufferedReader(
     new InputStreamReader(rawIn, "UTF-8") 
    ); 

    // code 

} finally { 
    rawIn.close(); 
} 

No debe crear los flujos de envoltura/lectores de fuera del bloque try (y antes de la asignación de recursos), ya que podría lanzar. Del mismo modo, su cierre podría arrojar (esto fue en realidad un error en BufferedOutputStream que podría lanzar en flush). Algunas secuencias de entrada pueden tener otros recursos, por lo que necesita dos try { ... finally { x.close(); } s.

Para la salida, generalmente debe flush en el curso normal de los eventos. Pero generalmente no en casos excepcionales. De hecho close generalmente hace un flush, por lo tanto, no debe cerrarlos en el caso excepcional. Si decoradores ambos flush y tienen recursos, entonces tendrá que sonreír y mostrarlo.

Hay ocasiones muy infrecuentes en que anular es una buena idea. Por ejemplo, si una variable es la única referencia a un objeto grande y va a crear un nuevo objeto grande para asignarle, puede ser mejor borrar la referencia para permitir que se recupere la antigua antes de asignar la nueva.

Answer 2

No es necesario. Solo input.close() es suficiente. Recuerde que siempre es aconsejable hacerlo dentro de un bloque finally. Y antes de llamar a su close() mejor que hacer un cheque nulo como esto

finally{ 
    if(input!=null){ 
    input.close(); 
    } 
} 

Answer 3

A menos que estés administración manual de un charco de rosources (el manejo de su propia memoria), no es necesario anular el flujo de entrada. Idealmente, cualquiera que sea la función en la que se encuentre es pequeña, y las referencias al objeto morirán a medida que el objeto salga del alcance, marcándolo de todos modos para la recolección de elementos no utilizados.

Menciono los recursos de la agrupación, como si ingenuamente cerrara la secuencia sin anular el objeto, podría retener accidentalmente una referencia del objeto que realmente no necesita.

Answer 4

Se puede hacer que el objeto Stream sí aptos para reciclaje, pero

1. En la mayoría de los casos va a pasar fuera de alcance justo después de todos modos
2. la cantidad de memoria por lo tanto "liberado" es completamente insignificante

Answer 5

En este caso no es necesario, el recolector de basura lo recogerá.

Pero no siempre es una mala práctica asignar null. Leer el artículo 6 de Effective Java, chapter 2

Answer 6

No, no lo es. El close() ya libera el recurso. La práctica habitual es la siguiente:

Resource resource = null; 
try { 
    resource = new Resource(); 
    // ... 
} finally { 
    if (resource != null) try { resource.close(); } catch (ResourceException logOrIgnore) {} 
} 

Dónde Resource puede ser cualquier recurso externo desea utilizar, como InputStream, OutputStream, Reader y Writer de API de Java IO, sino también, por ejemplo, Connection, y StatementResultSet de API JDBC.

La memoria no es un problema en un código bien diseñado. Si el código deja el bloque de método, ya es elegible para GC.

Puede refactorizar el close() a un método de utilidad como:

public static void close(Closeable resource) { 
    if (resource != null) { 
     try { 
      resource.close(); 
     } catch (ResourceException logOrIgnore) { 
      // Log it? 
     } 
    } 
} 

que se puede utilizar de la siguiente manera:

} finally { 
    close(resource); 
} 

El Apache Commons proporciona varios métodos de utilidad por el estilo.