如何优化EF多对象比较的Any()查询以降低SonarQube认知复杂度

你的代码存在两个核心问题：一是重复的多条件嵌套比较导致认知复杂度超标，触发SonarQube的"Refactor this method to reduce its Cognitive Complexity"提示；二是先调用Any再调用FirstOrDefault，做了两次重复的集合查询，浪费性能。以下是具体优化方案：

1. 提取匹配逻辑到独立方法 #

把多字段的比较逻辑封装成单独方法，既复用代码，又大幅降低主逻辑的认知复杂度。同时利用string.Equals的null安全特性，简化原本的嵌套null判断：

private bool IsMatchingApplicant(Applicant existing, Applicant item)
{
    if (existing.ApplicationId != item.ApplicationId)
        return false;
    if (!string.Equals(existing.FirstName, item.FirstName))
        return false;
    if (!string.Equals(existing.MiddleName, item.MiddleName))
        return false;
    if (!string.Equals(existing.LastName, item.LastName))
        return false;
    if (!string.Equals(existing.CreatedBy, item.CreatedBy))
        return false;
    if (!string.Equals(existing.MobilePhone, item.MobilePhone))
        return false;
    if (!string.Equals(existing.EmailAddress, item.EmailAddress))
        return false;
    
    return true;
}

注：string.Equals原生支持null值比较（两个null会返回true），无需额外嵌套三元判断，直接用该方法就能替代原本复杂的null校验逻辑。

2. 合并重复查询，提升性能 #

原本先通过Any判断存在性，再调用FirstOrDefault获取对象，相当于遍历了两次集合。直接调用FirstOrDefault后判断结果是否为null，只需要一次遍历：

if (applicantsForThisApplication != null)
{
    var existingApplicationSignatory = applicantsForThisApplication.FirstOrDefault(a => IsMatchingApplicant(a, item));
    
    if (existingApplicationSignatory != null)
    {
        // 执行后续业务逻辑...
    }
}

额外优化建议 #

• 如果applicantsForThisApplication是数据库IQueryable，封装的匹配方法在EF Core 3.0+版本中可以被正常解析，不会影响SQL查询生成；
• 可以为Applicant类实现IEquatable<Applicant>接口，把匹配逻辑放到Equals方法中，进一步提升代码规范性。

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