LINQ查询求助：统计含Hip Hop或Pop流派的艺术家数量

嘿，刚接触LINQ就能写出这样的代码已经很棒了！你的思路完全没问题——通过筛选流派属于Hip Hop或Pop的艺术家，再判断是否存在这类艺术家，核心逻辑是正确的。不过我们可以让代码更简洁高效，同时避开一个常见的小坑：

1. 简化逻辑，提升性能 #

你当前用Count() > 0来判断是否存在匹配项，其实LINQ提供了更合适的Any()方法：它会在找到第一个符合条件的元素时就停止遍历，比Count()（需要遍历所有元素）性能更好，代码也更直观：

private bool HasNonRockArtists() 
{
    var genres = new List<string> { "HIP HOP", "POP" };
    // 直接用Any()判断是否存在匹配的艺术家
    return Artists.Any(a => genres.Contains(a.Genre));
}

2. 处理大小写不匹配的潜在问题 #

你的目标流派是大写的"HIP HOP"和"POP"，但如果Artists集合里的Genre值是"Pop"、"hip hop"或者混合大小写的形式，Contains()会因为大小写敏感而匹配失败，导致统计结果不准确。可以统一转换大小写来解决这个问题，同时用HashSet来优化查找性能：

private bool HasNonRockArtists() 
{
    // 用HashSet存储小写的目标流派，查找更快且避免大小写问题
    var targetGenres = new HashSet<string> { "hip hop", "pop" };
    // 将艺术家的流派转换为小写后再判断，同时处理Genre为null的情况
    return Artists.Any(a => targetGenres.Contains(a.Genre?.ToLowerInvariant()));
}

这里的优化点说明： #

• HashSet<string>的Contains操作时间复杂度是O(1)，比List<string>的O(n)更快，尤其是当目标流派数量较多时。
• ToLowerInvariant()确保在不同区域设置下转换结果一致，避免因系统语言导致的大小写转换差异。
• ?.空条件运算符防止a.Genre为null时抛出空引用异常。

3. 关于你原代码的说明 #

其实你原来的代码是可以正常工作的，只是在性能和健壮性上有优化空间。比如blnHasNonRockArtists = nonRockArtistsCount > 0 ? true : false;可以简化为blnHasNonRockArtists = nonRockArtistsCount > 0;，因为比较表达式本身就返回bool值。

内容的提问来源于stack exchange，提问作者Jason Dymond