在C#中打印从BlockingCollection获取的数据时,Foreach抛出错误

时间:2015-04-20 06:00:23

标签: c# multithreading blockingcollection

我遇到了多线程的绊脚石。我想我知道问题是什么,但无法确定如何解决它。但我可能错了。

总之,我有生产者和消费者线程。生产者线程将来自外部源的数据收集到数据表中,然后将它们放入集合中。然后,消费者从集合中获取数据表。我使用BlockingCollection作为公共静态集合,因此两个线程都可以访问它,它存在于两个不同的类中。我现在将展示代码的主要部分,然后解释什么是和不起作用。

制片人主题:

try
{
     dataTable.Clear();
     adapter.Fill(dataTable);
     dataCaptured = true;
     timeout = 0;
     ThreadInfo.setCurrentDate(startDate);
     ThreadInfo.dataTableCollection.Add(dataTable);
}

消费者线程

while(true)
{
     DataTable testTable = ThreadInfo.dataTableCollection.Take();
     foreach (DataRow datarow in testTable.Rows)
     {
          foreach (var item in datarow.ItemArray)
          {
                Console.WriteLine(item);
          }
     }
}

所以我的测试表明,当生产者线程创建数据表时,它会成功地将它们添加到集合中。我可以通过在add方法之前和之后使用count来看到这一点。计算每个表中的行数我还可以确认添加的表是与创建的表相同。此外,take方法还成功删除了一个表,该表与输入的表匹配。我知道这一点既可以计算集合中的表数,也可以计算“'”中的行数。数据表。

我的问题是当我尝试运行foreach循环以打印出结果时。最初它工作并开始将数据打印到屏幕,但随后抛出此错误:

System.InvalidOperationException was unhandled
  HResult=-2146233079
  Message=Collection was modified; enumeration operation might not execute.
  Source=System.Data
  StackTrace:
       at System.Data.RBTree`1.RBTreeEnumerator.MoveNext()
       at pullPlexTable.InputThreads.dataConsumerThread() in \\srv-file01\users$\dkb\Visual Studio 2013\Projects\pullPlexTable\pullPlexTable\InputThread.cs:line 39
       at System.Threading.ThreadHelper.ThreadStart_Context(Object state)
       at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state, Boolean preserveSyncCtx)
       at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state, Boolean preserveSyncCtx)
       at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state)
       at System.Threading.ThreadHelper.ThreadStart()
  InnerException:

我相信这似乎发生在将新数据表添加到集合中。我相信这是由错误支持的,该错误表明.movenext()遇到错误,这意味着它试图移动到集合中的下一个值?

但我真的不知道该做什么,即使我是对的。我试过复制数据表,但仍然遇到错误。我原以为一旦它创建了一个数据表并使用该方法中的take方法复制了数据表,我就可以随心所欲地迭代它。我的预感是数据表仍然指向集合中的数据表,当它在新的数据表进入时被撞到它时会引发错误。但这是完全的推测,可能完全错误。有人可以帮忙吗?

如果您需要更多信息,我很乐意发布。

1 个答案:

答案 0 :(得分:5)

您需要每次都创建一个新的DataTable - 它们是可变的,并通过引用传递。

到目前为止,您所做的是使引用本身是线程安全的。但这还不够 - 所有线程仍然共享DataTable的相同实例。

相反,制作人可能看起来像这样:

 var dt = new DataTable();
 adapter.Fill(dt);

 ThreadInfo.dataTableCollection.Add(dt);

我已经删除了你所拥有的其他东西,因为它很可能也是错误的 - 你在线程中共享的任何资源都必须是线程安全的或同步的。最简单的方法是确保您只能在lock内读取和写入它们,明确地同步访问:

private static object syncObject = new object();

private static DateTime currentDate;
public static DateTime CurrentDate
{
  get { lock (syncObject) return currentDate; }
  set { lock (syncObject) currentDate = value; }
}

这些只是非常基础。你真的不想在多线程环境中猜测。多线程 hard 。至少,我建议通过辉煌的http://www.albahari.com/threading/ - 它会教你基本的概念。谦虚:D

确保安全多线程的最简单方法是确保永远不会在线程之间传递任何可变对象 - 当然,使用static s确保这一点非常困难;一个好的指导原则是任何public static成员默认都必须是线程安全的。如果您只使用给定(不可变或非共享)参数运行新的Task并使用返回值,则可以更轻松地实现多线程。

对于生产者 - 消费者队列,请确保您传递的是不可变数据,或者您不会重复使用的内容(例如“新数据表,填充,传递,忘记”方法)。如果您需要BlockingCollection本身未提供的任何信令,请确保它是线程安全的。理想情况下,您希望使用尽可能高级的结构 - CancellationTokenTaskManualResetEvent等 - 请参阅上面的链接。如果你使用按值类型,它会给你省去很多麻烦(一直向下 - 在结构中包装一个引用显然不会对你有所帮助),但即使这样,你也需要锁定读取和写入。