C#对比两个含相同MyObject的列表时IsChanged误判所有元素变更问题

问题根源 #

你当前的实现存在两个核心错误：

1. 变更匹配逻辑错误
   你用copy.Any(x2 => x.IsChanged(x2))的逻辑是只要副本列表中存在任意一个对象和当前对象对比判定为变更，就认为当前对象修改过。但你的实际需求应该是将当前对象和副本列表中同Id的对应对象做对比，而非和任意对象对比。如果列表中存在重复Id的元素，或者副本顺序变化，都会导致误判，甚至出现你遇到的所有对象都被判定为变更的情况。
2. 字符串比较存在隐性风险
   如果业务中允许Name/Description为null，null和空字符串""的直接!=判断会返回true，若你的数据源或深拷贝实现存在null与空字符串自动转换的逻辑（比如部分ORM、序列化组件的默认行为），哪怕你没有手动修改属性，也会被判定为变更。

修复方案 #

第一步：先将副本列表转为Id为键的字典，确保每个原对象都能匹配到唯一对应的副本对象，同时大幅提升对比性能。
第二步：可根据业务需要优化IsChanged的字符串对比逻辑，处理null和空字符串的等值问题。

// 先把副本转为字典，Key为Id，Value为副本对象
var copyDict = copy.ToDictionary(c => c.Id);

// 筛选变更对象：先匹配同Id的副本，再做变更判断
var changed = items.Where(x => 
    copyDict.TryGetValue(x.Id, out var correspondingCopy) 
    && x.IsChanged(correspondingCopy));

如果业务需要把null和空字符串视为相同，可修改IsChanged的字符串对比逻辑：

public static bool IsChanged(this MyObject original, MyObject other)
{
    if (original == null || other == null)
        return false;
    if (original.Id != other.Id)
        return false;
    // 处理null和空字符串等值的情况
    if (!string.Equals(original.Name ?? string.Empty, other.Name ?? string.Empty)
        || !string.Equals(original.Description ?? string.Empty, other.Description ?? string.Empty))
    {
        return true;
    }
    return false;
}

调试建议 #

如果修复后仍有问题，可以在IsChanged方法中加断点，查看未修改的对象判定为变更的具体原因：是Id匹配错误，还是字符串对比出现了非预期的结果，也可以验证你的深拷贝实现是否真的完整复制了所有属性的值。

内容的提问来源于stack exchange，提问作者Ivan-Mark Debono