Ist es empfehlenswert, Lambda-Ausdrücke anstelle von foreach zu verwenden?

Question

Heute habe ich an einer TextToSpeech App gearbeitet und ich bin auf eine Situation gestoßen, in der ich prüfen muss, ob die ausgewählte Stimme des Benutzers auf dem Computer installiert ist.

Dafür konnte ich entweder einen foreach:

bool foundVoice = false; 
foreach (var v in installedVoices) 
{ 
    if (v.VoiceInfo.Name.Contains(selectedVoice) && v.VoiceInfo.Culture.ToString() == selectedCulture) 
    { 
     foundVoice = true; 
     break; 
    } 
} 

Oder eine lamda Ausdruck:

var foundVoice = installedVoices.FirstOrDefault(v => v.VoiceInfo.Name.Contains(selectedVoice) && v.VoiceInfo.Culture.ToString() == selectedCulture); 

Die installedVoices ist ein Readonlycollection < InstalledVoice> von Speechsynthesizer.

Definitiv sieht der Lambda-Ausdruck sauberer aus als der foreach, aber welcher ist besser?

Von was ich getestet habe scheint die foreach leicht schneller als der Lambda-Ausdruck zu sein.

Außerdem können sowohl foreach als auch lambda in der Zukunft erweitert werden, wenn sofortige Maßnahmen auf InstalledVoice erforderlich sind.

Answer 1

der Lambda-Ausdruck sieht sauberer aus als der foreach, aber welcher ist besser?

"Sieht sauberer" ist ein perfekter Indikator für eine bessere Praxis. Es ist äußerst selten, dass sich die Leistung in Abhängigkeit von der verwendeten Sprachfunktion in einer sinnvollen Weise ändert. Daher ist die Lesbarkeit das wichtigste Maß dafür, wie gut ein bestimmtes Konstrukt für Ihren Code ist. Lesbarkeit entscheidet letztendlich über die Wartbarkeit Ihres Codes, auch wenn Sie nur der Leser Ihres Codes sind.

Beachten Sie, dass dies eine fallweise Entscheidung ist, da Lambdas in einer Situation besser sein können und Schleifen in einer anderen Situation besser sein könnten.

Answer 2

Ja, es ist gute Praxis, LINQ in diesem Fall zu verwenden, weil es besser lesbar ist. Allerdings wäre für mich installedVoices.Any statt installedVoices.FirstOrDefault noch besser lesbar.