IEnumerable e IEnumerator en la misma clase, ¿mala idea?

Question

¿Es esta una mala idea?

Private Class GH_DataStructureEnumerator(Of Q As Types.IGH_Goo) 
    Implements IEnumerable(Of Q) 
    Implements IEnumerator(Of Q) 
    .... 
    .... 
    'Current, MoveNext, Reset etc.' 
    .... 
    .... 

    Public Function GetEnumerator_Generic() As IEnumerator(Of Q) _ 
        Implements IEnumerable(Of Q).GetEnumerator 
    Return Me 
    End Function 
End Class 

Esta clase sólo es visible como un IEnumerable (Of T) Propiedad de sólo lectura, y me ahorra una clase adicional que envuelve IEnumerator (Of T). Pero de alguna manera, parece estar mal. ¿Hay una mejor manera?

Answer 1

Esta es una mala idea porque la solución se descompondrá si alguien intenta enumerar su colección más de una vez al mismo tiempo.

Por ejemplo: Esto romperá de alguna manera

For Each cur1 in yourObj 
    For Each cur2 in yourObj 
    Console.WriteLine("{0} - {1}", cur1,cur2) 
    Next 
Next 

Answer 2

Definitivamente una mala idea. En particular, significa que dos llamadas a GetEnumerator devolverán referencias al mismo objeto, cuando deberían devolver iteradores independientes.

Dicho esto, el compilador de C# generará clases que implementarán ambos tipos si usa bloques de iterador ... pero hace todo lo posible para asegurarse de hacerlo bien. Le sugiero que no se someta a ese dolor :)

Answer 3

De Implementing IEnumerable:

proporcionan un método GetEnumerator() que devuelve una estructura pública anidada llamados “enumerador” .

Por cierto, este sitio web es mantenido por Brad Abrams - uno de los autores de Framework Design Guidelines.