Éviter l'exposition à NullReferenceException lorsque vous utilisez LINQ pour XML

Question

J'utilise une chaîne d'appels à la méthode XElement.Element() de forer vers le bas dans un document XML et tirer la valeur d'un attribut:

XElement root = ...; 
XNamespace ns = "..."; 
var firstName = 
    root 
    .Element(ns + "Employee") 
    .Element(ns + "Identity") 
    .Element(ns + "Name") 
    .Attribute(ns + "FirstName"); 

Cependant, depuis la le document entrant n'a pas été validé par le schéma, il est possible qu'un document malformé provoque un NullReferenceException si l'un des éléments intermédiaires attendus n'existe pas.

Existe-t-il un moyen d'éviter ce risque, tout en gardant le code succinct?

Je peux envelopper le code ci-dessus dans un gestionnaire pour NullReferenceException mais cela ne semble pas correct et n'indiquerait pas non plus spécifiquement où la panne s'est produite. La construction d'un message d'erreur informatif serait manuelle, fastidieuse, sujette aux erreurs et risque de maintenance. Au lieu de cela, est-ce que je devrais utiliser XPath à la place, de cette façon je peux vérifier un retour null, puis construire facilement un message d'erreur indiquant que l'expression XPath n'a pas pu être résolue?

Answer 1

Une option consiste à utiliser Elements() au lieu de Element() - qui donnera une séquence vide si l'élément n'est pas trouvé. En utilisant les méthodes d'extension dans Extensions, vous pouvez passer d'une séquence d'éléments à une séquence d'éléments - et la même chose pour les attributs. Alors:

var firstName = 
    root 
    .Elements(ns + "Employee") 
    .Elements(ns + "Identity") 
    .Elements(ns + "Name") 
    .Attributes(ns + "FirstName") 
    .FirstOrDefault(); 

Il est une différence entre les deux extraits, l'esprit que vous - ce trouverez attribut le premier correspondant, même si elle vient de (par exemple) le premier élément de nom dans le troisième élément d'identité au sein de la seconde Élément employé. Cela peut ou peut ne pas être un problème pour vous.

(Juste pour vérifier, vous avez certainement besoin l'espace de noms sur l'attribut? Contrairement éléments, les attributs ne héritent pas un espace de noms « par défaut ». Il est plus rare d'utiliser des attributs sur les espaces de noms que sur les éléments.)

Answer 2

Vous pouvez définir la méthode en extension pour recouvrir la détection de null. Quelque chose comme ça, par exemple:

public static class XmlExtender 
{ 
    public static XAttribute AttributeOrDefault(this XElement el, XName name, string defValue) 
    { 
     var v = el.Attribute(name); 
     return v == null ? new XAttribute(name, defValue) : v; 
    } 

    public static string AttributeValue(this XElement el, XName name, string defValue) 
    { 
     var v = el.Attribute(name); 
     return v == null ? defValue : v.Value; 
    } 
} 

qui peut alors être utilisé comme ceci:

var firstName = root.ELement("elname") 
        .AttributeOrDefault("attrName", "N/A").Value; 

ou ceci:

var firstName = root.Element("elname") 
        .AttributeValue("attrName", "N/A");