我是否应该确保对象在调用 Dispose 后不可用？

Question

我有一个类 BleScanner，它包装了一个内部 BluetoothLEAdvertisementWatcher。它还实现了 IDisposable 以确保在扫描仪被处理时停止观察者。

public sealed class BleScanner : IDisposable
{
    public event AdvertisementReceivedHandler? AdvertisementReceived;

    private readonly BluetoothLEAdvertisementWatcher m_Watcher;

    public BleScanner() {
        m_Watcher = new() {
            // ...
        };
        // m_Watcher.Received += OnAdvertisementReceived;
    }

    // private void OnAdvertisementReceived(...) {
    //    code elided for brevity
    //    may eventually raise AdvertisementReceived
    // }

    public void Start() => m_Watcher.Start();

    public void Stop() => m_Watcher.Stop();

    public void Dispose() {
        if (m_Watcher.Status == BluetoothLEAdvertisementWatcherStatus.Started) {
            m_Watcher.Stop();
        }
    }
}

观察者不是一次性的。所以理论上，如果你在Dispose之后再次调用Start，扫描器仍然可以工作：

public async Task ScannerTest(CancellationToken token) {
    using var scanner = new BleScanner();
    scanner.AdvertisementReceived += OnAdvertisementReceived;

    scanner.Start(); // will start the scan
    await Task.Delay(3000, token); // raise events for 3 seconds
    scanner.Stop(); // could be forgotten
    scanner.Dispose(); // will stop the scan if indeed it was forgotten
    
    scanner.Start(); // everything will work, despite "scanner" being disposed already
}

我是否应该确保Start（也许Stop）在Dispose被调用后抛出ObjectDisposedException？ guidelines on the Dispose pattern 只要求Dispose 可以被调用多次而不会出现异常，但不说其他成员在Dispose 被调用后应该如何表现。 IDisposable interface 的 using disposable objects 也没有说明在已处置对象上调用方法时会发生什么。

Answer 1

在您的问题中，您引用了IDisposable Guidelines。第一行说“实现 Dispose 方法主要是为了释放非托管资源”。我不认为那是你在这里做的。如果 BluetoothLEAdvertisementWatcher 是 IDisposable，那么您可以在 Dispose() 函数中处理它；但事实并非如此。因此，垃圾回收将在您的对象超出范围后及时处理您的对象；让它做它的事情。

希望有所帮助。

Answer 2

使用IDisposable 来释放托管资源是完全没问题的，这真的任何一种需要在该范围的末尾运行代码的范围。

在这种情况下，我会说范围实际上在Start 和Stop 附近。所以我会让Start返回一个调用Stop的IDisposable（并使Stop私有）。你的类型不会是一次性的。例如，使用我的 Nito.Disposables 库中的 Disposable：

public sealed class BleScanner
{
    public event AdvertisementReceivedHandler? AdvertisementReceived;

    private readonly BluetoothLEAdvertisementWatcher m_Watcher;

    public BleScanner() {
        m_Watcher = new() {
            // ...
        };
        // m_Watcher.Received += OnAdvertisementReceived;
    }

    public IDisposable Start()
    {
        m_Watcher.Start();
        return Disposable.Create(() => Stop());
    }

    private void Stop() => m_Watcher.Stop();
}

public async Task ScannerTest(CancellationToken token) {
    var scanner = new BleScanner();
    scanner.AdvertisementReceived += OnAdvertisementReceived;

    using var scannerSubsctiption = scanner.Start();
    await Task.Delay(3000, token); // raise events for 3 seconds
}