Comment éviter de multiples si les contrôles null

Question

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

je dois faire une recherche dans un modèle d'objet profond comme celui-ci:

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

est là de toute façon pour évaluer cela et retourner null si o f la chaîne est nulle (OrganizationalUnit, parent, tête, etc.), sans avoir à faire un

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

Answer 1

Vous recherchez l'opérateur déréférence NULL-safe ?. (également connu sous le nom de navigation en toute sécurité) que certaines langues (par exemple Groovy), mais malheureusement, C# n'a pas cet opérateur.

Espérons qu'il sera mis en application un ....

jours

Voir aussi this post par Eric Lippert. La syntaxe qu'il propose est .?.

Answer 2

Avez-vous entendu parler du Law of Demeter?

Le chaînage de telles séquences d'appels n'est pas une bonne idée. Cela crée des dépendances terribles entre les classes dont vous n'avez pas besoin. Dans votre exemple, la classe contenant p devient dépendante de cinq autres classes. Je vous suggère de simplifier votre code et de faire en sorte que chaque classe vérifie les nullités à un seul niveau, dans leur propre contexte de connaissances.

Answer 3

Pour répondre à la question dans votre titre, vous pouvez éviter en appliquant la « loi de Déméter » et la création d'une méthode appelée GetHeadOfParentOrganizationalUnit()

Je ne sais pas si la solution applique pour votre cas spécifique, mais cela vaut le coup d'oeil si vous pouvez éliminer toutes ces vérifications nuls.

Voir aussi: a link

Answer 4

Check out this article. Il présente une excellente solution qui vous permet d'écrire des choses comme ça:

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

Answer 5

Vous pouvez utiliser la gestion des exceptions de base, ainsi que pour attraper. Je ne suis pas fou de cette solution, mais c'est une option. Si ces valeurs nulles imbriquées sont normales, les exceptions ne sont probablement pas la bonne réponse:

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."); 
     } 

    } 

}