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;
    }
}

或 lamda表达：

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

installedVoices是 ReadOnlyCollection＆lt;来自 SpeechSynthesizer 的InstalledVoice＆gt; 。

当然，lambda表达式看起来比foreach更干净，但哪一种更好用？

根据我的测试结果，foreach似乎比lambda表达式略微快。

此外，如果需要立即对InstalledVoice采取行动，将来可以扩展foreach和lambda。

Answer 1

lambda表达式看起来比foreach更干净但是哪种更好的做法？

“看起来更干净”是更好实践的完美指标。根据所使用的语言设施，以极有意义的方式更改性能极为罕见，因此可读性是衡量特定构造对代码有多好的最重要指标。可读性最终决定了代码的可维护性，即使代码的唯一读者是您。

请注意，这是一个逐案判断，因为在一种情况下lambdas可能更好，而在另一种情况下循环可能更好。

Answer 2

是的，在这种情况下使用LINQ是一种很好的做法，因为它更具可读性。但是，对我来说installedVoices.Any而不是installedVoices.FirstOrDefault会更具可读性。

使用lambda表达式而不是foreach是一种好习惯吗？

2 个答案: