Linq method body Bonne pratique question

Question

Laquelle de ces deux instructions est la plus rapide/meilleure pratique?

myList.Where(x => 
      { 
       bool itemOne= x.ItemOne == paramItemOne; 
       bool itemTwo = x.ItemTwo == paramItemTwo; 
       return itemOne && itemTwo; 
      }) 


myList.Where(x => x.ItemOne == paramItemOne).Where(x=>x.ItemTwo == paramItemTwo) 

ou sont-ils identiques?

Answer 1

Je ne dirais pas non plus. Le plus rapide va être:

myList.Where(x => x.ItemOne == paramItemOne && x.ItemTwo == paramItemTwo) 

(en supposant que le compilateur/JIT ne pas optimiser les affectations de variables loin dans votre 1ère forme)

La deuxième forme sera plus lente, car il peut impliquer beaucoup plus la méthode invocation sur les délégués fournis aux instructions Where.

Answer 2

Les performances sont identiques ou très proches. Le second pourrait avoir plus de frais généraux de méthode.

Plus important que la performance, je dirais que la première est une meilleure pratique car il est beaucoup plus clair.

Answer 3

Les résultats sont les mêmes, cependant, je vous recommande d'écrire ceci:

myList.Where(x => 
      { 
       return x.ItemOne == paramItemOne && x.ItemTwo == paramItemTwo; 
      }); 

Ceci est garanti pour travailler plus vite, parce que maintenant x.ItemTwo == paramItemTwo ne sera même pas calculé si x.ItemOne == paramItemOne

Answer 4

Le premier vérifie les deux conditions de réponse par élément de la liste. Le second filtre par la première condition, puis filtre le résultat par la deuxième condition.

La seconde est la création et le remplissage d'une autre collection temporaire, mais fait moins de comparaison pour la deuxième condition.