List<T>.RemoveAll方法在自定义对象List中未生效的原因及修复方案咨询

问题原因 #

问题核心在于引用类型的默认相等性判断逻辑：Member是自定义引用类型，默认情况下Contains方法比较的是对象的内存引用地址，而非属性值。你在toRemove中创建的是全新的Member实例（new Member { member = m }），这些实例和members列表里的原实例并非同一对象，因此toRemove.Contains(m)永远返回false，导致RemoveAll没有执行任何删除操作。

修复方案 #

方案1：直接比较字符串（最简洁高效）

无需创建新的Member实例，直接提取要删除的邮箱字符串集合，在RemoveAll中比较属性值：

public static List<Member> Remove()
{
    var specialMembers = new List<SpecialMember>();
    specialMembers.Add(new SpecialMember { Upn = "a@b.com" });

    var members = new List<Member>();
    members.Add(new Member { member = "a@b.com" });
    members.Add(new Member { member = "c@d.com" });

    // 提取要删除的邮箱集合，用HashSet提升查找效率
    var toRemoveEmails = members.Select(f => f.member)
                                .Intersect(specialMembers.Select(s => s.Upn))
                                .ToHashSet();

    members.RemoveAll(m => toRemoveEmails.Contains(m.member));

    return members;
}

使用HashSet而非List是因为HashSet.Contains的时间复杂度为O(1)，远优于List的O(n)，数据量大时性能差距明显。

方案2：重写Member的Equals和GetHashCode

让Member类型按member属性值进行相等性判断：

public class Member
{
    public string member { get; set; }

    public override bool Equals(object obj)
    {
        return obj is Member other && member == other.member;
    }

    public override int GetHashCode()
    {
        return member?.GetHashCode() ?? 0;
    }
}

修改后，原代码中的toRemove.Contains(m)会自动按member属性值比较，RemoveAll即可正常执行删除逻辑。

方案3：用Any替代Contains做属性匹配

无需修改Member类，直接在RemoveAll中通过Any方法判断属性是否一致：

members.RemoveAll(m => toRemove.Any(r => r.member == m.member));

这种方式适合临时场景，无需额外定义相等性逻辑。

内容的提问来源于stack exchange，提问作者Tech with Thiru