foreachの代わりにラムダ式を使用するのは良い方法ですか？

Question

今日、私はTextToSpeechアプリを開発していました。ユーザーが選択した音声がコンピュータにインストールされているかどうかを確認する必要があります。このため

、私はforeachの使用することができ、次のいずれか

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

または、ラムダ式：installedVoices

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

をSpeechSynthesizerからReadOnlyCollection < InstalledVoice>です。

確実に、ラムダ式はforeachよりもきれいに見えますが、どちらが優れていますか？

私がforeachをテストしたところからは、のようにわずかにがラムダ式より速いようです。

また、InstalledVoiceに即座に対応する必要がある場合は、foreachとlambdaの両方を将来拡張することができます。

Answer 1

ラムダ式はforeachよりもきれいに見えますが、どちらが優れていますか？

「Looks cleaner」は、より良いプラクティスの完全な指標です。意味のある方法で、使用する言語機能に応じてパフォーマンスが変化することは非常にまれです。したがって、コードの特定の構造がどれだけ優れているかの可読性が最も重要な指標です。読みやすさは、コードの唯一の読者があなたであっても、コードの保守性を最終的に決定するものです。

これはケースバイケースの決定です.1つの状況でlambdasが改善され、別の状況ではループが改善される可能性があるためです。

Answer 2

はい、この場合はLINQを使用することをお勧めします。これは読みやすいためです。しかし、私にはinstalledVoices.Anyの代わりにinstalledVoices.FirstOrDefaultが読めるようになります。