ASP.NET 6.1中Distinct与IEqualityComparer无法正常工作的问题

看起来你遇到的核心问题是自定义比较器根本没被执行——这就是为什么调试时没感觉到性能变化、重复条目也没被过滤的原因。我帮你梳理下关键问题点和修复方案：

1. 先确认你的比较器实现是否正确 #

你需要确保自定义比较器严格实现IEqualityComparer<T>，并且Equals和GetHashCode的逻辑完全匹配你的需求（邮箱+电话都一致才视为重复）。这里给你一个标准实现示例，还处理了电话格式差异（比如带括号、空格的情况）：

public class ContactComparer : IEqualityComparer<Contact>
{
    // 判定两个Contact是否重复：邮箱忽略大小写，电话标准化后匹配
    public bool Equals(Contact x, Contact y)
    {
        if (ReferenceEquals(x, y)) return true;
        if (x is null || y is null) return false;

        bool emailsMatch = string.Equals(x.Email, y.Email, StringComparison.OrdinalIgnoreCase);
        bool phonesMatch = NormalizePhoneNumber(x.Phone) == NormalizePhoneNumber(y.Phone);

        return emailsMatch && phonesMatch;
    }

    // 生成哈希值：必须基于Equals中用到的字段，否则Distinct会出错
    public int GetHashCode(Contact obj)
    {
        if (obj is null) return 0;

        int emailHash = obj.Email?.ToLowerInvariant().GetHashCode() ?? 0;
        int phoneHash = NormalizePhoneNumber(obj.Phone)?.GetHashCode() ?? 0;

        // 组合两个哈希值，降低冲突概率
        return emailHash ^ (phoneHash * 17);
    }

    // 标准化电话号码：移除所有非数字字符，统一格式
    private string NormalizePhoneNumber(string phone)
    {
        if (string.IsNullOrWhiteSpace(phone)) return null;
        return new string(phone.Where(char.IsDigit).ToArray());
    }
}

2. 正确调用Distinct方法 #

这是最容易踩坑的地方：必须显式传入你的比较器实例，否则Linq会使用默认的引用相等比较（对于引用类型），完全不会触发你的自定义逻辑。

错误的调用方式（不会生效）：

// 错误：用了默认比较器，只会比较对象引用
var uniqueContacts = contacts.Distinct().ToList();

正确的调用方式：

// 正确：传入自定义比较器实例
var uniqueContacts = contacts.Distinct(new ContactComparer()).ToList();

3. 调试验证技巧 #

你可以在Equals和GetHashCode方法里加断点，或者插入Console.WriteLine日志，运行时看这些方法是否被调用。如果没被调用，说明：

• 你没正确传入比较器
• 集合本身是空的
• 如果是EF/Linq to SQL场景：自定义比较器无法被转译为SQL，需要先把数据加载到内存（比如先调用ToList()）再执行Distinct

额外性能优化建议 #

如果你的数据量很大，Distinct加自定义比较器确实会有性能开销（因为要逐个比较），这时候可以考虑先按邮箱+标准化电话分组，再取每组的第一条：

var uniqueContacts = contacts
    .GroupBy(c => new { 
        Email = c.Email?.ToLowerInvariant(), 
        Phone = NormalizePhoneNumber(c.Phone) 
    })
    .Select(g => g.First())
    .ToList();

这种方式的哈希计算逻辑更直接，有时候性能会比Distinct更稳定。

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