approche LINQ à ce code

Question

J'ai actuellement une méthode qui boucle dans une liste d'objets métier (propriété Propriétés) pour tester si la propriété SerialNumber est un numéro de série ou non. Si je trouve un numéro de série, je quitte la boucle et renvoie true, sinon je renvoie false.

code

est la suivante:

public bool HasSerialNumber() 
    { 
     if (this.Properties != null && this.Properties.Count > 0) 
     { 
      foreach (var property in Properties) 
      { 
       if (!string.IsNullOrEmpty(property.SerialNumber)) 
        return true; 
      } 
     } 
     return false; 
    } 

Y at-il une meilleure approche LINQ à cela?

Je donne les résultats suivants à l'esprit:

return Properties.Where(x => !string.IsNullOrEmpty(x.SerialNumber)).ToList().Count > 0; 

Y at-il une meilleure/méthode plus rapide pour le contrôle de chaîne non vide?

Answer 1

Vous pouvez utiliser Any au lieu de vérifier si le nombre est supérieur à zéro.

return Properties.Any(x => !string.IsNullOrEmpty(x.SerialNumber)) 

et bien sûr votre Properties.Count > 0 vérification est redondant.

Answer 2

Cela devrait faire l'affaire:

return Properties.Any(x => !string.IsNullOrEmpty(x.SerialNumber)); 

Answer 3

Check out IEnumerable<T>.Any():

public bool HasSerialNumber() 
{ 
    if(this.Properties != null) 
     return Properties.Any(p => !string.IsNullOrEmpty(p.SerialNumer)); 
    return false; 
} 

Answer 4

Je ne pense pas que vous allez améliorer particulièrement sur les performances de string.IsNullOrEmpty(), mais un piège vous devriez être éviter est les 2 derniers appels sur votre requête - spécifiquement ToList() et Count(). Ce que vous faites ici, c'est de parcourir chaque élément, de le convertir en une liste (créer une liste et ajouter des éléments dans le processus, puis parcourir tous les éléments de la liste pour en compter le nombre - tout pour vérifier . si une seule valeur est vide

vous pouvez utiliser la méthode Any pour trouver si un seul élément correspond à certains critères, comme suit:

return Properties.Any(x => !string.IsNullOrEmpty(x.SerialNumber));