Come evitare multipla se controlli nulli

Question

possibili duplicati:
Deep Null checking, is there a better way?
C# elegant way to check if a property's property is null

devo fare una ricerca in un modello a oggetti profonda come questo:

p.OrganisationalUnit.Parent.Head.CurrentAllocation.Person; 

c'è comunque da valutare e restituire null se c'è o f la catena è nullo (OrganizationalUnit, genitore, di testa, ecc), senza dover fare un

if (p.org == null && p.org.Parent == null && p.org.Parent.Head . . .  

Answer 1

Stai cercando il nulla-safe operatore dereference ?. (noto anche come una navigazione sicura) che alcune lingue (per esempio Groovy), ma sfortunatamente C# non ha questo operatore.

Speriamo che sarà attuato uno giorno ....

Vedi anche this post da Eric Lippert. La sintassi che propone è .?.

Answer 2

Avete sentito parlare del Law of Demeter?

Concatenare sequenze di chiamate così lunghe non è una buona idea. Crea dipendenze terribili tra le classi che non ti servono.

Nell'esempio, la classe contenente p diventa dipendente da cinque classi cinque. Ti suggerisco di semplificare il tuo codice e fare in modo che ogni classe controlli i null a un unico livello, nel loro contesto di conoscenza.

Answer 3

Per rispondere alla domanda nel titolo, è possibile evitare applicando la 'legge di Demetra' e la creazione di un metodo chiamato GetHeadOfParentOrganizationalUnit()

Non sono sicuro se la soluzione si applica per il vostro caso specifico, ma vale la pena dare un'occhiata se è possibile eliminare tutti quei controlli nulli.

Consulta anche: a link

Answer 4

Partenza this article. Presenta una grande soluzione che permette di scrivere cose del genere:

p.With(x => x.OrganisationalUnit) 
.With(x => x.Parent) 
.With(x => x.Head) 
.With(x => x.CurrentAllocation 
.With(x => x.Person); 

Answer 5

È possibile utilizzare la gestione di base eccezioni come pure per la cattura di questo. Non sono pazzo di quella soluzione, ma è un'opzione. Se questi valori nidificati sono normali, le eccezioni probabilmente non sono la risposta corretta:

public class A 
{ 
} 
public class B 
{ 
    public A a; 
} 
public class C 
{ 
    public B b; 
} 
class Program 
{ 
    static A GetA(C c) 
    { 
     A myA; 
     try 
     { 
      myA = c.b.a; 
     } 
     catch 
     { 
      myA = null; 
     } 
     return myA; 
    }   

    static void Main(string[] args) 
    { 
     C theC = new C(); 
     theC.b = new B(); 
     theC.b.a = new A(); 
     A goodA = GetA(theC); 
     if (goodA != null) 
     { 
      Console.WriteLine("Expected nominal path."); 
     } 
     else 
     { 
      Console.WriteLine("Unexpected nominal path."); 
     } 
     theC.b.a = null; 
     A badA = GetA(theC); 
     if (badA == null) 
     { 
      Console.WriteLine("Expected off-nominal path."); 
     } 
     else 
     { 
      Console.WriteLine("Unexpected off-nominal path."); 
     } 

    } 

}