2

我有以下给我带来问题的代码，我将不胜感激：

private static string CreateOptionString(List<VehicleOption> Options)
{
    StringBuilder returnValue = new StringBuilder();
    foreach (VehicleOption option in Options)
    {
        if (option.OptionStatus == ExtendedResponse.OptionState.Included)
        {
            if (returnValue.Length > 0)
            {
                returnValue.Append(", ");
            }
            returnValue.Append(option.OptionName);
        }
    }
    return returnValue.ToString();
}

我最初的问题是我在 foreach 循环中收到 System.InvalidOperationException: collection was modified 错误。

1）我仍然无法弄清楚为什么我会收到这个错误，因为我没有看到它被修改的任何地方。

有人建议我将列表复制到新列表并循环遍历新列表。我这样做了，它摆脱了 InvalidOperationException。但是，我尝试了 2 种不同的方式来处理列表，并且都给了我一个 System.ArgumentException：目标数组不够长。这是我尝试复制列表的两种方法

List<VehicleOption> newOptions = new List<VehicleOption>(Options);

和

List<VehicleOption> newOptions = new List<VehicleOption>();
newOptions.AddRange(Options);

这两个都给了我一个 System.ArgumentException：目标数组不够长。

2）为什么这些方法中的任何一种都会给我这个例外？

任何帮助将不胜感激，因为我很难过。

谢谢！

4

3 回答 3

5

确保在迭代时没有其他线程正在更改集合。

此外，另一种方法是：

return string.Join(", ", options.Where(
    op => op.OptionStatus == ExtendedResponse.OptionState.Included));

在这种情况下，这比使用 StringBuilder 更好并且（令人惊讶地）更快。

尽管如此，这并不能解决您的问题 - 这可能是由更改集合的不同线程引起的。

我的第一次尝试是：

private static string CreateOptionString(List<VehicleOption> Options)
{
    lock (Options)
    {
        return string.Join(", ", options.Where(
            op => op.OptionStatus == ExtendedResponse.OptionState.Included));
    }
}

但是，当然，如果我们有任何关于其他线程与该集合混在一起的信息，那么提供更好的线程安全解决方案会更容易。

于 2012-06-12T22:46:18.800 回答
4

听起来好像引用的列表对象Options正在被另一个线程更改。这将解释这两个问题。

  1. 显然，如果另一个线程在您迭代列表时更改了它，您会看到“集合已修改”异常，因为在枚举其元素时您无法更改列表。
  2. 在后一种情况下，它听起来像构造函数并且AddRange正在分配数组，然后从提供的枚举中填充它。如果列表在分配和枚举以填充新数组之间增长，突然分配的存储空间不够大，您会看到这种行为。

您需要设计一些锁定机制，以便此代码可以在处理元素或制作副本以供以后处理时阻塞可能正在写入列表的任何线程。

于 2012-06-12T22:46:06.087 回答
0

好的，我最终解决了这个问题。我在对方法的调用周围加了一个锁（在选项列表上）。这解决了我所有的问题。

感谢大家！

于 2012-06-14T16:27:22.740 回答