我有一个需要一次处理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函数创建webrequest并处理响应(回调)。这是一项耗费少量时间的功能。
但是,如果我正确理解任务,则一个线程可以有多个任务。因此,如果我睡眠线程,其他任务可能会受到影响。
是否有更有效的方法来实现上述目标,并且可以在Parallel.Foreach中使用任务吗?
答案 0 :(得分: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)。这种辅助函数的好处在于它从重要部分解开了批处理逻辑。
答案 1 :(得分:1)
使用
Task.Delay
代替
static async Task DoSomeProcess()
{
await Task.Delay(3000);
}
你是对的,Thread.Sleep会阻止其他任务
是的,你可以将async / await模式与Parallel配对。
答案 2 :(得分: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);
}
}
唯一的好处是你不会阻止ProcessItems
线程Task.WaitAll()
和Thread.Sleep()
,正如@usr在他的回答中指出的那样。是采用这种方法还是Parallel.ForEach
可能取决于代码的运行环境。 Async/await
不会让你的代码运行得更快,但它会提高服务器端执行的可扩展性,因为它可能需要更少的线程来运行,因此可以提供更多的客户端。
另请注意,现在ProcessItemsAsync
本身就是一个异步任务,所以为了保持调用它的代码流不变,你需要像这样调用它:
ProcessItemsAsync().Wait();
这本身就是一个封锁电话,可能会扼杀我们刚刚获得的async
优势。是否可以在应用程序中完全消除这样的块,很大程度上取决于应用程序工作流程的其余部分。