正确操作IEnumerable <foo> </foo>

时间:2015-02-17 22:51:52

标签: c# multithreading locking concatenation ienumerable

关注我之前的question

在多线程程序中,不同的线程每个都会生成很长的结果列表。当线程完成它的任务时,我想将连接不同的列表放到一个列表中。请注意以下几点:

public struct AccEntry
{
    internal AccEntry(int accumulation)
        : this()
    {
        Accumulation = accumulation;
    }
    public int Accumulation { private set; get; }
}


internal class Functions
{
     internal Functions(Object lockOnMe, IEnumerable<AccEntry> results)
     {
          _lockOnMe = lockOnMe;
          _results = results;
          _result = new List<AccEntry>();
     }

     private IEnumerable<AccEntry> _results { set; get; }
     private List<AccEntry> _result { set; get; }

     internal void SomeFunction()
     {
          /// some time consuming process that builds _result
          lock(_lockOnMe)
          {
              /// The problem is here! _results is always null.
              if (_results == null) _results = _result;
              else _results = _results.Concat(_result);
          }
     }
}

public class ParentClass
{
     public void DoJob()
     {
          IEnumerable<AccEntry> results = null;
          /// initialize and launch multiple threads where each 
          /// has a new instance of Functions, and call SomeFunction.
     }
}

问题,如代码中所述,_results总是 null 。当线程更改将其设置为 _result 时,另一个线程再次发现它 null 。我还尝试在函数构造函数中使用 ref 关键字来获取结果,但它没有改变任何东西。

假设以下按预期执行,我想知道上面提到的代码可能会出现什么问题?!!

List<int> listA = new List<int>();
List<int> listB = new List<int>();
listA.Add(10);
listB.Add(12);
IEnumerable<int> listC = null;
listC = listA;
listC = listC.Concat(listB);

1 个答案:

答案 0 :(得分:1)

在连接项目并将其分配回_results变量时,将替换分配给变量的原始值。新的项目集合将是该实例的本地项目。

使用可以添加项目的IEnumerable<>,而不是使用必须替换的List<>进行更新:

internal class Functions
{
     internal Functions(Object lockOnMe, List<AccEntry> results)
     {
          _lockOnMe = lockOnMe;
          _results = results;
          _result = new List<AccEntry>();
     }

     private object _lockOnMe;
     private List<AccEntry> _results;
     private List<AccEntry> _result;

     internal void SomeFunction()
     {
          /// some time consuming process that builds _result
          lock(_lockOnMe)
          {
              _results.AddRange(_result);
          }
     }
}

只需确保在创建Functions个实例之前创建列表。