Question

我很惊讶反应性扩展CompositeDisposable类未通过以下测试

[Test]
public void TestDisposable()
{
    var ds = new List<IDisposable>();
    int[] a = { 1, 2, 3 };
    using (new CompositeDisposable(ds))
    {
        ds.Add(Disposable.Create(() => a[0] = 3));
        ds.Add(Disposable.Create(() => a[2] = 1));
    }
    Assert.That(a[0],Is.EqualTo(3)); //Failed here
    Assert.That(a[1], Is.EqualTo(2));
    Assert.That(a[2], Is.EqualTo(1));
}

所以这意味着如果我通过给定CompositeDisposable来创建IEnumerable<IDisposable>，它实际上会迭代其所有元素，而不是将其推迟到被调用的Dispose。

这在其他情况下可能有用，但在我的情况下它很不方便。是否还有其他类可以达到上述优势？

Answer 1

CompositeDisposable的构造函数将复制您的列表。因此，当您在之后向列表中添加元素时，CompositeDisposable将不会使用它们。

这是一个简单的解决方案：

using (var composite = new CompositeDisposable())
{
    composite.Add(Disposable.Create(() => a[0] = 3));
    composite.Add(Disposable.Create(() => a[2] = 1));
}

或者，如果这不方便，这是一个解决方法：

public static IDisposable AsLazyComposite(this IEnumerable<IDisposable> sequence)
{
    return Disposable.Create(() =>
    {
        foreach (var disposable in sequence)
            disposable.Dispose();
    });
}

用法：

using (ds.AsLazyComposite())

CompositDisposable不够懒吗？

1 个答案: