Cheque elegante para nulo y salida en C#

Question

¿Qué es una forma elegante de escribir esto?

changeColor() es una función nula y la función que ejecuta el código anterior es una función void así.

Answer 1

puede reducir el desorden mediante la inversión de la condición:.

if (lastSelection == null) 
{ 
    MessageBox.Show("No Selection Made"); 
    return; 
} 

lastSelection.changeColor(); 

Answer 2

Aunque personalmente realmente no les gusta tener múltiples sentencias de retorno de una función, de acuerdo con la programación defensiva que he visto, que había trampa de la condición de error y salida, y dejar todo lo demás a través de:

if(lastSelection == null) 
{ 
    MessageBox.Show("No Selection Made"); 
    return; 
} 

lastSelection.changeColor(); 

es difícil decir cómo me gustaría hacer esto en mi propio trabajo sin ver toda la función de esto pertenece en

Answer 3

Tendría la tentación de asegurarme de que lastSelection nunca fue nulo, y hacer que apunte a un objeto "vacío" en los casos en que sería nulo en el diseño actual. Entonces, no hay necesidad de un cheque nulo. La ingeniería de posibles errores fuera del sistema siempre es una buena práctica.

OIA, me gustaría preguntar si la falta de elegancia en el código es como se hace el cheque nulo, o si se trata de el hecho de que se requiere un cheque nulo en absoluto.

Answer 4

Creo que su código es bueno: primero verificó el caso normal (lastSelection != null) para el caso anormal.

que sólo puede añadir una sola modificación:

if (lastSelection != null) 
{ 
    lastSelection.changeColor(); 
    //your other normal code 
} 
else 
{ 
    MessageBox.Show("No Selection Made"); 
    //return; //no need for this now 
} 

Answer 5

Al ver a todos los demás dieron las buenas respuestas, no hace esto:

if (lastSelection == null) goto ERR; 
lastSelection.changeColor(); 
//... possibly more stuff... 
return; 
ERR: 
MessageBox.Show("No Selection Made"); 

Answer 6

Una práctica avanza a pasos agigantados es escribir null primero como a continuación:

if (null == lastSelection) 
{ 
    MessageBox.Show("No Selection Made"); 
    return; 
} 

lastSelection.changeColor(); 
 

Answer 7

Esto es aún otra forma de implementar esto.

 Action _action = (lastSelection != null 
           ? 
            new Action(lastSelection.changeColor) 
           : 
           () => Console.WriteLine("No Selection Made")); 
     _action.Invoke(); 

Tal vez un poco exagerado, pero que sin duda tendrá la gente mantener su código rascándose la cabeza :-)