Ошибки с Foreach цикл по списку

Question

У меня есть следующий фрагмент кода, который дает мне проблемы, и я был бы признателен за любую помощь:

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(); 
} 

Моя первоначальная проблема заключалась в том, что я получал System.InvalidOperationException: сбор была изменена ошибка в моем цикле foreach.

1) Я все еще не могу понять, почему я получил бы эту ошибку, потому что я не вижу, чтобы она была изменена.

Кто-то предложил мне скопировать список в новый список и пропустить новый список. Я сделал это и избавился от InvalidOperationException. Тем не менее, я попытался переписать список 2 разными способами, и оба дали мне System.ArgumentException: массив назначения был недостаточно длинным. Вот два пути я пытался скопировать список

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

и

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

Оба эти дали мне System.ArgumentException: массив назначения не было достаточно долго.

2) Почему любой из этих методов дал мне это исключение?

Любая помощь будет оценена, потому что я в тупике.

Спасибо!

Answer 1

Похоже, что объект списка, на который ссылается Options, меняется на другой поток. Это объясняет обе проблемы.

1. Очевидно, что если другой поток изменяет список в то время как вы итерацию, вы увидели бы «коллекция была изменена» исключение, так как вы не можете изменить список, а его элементы быть перечислены.
2. В последнем случае это звучит как конструктор, а AddRange выделяет массив, а затем заполняет его из поставляемого перечислимого. Если список вырос между распределением, и когда он будет заполнен новым массивом, внезапно выделенное хранилище не будет достаточно большим, и вы увидите это поведение.

Вам необходимо создать механизм блокировки, чтобы этот код мог блокировать любые потоки, которые могут записываться в список, когда он либо обрабатывает элементы, либо создает копию для последующей обработки.

Answer 2

Убедитесь, что никакие другие нити не меняют коллекцию во время ее повторения.

Кроме того, еще один способ сделать это будет:

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)); 
    } 
} 

Но, конечно, если бы мы имели какую-либо информацию о других потоках возни с этой коллекцией, это было бы проще, чтобы обеспечить лучшее потокобезопасное решение.

Answer 3

Хорошо, я решил решить проблему. Я поместил блокировку (в список опций) вокруг вызова метода. Это устранило все мои проблемы.

Ценит всех!