如何移除列表中CanvasId属性存在于另一列表的元素？代码异常求助

嘿，我来帮你搞定这个问题～ 先说说你原来的代码为啥没效果，再给你几个靠谱的方案：

原代码的问题所在 #

1. 修改了副本而非原列表：你写的rawData.ToList().RemoveAt(index)里，ToList()会创建一个rawData的全新副本，你移除的是副本里的元素，原列表rawData根本没被改动，这就是你看不到效果的核心原因。
2. 遍历过程中修改集合风险高：用foreach遍历rawData的同时试图修改它，很容易引发InvalidOperationException（集合已修改，无法枚举），即使没报错，也可能出现遍历遗漏的情况。
3. 效率低下的存在性判断：activitiesOpen.Select(...).Count() > 0这种写法会先遍历整个activitiesOpen生成中间序列，再统计数量，不如直接用Any()来判断是否存在匹配项，而且如果列表很大，性能差异会很明显。

最佳解决方案 #

方案1：用LINQ创建新列表（推荐）

这种方法简洁高效，而且不会修改原列表，避免副作用，是C#里处理这类集合过滤的常规操作：

internal static List<DashboardData> CloseWhereOpen(List<DashboardData> rawData, List<DashboardData> activitiesOpen)
{
    // 先把所有需要排除的CanvasId提取到HashSet，Contains操作是O(1)，比List快N倍
    var excludedCanvasIds = new HashSet<int>(activitiesOpen.Select(ao => ao.CanvasId));
    
    // 过滤出rawData中CanvasId不在排除列表里的元素，返回新列表
    return rawData.Where(item => !excludedCanvasIds.Contains(item.CanvasId)).ToList();
}

为啥用HashSet？因为List.Contains()是O(n)时间复杂度，每次查找都要遍历整个列表，而HashSet.Contains()是O(1)，当activitiesOpen元素较多时，性能提升非常明显。

方案2：直接修改原列表（如果必须的话）

如果你一定要在原列表上修改（而不是返回新列表），可以用倒序遍历的方式，避免索引错位的问题：

internal static List<DashboardData> CloseWhereOpen(List<DashboardData> rawData, List<DashboardData> activitiesOpen)
{
    var excludedCanvasIds = new HashSet<int>(activitiesOpen.Select(ao => ao.CanvasId));
    
    // 倒序遍历，移除元素时不会影响未遍历的元素索引
    for (int i = rawData.Count - 1; i >= 0; i--)
    {
        if (excludedCanvasIds.Contains(rawData[i].CanvasId))
        {
            rawData.RemoveAt(i);
        }
    }
    
    return rawData;
}

倒序遍历的好处是，当你移除第i个元素时，后面的元素（索引比i小的）还没被遍历到，不会因为元素前移导致跳过某个元素。

总结 #

优先用方案1，因为LINQ的写法更简洁、可读性更高，而且函数式的处理方式（不修改输入，返回新结果）能减少很多潜在的bug。如果业务场景要求必须修改原列表，再用方案2。

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