Cuándo llamar a WebResponse.Close()

Question

WebResponse response; 
try 
{     
HttpWebRequest request = (HttpWebRequest)WebRequest.Create(url); 
request.Timeout = 20000; 
response = request.GetResponse(); 

request = (HttpWebRequest)WebRequest.Create(url2); 
response = request.GetResponse(); 
} 
catch(Exception ex) 
{ 
//do something 
}    
finally 
{ 
} 

donde debe llamarse response.Close()?

• después de cada GetResponse() in try?

• después del último GetResponse() in try - once?

• en el bloque finalmente?

Answer 1

Ninguna de las anteriores. Usted debe utilizar un bloque using:

HttpWebRequest request = (HttpWebRequest)WebRequest.Create(url); 
request.Timeout = 20000; 
using (WebResponse response = request.GetResponse()) 
{ 
    using (var stream = response.GetResponseStream()) 
    { 
     using (var reader = new StreamReader(stream)) 
     { 
      var result = reader.ReadToEnd(); 
      // Do something with result 
     } 
    } 
} 

Un bloque using se asegurará de que el método Dispose se llama, si existe o no es una excepción. Dispose hará lo mismo que Close.

using (var d = new DisposableClass()){code;} 

es equivalente a:

DisposableClass d = null; 
try 
{ 
    d = new DisposableClass(); 
    code; 
} 
finally 
{ 
    if (d != null) 
     ((IDisposable)d).Dispose(); 
} 

Answer 2

Colóquelo en el bloque finally. Según MSDN:

El bloque finally es útil para la limpieza de los recursos asignados en el bloque try , así como ejecutar cualquier código que debe ejecutar incluso si no es una excepción. El control siempre es pasado al bloque finally independientemente de de cómo sale el bloque try.

Answer 3

que sugeriría el siguiente

 try 
     { 
      HttpWebRequest request = (HttpWebRequest)WebRequest.Create("http://www.google.com"); 
      request.Timeout = 20000; 
      using (var response = request.GetResponse()) 
      { 
       //Do something with response. 
      } 


      request = (HttpWebRequest)WebRequest.Create("http://www.bing.com"); 
      using (var response = request.GetResponse()) 
      { 
       //Do somehing with response 
      } 
     } 
     catch (Exception ex) 
     { 
      //do something 
     } 
     finally 
     { 
     } 

Answer 4

Tenga en cuenta que el uso de bloques anidados no necesitan llaves, mejorar la legibilidad. Así código de John Saunders se podría escribir:

HttpWebRequest request = (HttpWebRequest)WebRequest.Create(url); 
request.Timeout = 20000; 
using (WebResponse response = request.GetResponse()) 
using (var stream = response.GetResponseStream()) 
using (var reader = new StreamReader(stream)) 
{ 
    var result = reader.ReadToEnd(); 
    // Do something with result 
} 

VS.NET entiende que tales bloques anidados no necesitan sangría. Tenga en cuenta que si conoce la codificación de la respuesta o la va a ignorar, WebClient proporciona una API más simple: falta información de encabezado, por lo que la detección de codificación basada en encabezado (transferencia/texto) se vuelve imposible, pero de lo contrario funciona bien.