同步事件

Question

我注意到，如果事件触发得太快，我的代码有时会变得不同步。我想知道是否有更好的方法。在正常情况下，DeviceOpenedEvent 在我在 TestDevice 方法中告诉线程到 WaitOne 后触发，但我看到在某些情况下，事件在线程有机会等待之前被触发。

    protected AutoResetEvent TestAutoResetEvent = new AutoResetEvent(false);
    public EventEnum WaitForEvent = EventEnum.None;

    bool TestDevice()
    {
        OpenDevice();

        WaitForEvent = EventEnum.DeviceOpened;
        TestAutoResetEvent.WaitOne();
        WaitForEvent = EventEnum.NoWait;

        //Continue with other tests
    }

    void DeviceOpenedEvent()
    {
        if (WaitForEvent == EventEnum.DeviceOpened)         
            TestAutoResetEvent.Set();                           
    }

正常情况下是这样的：

1. 打开设备
2. WaitOne()
3. DeviceOpenedEvent 发生
4. 设置（）

这就是我有时看到的日志：

1. 打开设备
2. DeviceOpenedEvent 发生
3. WaitOne() 基本上永远卡在这里

Answer 1

由于OpenDevice 是异步的（正如您在评论中提到的），它在与其调用者不同的线程中运行。有时它会在源代码中的下一行执行之前完成：

    OpenDevice(); // Async: may finish before the next line executes!
    WaitForEvent = EventEnum.DeviceOpened;

当这种情况发生时，DeviceOpenedEvent 不会做你想做的事，因为WaitForEvent 仍然是EventEnum.None：

if (WaitForEvent == EventEnum.DeviceOpened)         
    TestAutoResetEvent.Set(); 

解决方案是更改代码，以便在保证以正确顺序运行的方法内发出完成信号。这是一个简单的实现，它删除了枚举并为您需要等待的每个事件使用单个等待句柄：

protected AutoResetEvent deviceOpenedEvent = new AutoResetEvent(false);
protected AutoResetEvent deviceLockedEvent = new AutoResetEvent(false);

bool TestDevice() {
    OpenDevice();
    // Do some unrelated parallel stuff here ... then
    deviceOpenedEvent.WaitOne();
    LockDevice();
    deviceLockedEvent.WaitOne();
}

void DeviceOpenedEvent() {
    deviceOpenedEvent.Set();                           
}

如果您控制OpenDevice，那就更容易了：完成后只需调用deviceOpened.Set()。您甚至可以更改 OpenDevice 以接受自动重置事件并在 TestDevice 内构建它，这将减少您遇到多线程错误的风险。

Answer 2

这应该不是问题。 documentation for AutoResetEvent 声明：

如果线程调用 WaitOne 而 AutoResetEvent 在有信号 状态，线程不会阻塞。

以下代码不会导致WaitOne阻塞，例如：

AutoResetEvent waitHandle = new AutoResetEvent(false);
waitHandle.Set();
waitHandle.WaitOne();
Console.WriteLine("After WaitOne");