¿Debo cerrar explícitamente el StreamReader en C# cuando lo uso para cargar un archivo en una variable de cadena?

Question

Ejemplo:

variable = new StreamReader(file).ReadToEnd(); 

¿Es esto aceptable?

Answer 1

No, esto no cerrará el StreamReader. Tienes que cerrarlo. Usando lo hace por usted (y la dispone por lo que se GC'd antes):

using (StreamReader r = new StreamReader("file.txt")) 
{ 
    allFileText = r.ReadToEnd(); 
} 

o alternativamente en .Net 2 se puede utilizar el nuevo archivo. miembros estáticos, entonces no es necesario que cierre nada:

variable = File.ReadAllText("file.txt"); 

Answer 2

Siempre debe disponer de sus recursos.

// the using statement automatically disposes the streamreader because 
// it implements the IDisposable interface 
using(var reader = new StreamReader(file)) 
{ 
    variable = reader.ReadToEnd(); 
} 

O por lo menos llamar manualmente:

reader = new StreamReader(file); 
variable = reader.ReadToEnd(); 
reader.Close(); 

Answer 3

Usted necesita deshacerse de objetos que implementan IDisposable. Use una declaración using para asegurarse de que se elimine sin llamar explícitamente al método Dispose.

using (var reader = new StreamReader(file)) 
{ 
    variable = reader.ReadToEnd(); 
} 

Como alternativa, utilice File.ReadAllText(String)

variable = File.ReadAllText(file); 

Answer 4

Es una buena práctica para cerrar StreamReader. Utilice la siguiente plantilla:

string contents; 

using (StreamReader sr = new StreamReader(file)) 
{ 
    contents = sr.ReadToEnd(); 
} 

Answer 5

Es mejor utilizar la palabra clave usando; entonces no es necesario cerrar explícitamente nada.

Answer 6

Sí, siempre que se crea un objeto desechable debe disponer de él de preferencia con una instrucción using

using (var reader = new StreamReader(file)) { 
    variable = reader.ReadToEnd(file); 
} 

En este caso, sin embargo, puedes usar el método File.ReadAllText para simplificar la expresión

variable = File.ReadAllText(file);