utilizando & try/catch anidación

Question

Esta pregunta es más de un ¿cuál es la forma correcta de hacer algo ...

La pregunta ... ¿hay una orden de anidamiento adecuado entre un bloque y una usingtry/catch?

¿Está bien anidar toda la declaración using dentro de un try/catch y mantener los beneficios de un bloque using? (¿o una excepción hará que la porción de cierre de la sentencia using se descarte en la ventana)

¿O debe anidar el try/catch dentro de las declaraciones using y rodear solo las declaraciones que hacen acceso a la base de datos?

ES ...

try { 
    using(tsmtcowebEntities db = new tsmtcowebEntities()) { 
      violationList = (from a in db.DriverTrafficViolationDetails 
          where a.DriverTrafficViolation.DriverApplicationId == DriverAppId 
          orderby a.DateOfOccurance descending 
          select a).ToList<DriverTrafficViolationDetail>(); 
      GeneralViolation = (from a in db.DriverTrafficViolations 
           where a.DriverApplicationId == DriverAppId 
           select a).FirstOrDefault(); 
    } 
} catch { } 

menos/más correcto que ...

using(tsmtcowebEntities db = new tsmtcowebEntities()) { 
    try { 
      violationList = (from a in db.DriverTrafficViolationDetails 
          where a.DriverTrafficViolation.DriverApplicationId == DriverAppId 
          orderby a.DateOfOccurance descending 
          select a).ToList<DriverTrafficViolationDetail>(); 
      GeneralViolation = (from a in db.DriverTrafficViolations 
           where a.DriverApplicationId == DriverAppId 
           select a).FirstOrDefault(); 
    } catch { } 
} 

Answer 1

Lo posterior es mejor: evitará enmascarar las excepciones eventualmente lanzadas dy dispose. Ver esta article.

Answer 2

Yo sugeriría colocar el try/catch en el uso porque independientemente de si es o no es una excepción tirar debe deshacerse de los tipos el contenedor de entidad desechable

Answer 3

Es realmente una cuestión de estilo y qué tan estrecho quiere mantener el alcance de db:

Si el uso está dentro del bloque try/catch, la variable db solo será accesible dentro de la porción de prueba.

Si el uso está fuera del bloque try/catch, será visible dentro de la porción de captura.

Independientemente, la variable se eliminará correctamente porque el bloque de uso es el equivalente a un try/finally.

Personalmente, me pregunto por qué necesita detectar excepciones allí y, en todo caso, qué puede hacer con ellas.

Answer 4

using nidos predecible con try/catchDispose y se llamará todo en los caminos. Previsiblemente significa control siempre flujos desde el interior -> ámbitos externos (para Excepciones y retorno de flujo normal).

La pregunta es entonces: cuando caso de que el retén ser ejecutados en relación con el Disposey lo caso de que el alcance de la captura ser? La respuesta a esto variará según el código pero obviamente debe estar "en el interior" si se requiere acceso a db y "en el exterior" si el código ejecutado como parte de el using * podría ser la fuente de la excepción.

(Además, los bloques catch vacías son repulsivo! Estoy asumiendo que están ahí "para fines de demostración".)

feliz de codificación.

---

* Tenga en cuenta que una captura accesoria externa cogerá las excepciones lanzadas desde new tsmtcowebEntities() o (como señala J. N.) el Dispose, en caso de existir. (Es otro tema completamente si es aceptable que cualquiera de los constructores arroje una excepción ;-) Prefiero detectar excepciones lo más cerca posible de la fuente y dejar que las excepciones que no sé cómo manejar "sangren" no sean detectadas excepto en ciertas construcciones de alto nivel (por ejemplo, controladores de eventos).