如何正确锁定 Task.Run() 块

Question

我正在编写一个应用程序，其中我使用了几种访问某些共享资源的方法，因此通过lock(thisLock){ [...] ] 实现了一些安全性

一切都很好，直到我不得不在异步任务中使用资源。 这是代码：

private object thisLock = new object();
[...]
private void UpdateStuff()
{
lock(thisLock)
   {
     Task.Run(()=>{[code where I use shared resources]});
   }
}

它似乎工作正常，但我想知道这是否是正确的做法，或者我是否应该将 lock() 部分放在 Run() 中，如下所示：

private void UpdateStuff()
{

     Task.Run( () => {
        lock(thisLock)
        {
         [code where I use shared resources]
        }
     });    
}

我试图获取一些信息，但我发现的所有内容都提到了使用关键字 async 时的情况，而事实似乎并非如此。

所以，我的问题是：哪一种是最好/正确的方式来使用带有 Task.Run() 的 lock()？为什么？

谢谢！

编辑：为了澄清，我的疑问是锁定何时以及多少有效，所以基本上我问的是在第一种情况下是否有可能锁定仅对 Run() 调用有效，因为调用本身的内容可能存在于另一个线程中，而在第二种情况下，锁将与内容相关联，因此是有效的。

Answer 1

您的第一种方法是完全错误的。 Task.Run 可以在同一个线程上同步执行工作或将其排队到线程池中。你不知道会发生什么。

您无需等待任务完成，因此如果它排队到线程池中，您的函数将释放锁并返回，可能在工作开始之前。

一般来说，最好的做法是只在需要的时间内获取锁，并且永远不要在等待另一个线程时保持锁，因为如果你不特别小心，很容易导致死锁。

Answer 2

这两者之间没有最好的方法，你只会得到两种不同的结果：

在第一种情况下，在锁打开之前您不会开始任务。

在第二种情况下，您将启动一个在锁打开之前不会执行任何操作的任务。

在我看来，在需要之前不要打开任务，这是一种更好的方法，但这只是个人意见。开启和锁定之后，会分配无用的资源。