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

我在尝试多线程时遇到了一个绊脚石。我想我知道问题是什么，但无法决定如何解决。但我可能错了

总之，我有生产者线程和消费者线程。生产者线程从外部源将数据收集到数据表中，然后将它们放入集合中。然后使用者从集合中获取数据表。我将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方法还成功删除了一个表，该表与输入的表匹配。我通过计算集合中的表数和计算“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:

我相信这似乎是在向集合中添加新的datatable时发生的。我相信这一点得到了错误的支持，该错误表明.movenext（）遇到了错误，这意味着它试图移动到集合中的下一个值

但我真的不知道该怎么做，甚至不知道我是否是对的。我尝试过复制datatable，但仍然遇到了一些问题和错误。我本以为，一旦它创建了一个datatable并使用集合中的take方法复制了datatable，我就可以随意地对它进行迭代了。我的直觉是，datatable仍然指向集合中的datatable，当新datatable进入时，它会抛出错误。但这完全是猜测，可能是完全错误的。有人能帮忙吗

---

如果你需要更多的信息，我很乐意发布

每次都需要创建一个新的

DataTable

——它们是可变的，并且通过引用传递

到目前为止，您已经完成了使引用本身线程安全的工作。但这还不够——所有线程仍然共享同一个

DataTable

实例

相反，制作人可能看起来像这样：

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

 ThreadInfo.dataTableCollection.Add(dt);

我已经删除了您周围的其他内容，因为这很可能也是错误的-您在跨线程共享的任何资源都必须是线程安全的或同步的。最简单的方法是确保您只能在

锁

内读写它们，并显式同步访问：

private static object syncObject = new object();

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

这些只是最基本的。您真的不想在多线程环境中四处猜测。多线程是很难的。至少，我建议你经历一下精彩的课程，它会教你一些基本的概念。和谦逊：D

确保安全多线程的最简单方法是确保在线程之间不传递任何可变对象-当然，使用

static

s很难确保这一点；一个好的指导原则是，默认情况下，任何

公共静态

成员都必须是线程安全的。如果您只使用给定（不可变或非共享）参数运行新的

任务

，并使用返回值，那么多线程就容易多了

---

对于生产者-消费者队列，确保传递的是不可变的数据，或者是不打算重用的数据（例如，“新数据表，填充它，传递它，忘记它”方法）。如果需要

BlockingCollection

本身未提供的任何信号，请确保它是线程安全的。理想情况下，您希望尽可能使用高级构造-

CancellationToken

、

Task

、

ManualResetEvent

等-请参阅上面的链接。如果按值类型使用，这将为您节省很多麻烦（一直到最后——显然，将引用包装到结构中对您没有帮助），但即使这样，您也需要锁定读写操作。

相关：您是否在生产者线程中重用DataTable对象？显示从生产者线程创建数据表的更多代码。我假设问题是在行上迭代，因为在一个线程中，你有清除和填充，在另一个线程中，你有行上的循环。Soner我确实看到了，但大多数评论都与他在foreach循环期间改变了事情有关，而我没有。嗨，Luaan，这是非常有用的建议。只是想让你知道我已经用锁同步了你看到的其他属性。