3

我有一个需要一次处理项目 3 的函数，如果总时间少于 x 秒，线程应该在继续进行之前休眠剩余的秒数。

所以我正在做以下事情：

    private void ProcessItems()
    {
        for (int i = 0, n = items.Count; i < n; i++)
        {
            Stopwatch stopwatch = new Stopwatch();
            stopwatch.Start();

            batch.Add(items[i]);

            if (batch.Count == 3 || i >= items.Count - 3)
            {
                List<Task> tasks = new List<Task>(3);

                foreach (Item item in batch)
                    tasks.Add(Task.Factory.StartNew(() => ProcessItem(item)));

                Task.WaitAll(tasks.ToArray());

                batch.Clear();
            }

            stopwatch.Stop();

            int elapsed = (int)stopwatch.ElapsedMilliseconds;
            int delay = (3000) - elapsed;

            if (delay > 0)
                Thread.Sleep(delay);
        }
    }

ProcessItem 函数发出网络请求并处理响应（回调）。这是需要少量时间的功能。

但是，如果我正确理解任务，一个线程可以有多个任务。因此，如果我让线程休眠，其他任务可能会受到影响。

有没有更有效的方法来实现上述目标，是否可以在 Parallel.Foreach 中使用任务？

4

3 回答 3

2

任务在自动管理的线程上运行。阻塞线程本质上没有错。这只是有点浪费。

以下是我将如何非常干净地实现它：

MyItem[] items = ...;
foreach(MyItem[] itemsChunk in items.AsChunked(3)) {
 Parallel.ForEach(itemsChunk, item => Process(item));
 //here you can insert a delay
}

这不会浪费一个线程，而且非常简单。Parallel.ForEach也使用当前线程来处理工作项，因此它不会闲置。您也可以添加延迟逻辑。实施AsChunked留给读者作为练习......这个函数应该将列表拆分为给定大小的块（3）。这样一个辅助函数的好处是它将批处理逻辑与重要部分分开。

于 2013-10-13T13:17:19.320 回答
1

采用

Task.Delay

反而

    static async Task DoSomeProcess()
    {
        await Task.Delay(3000);
    }

你是对的，Thread.Sleep 会阻塞其他任务

是的，您可以将 async/await 模式与 Parallel 配对。

于 2013-10-13T13:14:06.923 回答
0

您的ProcessItems方法可以很容易地转换为异步版本ProcessItemsAsync（我没有验证“批处理”逻辑）：

private async Task ProcessItemsAsync()
{
    for (int i = 0, n = items.Count; i < n; i++)
    {
        Stopwatch stopwatch = new Stopwatch();
        stopwatch.Start();

        batch.Add(items[i]);

        if (batch.Count == 3 || i >= items.Count - 3)
        {
            List<Task> tasks = new List<Task>(3);

            foreach (Item item in batch)
                tasks.Add(Task.Run(() => ProcessItem(item)));

            await Task.WhenAll(tasks.ToArray());

            batch.Clear();
        }

        stopwatch.Stop();

        int elapsed = (int)stopwatch.ElapsedMilliseconds;
        int delay = (3000) - elapsed;

        if (delay > 0)
            await Task.Delay(delay);
    }
}

正如@usr 在他的回答中指出的那样，唯一的好处是你不会用and阻塞ProcessItems线程。是否采用这种方法可能取决于代码的运行环境。不会使您的代码运行得更快，但会提高其在服务器端执行的可伸缩性，因为它可能需要更少的线程来运行，因此可以为更多的客户端提供服务。Task.WaitAll()Thread.Sleep()Parallel.ForEachAsync/await

另请注意， nowProcessItemsAsync本身就是一个异步任务，因此要保持调用它的代码流不变，您需要像这样调用它：

ProcessItemsAsync().Wait();

这本身就是一个阻塞调用，可能会扼杀async我们刚刚获得的优势。您是否可以在应用程序中完全消除这样的块，很大程度上取决于应用程序的其余工作流程。

于 2013-10-14T01:48:05.367 回答