¿Salir de un método vacío?

Question

tengo este método (código modificado):

public static void PublishXmlForCustomTypes(MyOwnClass DefaultOutputInformation) 
{ 
    if (DefaultOutputInformation != null) 
    { 
     ///lot of code 
    } 
} 

y todo mi código estaba dentro de la sentencia if y después de pensarlo, he cambiado a esto:

public static void PublishXmlForCustomTypes(MyOwnClass DefaultOutputInformation) 
{ 
    if (DefaultOutputInformation == null) 
    { 
     return; 
    } 
    ///lot of code 
} 

Por lo que he probado parece ser estrictamente equivalente, pero ¿realmente es así? Quiero decir, la declaración de "devolución" nos saca del método ¿verdad?

Answer 1

Esto es estrictamente equivalente y la segunda versión es el camino a seguir :)

Answer 2

Sí, su hipótesis es correcta.

Para obtener más información, obtenga más información acerca de duality.

Answer 3

Sí, es exactamente la misma, se puede leer la documentación de MSDN sobre el regreso de palabras clave para entender completamente cómo funciona: http://msdn.microsoft.com/en-us/library/1h3swy84.aspx

En cuanto a decidir qué camino es mejor: ambos son buenos, pero la segunda versión lo hace más legible porque entonces todo su código no está dentro de un bloque if. De esta manera, puede ver lo que hace la condición realmente fácil en lugar de leer todo el código del método.

Answer 4

sí return le saca del método; si tiene un bloque finally y llama a return desde el bloque try, el bloque finally se ejecuta de todos modos.

Answer 5

Sí, eso está absolutamente bien.

Algunas personas se adhieren dogmáticamente a "un punto de salida por método", que era apropiado cuando era relativamente complicado asegurarse de que siempre realizara la cantidad correcta de limpieza al final de una función en C, por ejemplo. .. pero no es realmente necesario en C#.

Personalmente, creo que es apropiado regresar tan pronto como sepa que ya ha realizado todo el trabajo que realmente desea en un método. Use las declaraciones try/finally o using para realizar cualquier trabajo extra de "limpiar, pero salgo".

Answer 6

De hecho, el return le saca del método, por lo que es equivalente a la primera forma que utilizó. Qué camino es mejor depende de tu código, aunque en general preferiría la segunda versión.

Answer 7

Sí, la declaración de return finaliza el método.

Answer 8

Sí, la devolución saldrá del código. En general, es una buena práctica como el primer paso de una función verificar que los parámetros pasados ​​son los que usted cree que son y salir (mediante el retorno o lanzar una excepción) para que no haga ningún procesamiento innecesario solo para tiene que abortar más tarde en la función.

Answer 9

Al mirar el código revisado, el segundo es el camino a seguir. Si bien es funcionalmente equivalente, piense en el caso en que transfirió 4 variables diferentes a una función que desea verificar. En lugar de hacer una mala declaración de 4 niveles si con {en todas partes, el segundo método le permite limpiar la apariencia del código y no agregar niveles innecesarios de corchetes. Si está escribiendo en C/C++, incluso puede hacer que esta sea una macro como VERYIFY_NOT_NULL (x) y hacer que el código sea agradable y ordenado.

El código legible/mantenible supera los nano segundos de rendimiento el 99% del tiempo.