Controllo elegante per null e uscita in C#

Question

Qual è un modo elegante di scrivere questo?

changeColor() è una funzione vuoto e la funzione che esegue il codice sopra è una funzione void pure.

Answer 1

È possibile ridurre l'ingombro invertendo la condizione:.

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

lastSelection.changeColor(); 

Answer 2

Anche se io personalmente piace davvero avere più istruzioni return in una funzione, in base alla programmazione difensiva che ho visto, che ci si intrappolare la condizione di errore e di uscita, e lasciate che qualsiasi altra cosa attraverso:

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

lastSelection.changeColor(); 

E 'difficile dire come mi piacerebbe fare questo nel mio lavoro senza vedere l'intera funzione questo appartiene a

Answer 3

Sarei tentato di verificare che lastSelection non sia mai stato null e che punti a un oggetto "vuoto" nei casi in cui sarebbe nullo nel progetto corrente. Quindi, non è necessario un controllo nulla. Progettare eventuali errori fuori dal sistema è sempre una buona pratica.

IOW, chiederei se l'ineleganza nel codice è come si effettua il controllo Null o se è il fatto che sia richiesto un controllo Null su tutti.

Answer 4

Penso che il tuo codice sia buono: prima hai controllato il caso normale (lastSelection != null) per il caso anomalo.

posso solo aggiungere una singola modifica:

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

Answer 5

Vedendo tutti gli altri ha le buone risposte, non farlo:

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

Answer 6

Una pratica recuperando veloce è quello di scrivere Nulla come prima:

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

lastSelection.changeColor(); 
 

Answer 7

Questo è ancora un altro modo di implementarlo.

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

Forse un po 'eccessivo, ma si avrà certamente la gente mantenere il vostro codice a grattarsi la testa :-)