从 Lock() 中调用 UI 线程上的方法

Question

我有两种方法，MethodA 和 MethodB。 MethodB 必须在 UI 线程上运行。我需要它们一个接一个地运行，不允许MethodC 在它们之间运行。

MethodC 在用户点击一个可爱的小按钮时被调用。

我为确保这一点所做的工作是在代码周围放置一个Lock：

 lock (MyLock)
 {
   MethodA(param1, param2);

   MyDelegate del = new MyDelegate(MethodB);
   if (this.IsHandleCreated) this.Invoke(del);
 }

对于MethodC：

public void MethodC()
 lock (MyLock)
 {
   Do bewildering stuff.....
 }

问题是我卡住了。我的代码似乎陷入了僵局。

当我查看线程时，我看到按钮单击调用的代码卡在MethodC 中的lock (MyLock)，而我的其他线程似乎卡在this.Invoke(del)。

我已经读过从 Lock 中调用方法很危险，但由于我是在那里编写代码的人，即使只有 Thread.Sleep 似乎也会发生这种情况，我认为这不是代码给我惹麻烦了。

为什么 Invoked 方法会停止工作？ 是否可能等待methodC 上的锁被释放，然后再返回调用它的原始锁？

Answer 1

所以，想象一下以下情况：

1. 您的后台线程开始运行代码。它抓住锁然后开始运行MethodA。

2. MethodC 被调用，而MethodA 正在其工作中。 MethodA 等待锁释放，阻塞 UI 线程，直到发生这种情况。

3. 后台线程完成MethodA 并在UI 线程上调用MethodB。 MethodB 在消息泵队列中的所有先前项目都完成之前无法运行。

4. MethodC 位于消息泵队列的顶部，等待 MethodB 完成，MethodB 在队列中等待 MethodC 完成。他们都在互相等待，这是一个僵局。

那么，你如何解决这个问题？您真正需要的是某种“等待”锁而不实际阻塞线程的方式。幸运的是（在 .NET 4.5 中）这很容易做到，这要归功于任务并行库。 （我在引号中等待，因为我们实际上并不想等待，我们只想在锁被释放后立即执行MethodC而不实际等待/阻塞当前线程。） p>

不要使用object 代替MyLock 使用：

private static SemaphoreSlim semaphore = new SemaphoreSlim(1, 1);

现在MethodC 你可以这样做：

public async Task MethodC() //you can change the signature to return `void` if this is an event handler
{
    try
    {
        await semaphore.WaitAsync();
        //Do stuff
    }
    finally
    {
        semaphore.Release();
    }
}

这里的关键是因为我们await 一个代表信号量何时真正空闲的任务我们没有阻塞当前线程，这将允许其他后台任务将MethodB 编组到 UI 线程，完成方法，释放信号量，然后让这个方法执行。

您的其他代码不需要（但如果您愿意，仍然可以）在信号量上使用异步等待；阻塞后台线程几乎不是什么大问题，所以唯一的关键变化是使用信号量而不是lock：

public void Bar()
{
    try
    {
        semaphore.Wait();
        MethodA(param1, param2);

        MyDelegate del = new MyDelegate(MethodB);
        if (this.IsHandleCreated) this.Invoke(del);
    }
    finally
    {
        semaphore.Release();
    }
}

Answer 2

假设您的 MethodA 还包含类似的内容：

lock(MyLock) {

}

你是绝对正确的，你有一个死锁。 MethodA 无法获得对 MyLock 的锁定，因为在进入方法之前它已经被锁定。

Answer 3

你可以试试这个：

Lock (MyLock)
 {
   MethodA(param1, param2);

   MyDelegate del = new MyDelegate(MethodB);
   MyDelegate del2 = new MyDelegate(MethodC);
   MyDelegate del3 = del+del2
   if (this.IsHandleCreated) this.Invoke(del3);
 }

Answer 4

你把使用锁的人搞糊涂了。此任务与多线程本身无关。

您需要简单的可用性解决方案 - 禁用您可爱的小按钮，直到您准备好运行 MethodC。