使用 IEnumerator的模式在接口中

Question

我有一个 C# 类，需要通过一堆方法处理一系列项目 (IEnumerable)，因此我不能简单地在方法内 foreach 。我调用 .GetEnumerator() 并传递此 IEnumerator，它非常有效，为我在循环单个序列时提供了所需的灵活性。

现在我想允许其他人在这个过程中添加逻辑。最自然的方法是为它们提供一个接口，其中包含接受 IEnumerator 的方法。简单，完成，并且有效。

但我担心这是一种反模式。他们必须知道 IEnumerator 已经调用了 .MoveNext()，因此他们可以简单地访问 .Current。另外，我没有看到任何在要实现的接口中使用 IEnumerator 的先例。

1. 我没有考虑哪些陷阱？
2. 是否有另一种模式允许我使用相同的高效机制（即我不希望创建/销毁多个副本）而不暴露IEnumerator本身？

更新：正如我在下面的评论中提到的：我想要的是某种通用的Stream。我需要能够有效地查看下一个项目 (IEnumerator.Current -> .Peek()) 并使用它 (IEnumerator.MoveNext () -> .Pop())。

我使用了 IEnumerator 因为它符合账单界面。我更喜欢在合适的情况下使用常见的 BCL 类型，但我似乎滥用了这一类型。

那么问题3）有没有一个类可以满足这个需求？或者我应该创建自己的 Stream，它在内部延迟执行 IEnumerator？然后它将被完全封装。我不想使用许多现有集合，因为它们具有内部存储，而我希望存储为 IEnumerable iteslf。

---

好吧，听起来共识是 IEnumerator 通常是 ValueType 并且不知道 IEnumerator的先验状态。，传递它通常是一个坏主意。

我听到的最好的建议是创建我自己的类并进行传递。还有其他建议吗？

Answer 1

您绝对不应该传递IEnumerator。除此之外，在某些情况下它可能会产生一些非常奇怪的效果。例如，您希望这里发生什么？

using System;
using System.Collections.Generic;

class Test
{
    static void ShowCurrentAndNext(IEnumerator<int> iterator)        
    {
        Console.WriteLine("ShowCurrentAndNext");
        Console.WriteLine(iterator.Current);
        iterator.MoveNext(); // Let's assume it returns true
        Console.WriteLine(iterator.Current);
    }

    static void Main()
    {
        List<int> list = new List<int> { 1, 2, 3, 4, 5 };
        using (var iterator = list.GetEnumerator())
        {
            iterator.MoveNext(); // Get things going
            ShowCurrentAndNext(iterator);
            ShowCurrentAndNext(iterator);
            ShowCurrentAndNext(iterator);
        }
    }
}

尝试进行一些更改：

using (List<int>.Enumerator iterator = list.GetEnumerator())

并

using (IEnumerator<int> iterator = list.GetEnumerator())

尝试预测每种情况下的结果:)

现在不可否认，这是一个特别邪恶的示例，但它确实演示了与传递可变状态相关的一些极端情况。我强烈鼓励您在“中心”方法中执行所有迭代，该方法仅使用当前值调用适当的其他方法。

Answer 2

我强烈建议不要传递枚举器本身；除了需要当前值之外，您还有什么理由这样做？

除非我遗漏了一些明显的东西，否则我建议您的实用程序函数只需将您枚举的类型作为参数，然后使用一个外部 foreach 循环来处理实际的枚举。

也许您可以提供一些额外的信息来说明您迄今为止为何做出此设计决策。

Answer 3

在我看来，您可能会受益于使用事件，以便您可以将要处理的项目的通知推送给侦听器。常规 .NET 事件按照订阅的顺序进行处理，因此如果需要排序，您可能会采用更明确的方法。

您可能还想查看响应式框架。

Answer 4

如果我理解正确的话，您有许多方法都可以在序列上调用 MoveNext，并且您希望这些方法彼此协作，因此您传递了 IEnumerator。正如您所提到的，这里肯定存在一些紧密耦合，因为您希望枚举器在每个方法的入口处处于特定状态。听起来你真正想要的是类似 Stream 类的东西，它既是一个集合（某种程度上）又是一个迭代器（具有当前位置的概念）。我会将您的迭代和您需要的任何其他状态包装在您自己的类中，并将各种方法作为该类的成员