任务使用一种方法。如何防止代码重复？

Question

假设有两个任务：

Task.Run(() => {
    while (!this.isCanceled)
        DoOperation(obj);
});

和

Task.Run(() => {
    foreach(var obj in objects)   // Another objects' subset
        while(!this.isCancelled)
            DoOperation(obj);
});

和DoOperation(Foo obj) 实现：

DoOperation(Foo obj)
{
    if (obj.IsBar && obj.IsFoo)
        obj.MakeSad();
    else
        obj.MakeHappy();
}

我们可以锁定方法DoOperation，一切都应该很好，但是这种方法有一些性能负面的方面。此外，使用DoOperation 方法可以一次处理大量任务。 第二种方法是将整个方法复制到每个任务中。但情况更糟。

那么问题是如何避免锁定并防止代码重复？

Answer 1

如果您的操作，关键代码是什么？必须有一个或多个竞争条件，否则根本不需要使用锁。

如果在您的示例中它是 DoSomething 中的内容，则使用那里的锁：

DoOperation(Foo obj)
{
    if (obj.IsBar && obj.IsFoo)
        obj.MakeSad();
    else {
        lock(lockObject1) {
             obj.MakeHappy();
        }
    }
}

这样，如果功能在没有被阻塞的情况下工作，您就不会复制代码和其他所有内容。如果使用了另一个有争议的块，请使用另一个锁对象，这样它们就不会相互阻塞。

顺便说一句，由于您正在使用任务和同步编程，最好用户SemaphoreSlim锁定https://blog.cdemi.io/async-waiting-inside-c-sharp-locks/amp/