如何解决ComboBox导入XML数据后出现重复内容的问题

你的问题出在两个核心点：现有去重逻辑存在错误，且没有在数据源层面提前处理重复值，反而绕了弯路。咱们一步步理清楚：

先说说你现有代码的问题 #

1. 无效的判断逻辑：if (comboBox2.Contains(itemTar.ToString()))完全起不到作用，因为itemTar是一个LINQ查询对象，ToString()返回的是它的类型字符串（比如System.Linq.Enumerable+WhereSelectEnumerableIterator...），不是具体的donem值，所以这个判断永远不会成立。
2. 嵌套循环的索引陷阱：你写的双层循环从0开始遍历，当删除Items[j]时，ComboBox的Items集合会收缩，后面的元素会往前移动，导致后续的索引错乱——比如删除j位置的元素后，原来j+1的元素现在在j位置，但循环j会继续递增，直接跳过了这个元素；而且当i=j时，你会把当前元素删掉，这完全是不合理的。

最简洁高效的解决方案：在数据源层面去重 #

与其先把所有重复值加载到ComboBox再删除，不如直接在读取XML的时候就获取唯一值，用LINQ的Distinct()方法就能轻松实现：

// 读取所有donem元素的值，并去重得到唯一列表
var uniqueDonemValues = xmlDocument.Descendants("donem")
                                  .Select(key => key.Value)
                                  .Distinct()
                                  .ToList();

// 直接绑定到ComboBox
comboBox2.DataSource = uniqueDonemValues;

这样一来，你得到的就是没有重复的日期区间列表，直接绑定到ComboBox即可，完全不需要后续的去重操作，逻辑清晰还高效。

如果你必须先加载所有元素再去重（不推荐） #

如果因为某些特殊场景，你需要先把所有元素添加到ComboBox再处理重复，那一定要从后往前遍历，避免索引错乱的问题：

// 先加载所有元素
var allDonemValues = xmlDocument.Descendants("donem")
                                .Select(key => key.Value)
                                .ToList();
comboBox2.DataSource = allDonemValues;

// 从后往前遍历去重
for (int i = comboBox2.Items.Count - 1; i >= 0; i--)
{
    for (int j = i - 1; j >= 0; j--)
    {
        // 注意要转成字符串比较，避免引用类型的比较问题
        if (comboBox2.Items[i].ToString() == comboBox2.Items[j].ToString())
        {
            comboBox2.Items.RemoveAt(j);
        }
    }
}

这种方式从后往前删，删除前面的元素不会影响后面未遍历的元素索引，能保证所有重复项都被正确清理，但相比第一种方法还是麻烦很多，所以优先推荐数据源层面去重的方案。

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