¿Por qué atrapar una excepción solo para tirarla de nuevo?

Question

En un servicio web que ver este código:

<WebMethod()> _ 
Public Function dosomething() As Boolean 
    Try 
     If successful Then 
      Return True 
     Else 
      Return False 
     End If 
    Catch ex As Exception 
     Throw ex 
    End Try 
End Function 

Cuál es el punto de agarrar la excepción y sólo tirar de nuevo? ¿Extraño algo?

Edit: Gracias por las respuestas! Pensé que era algo así, pero no estaba seguro si podría/podría refactorizarlos sin ninguna implicación.

Answer 1

No puedo pensar en ninguna razón para hacer esto por la funcionalidad. Sin embargo, puede surgir cuando anteriormente se eliminó algún tipo de manejo de errores (normalmente el registro), y el desarrollador eliminó el manejo del registro pero no reestructuró el código para eliminar el try/catch redundante.

Answer 2

Probablemente queda un poco de código de depuración (establecería un punto de interrupción en el lanzamiento para que pueda examinar la excepción en el depurador). Podría hacer algo como esto si quisiera registrar la excepción y luego pasarla por la cadena, aunque probablemente incluiría la excepción en otra con un mensaje de error más significativo (para mi aplicación).

Answer 3

No haga esto.

Si necesita volver a generar la excepción, simplemente use throw; usando throw ex; borra el rastreo de la pila y es absolutamente incorrecto.

Answer 4

Es posible que desee hacerlo si desea detectar una excepción, excepto una subclase de la misma.

Por ejemplo,

try { 
    // Something stupid 
} 
catch(RuntimeException e) { 
    throw e; //Handle it outside 
} 
catch (Exception e) { 
    // I'm dead 
} 

Answer 5

Una de las arquitecturas (design patterns) pude ver esta siendo utilizado en está donde está siendo manejado una transacción. La función hace su trabajo, falla y el bloque catch completa la transacción a un estado conocido (por lo general, un rollback) y luego arroja una excepción definida por el usuario.

Como está parado ahora, refactorice ese código a un estado más sano.

Answer 6

El try/catch no solo es inútil, sino que también lo es el IF. Toda la función se puede reducir a una línea:

return successful 

En ese momento, ¿para qué molestarse? ¿Por qué no simplemente probar "con éxito" en lugar de llamar a la función?

Bueno, está bien, es un método web, supongo que necesitas la función solo para darle a Ajax o quien sea un controlador.

(Sí, soy consciente de que mi respuesta es de 7 años de retraso. Yo sólo me encontré con este en la búsqueda de algo sin relación.)

Answer 7

El patrón para el monitor hace que este muy probable que volver a lanzar un error como sea necesario un Monitor.Exit fin de asegurar se llama

https://msdn.microsoft.com/en-us/library/4tssbxcw(v=vs.110).aspx

Dim lockObj As New Object() 

If Monitor.TryEnter(lockObj) Then 
    Try 
     ' The critical section. 
    Catch 
     throw 
    Finally 
     ' Ensure that the lock is released. 
     Monitor.Exit(lockObj) 
    End Try 
End If