1

我有多个生产者和多个消费者。我的共享资源是 BlockingCollection。但是，我的代码只有在我有一个消费者时才有效。我知道这是一种竞争条件，因为每次运行代码时输出都不同。

我认为 BlockingCollection 会处理所有同步等，但事实并非如此。

那么如何在所有生产者和消费者之间同步我的共享资源呢？

这是我的代码：

/// <summary>
/// PURE PRODUCER TYPE
/// </summary>
class Caller
{

    private BlockingCollection<Call> incommingCalls;

    public Caller(BlockingCollection<Call> calls)
    {
        incommingCalls = calls;
        //start the producer thread
        Thread thread = new Thread(new ThreadStart(placeCall));
        thread.Start();
    }

    public void placeCall()
    {
            incommingCalls.Add(myCall);
    }

}



/// <summary>
/// CONSUMER
/// </summary>
class Fresher : Employee
{

    private BlockingCollection<Call> calls;

    public Fresher(BlockingCollection<Call> incalls)
    {
        calls = incalls;
        Thread thread = new Thread(new ThreadStart(HandleCalls));
        thread.Start();

    }

    /// <summary>
    /// 
    /// </summary>
    public void HandleCalls()
    {
        while (!incommingCalls.IsCompleted)
        {
            Call item;
            if (incommingCalls.TryTake(out item, 100000))
            {
                //do something with the call

            } //else do nothing - just wait

        }


}






/// <summary>
/// 
/// </summary>
class CallCenter
{

    private BlockingCollection<Call> fresherCalls;


    private List<Caller> myCallers;
    private List<Employee> myFreshers;


    public CallCenter() 
    {
        //initial incomming calls to the fresher queue
        fresherCalls = new BlockingCollection<Call>();

        myFreshers = new List<Employee>();
        myCallers = new List<Caller>();

        generate_freshers();

        //generate to start the producer
        generate_callers();

    }

    /// <summary>
    /// 
    /// </summary>
    private void generate_freshers() 
    {
        for (int i = 0; i < 1; i++ )
        {
            myFreshers.Add(new Fresher(fresherCalls, tlCalls, locker2));
        }
    }

    /// <summary>
    /// 
    /// </summary>
    private void generate_callers() 
    {
        for (int i = 0; i < 20; i++ )
        {
            myCallers.Add(new Caller(fresherCalls, locker));
        }

    }
}
4

1 回答 1

3

我知道这是一种竞争条件，因为每次运行代码时输出都不同。

这在多线程中很常见，不一定是由于竞争条件（至少不是一个糟糕的条件）。多线程场景中的订单处理往往不是确定性的——这可能会改变输出。

话虽如此，使用BlockingCollection<T>，通常更容易将您的消费者编写为：

public void HandleCalls()
{
    foreach(var item in incommingCalls.GetConsumingEnumerable())
    {
        //do something with the call
    }
}

这将为您处理所有同步和检查，对于BlockingCollection<T>.

编辑：如果您需要控制调度并实现某种形式的Round-Robin Scheduling，您可能需要查看TPL 示例中的Parallel Extension Extras 。它们提供了一个RoundRobinTaskScheduler可用于调度Task<T>以可预测方式工作的实例。

于 2013-01-28T18:02:15.323 回答