2

在我朋友的代码中,他有一个List

 List<int> listOfIds = new List<int>();

然后我用来AddRange()向它添加一个 int 集合:

   listOfIds.AddRange(this._employeeList
                       .Where(r => r.EmployeeID != null)
                       .Select(r => r.EmployeeID != null ? r.EmployeeID.Value : 0));

但是,在日志中,它说:

 System.IndexOutOfRangeException: Index was outside the bounds of the array.
   at System.Collections.Generic.List`1.Enumerator.MoveNext()
   at System.Linq.Enumerable.WhereSelectListIterator`2.MoveNext()
   at System.Collections.Generic.List`1.InsertRange(Int32 index, IEnumerable`1 collection)
   at the method which called the codes above.....

我不太确定为什么会这样。我很确定这个错误发生在上面的代码上,但我似乎无法理解为什么IndexOutofRangeException会出现。

你能帮我指出是什么原因造成的吗?

更新:

我错了。我很抱歉。该方法不使用多线程。但是,另一种调用此方法的方法是使用 Parallel.Foreach,这意味着多线程。可能当 _employeeList 被用作 AddRange() 的源时,另一个线程也在修改它。因此,addrange() 不是线程安全的答案是合理的。十分感谢大家。

4

1 回答 1

2

很难说出更具体的想法,但这可能会奏效。

AddRange不是线程安全的。尝试使用lock primitives添加许多项目。宣布:

private static object mutex = new object();

用法

lock(mutex)
{
    listOfIds.AddRange( /* code goes here */ )
}

反编译后AddRange发现

public void AddRange(IEnumerable<T> collection)
{
  this.InsertRange(this._size, collection);
}

的内容InsertRange,您可以在堆栈跟踪中看到:

  ...
  ICollection<T> collection1 = collection as ICollection<T>;
  if (collection1 != null)
  {
     ...
  }
  else
  {
    foreach (T obj in collection)
      this.Insert(index++, obj);
  }

换句话说 - 它遍历序列并尝试使用递增的索引逐项添加。这不应该出错,不是吗?反编译的845行。所以我不相信堆栈跟踪,除非我会看到整个堆栈跟踪。List.csdotPeek

于 2013-01-10T15:13:10.237 回答