IEnumerable und IEnumerator in der gleichen Klasse, schlechte Idee?

Question

Ist das eine schlechte Idee?

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 

Diese Klasse ist nur sichtbar, als IEnumerable (Of T) Nur-Lese-Eigenschaft, und es spart mir eine zusätzliche Klasse, die IEnumerator (Of T) umschließt. Aber irgendwie scheint es einfach falsch zu sein. Gibt es einen besseren Weg?

Answer 1

Dies ist eine schlechte Idee, weil die Lösung zusammenbrechen wird, wenn jemand versucht, Ihre Sammlung mehr als einmal zur gleichen Zeit aufzuzählen.

Zum Beispiel: Das in irgendeiner Weise brechen

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

Answer 2

Definitiv eine schlechte Idee. Insbesondere bedeutet dies, dass zwei Aufrufe an GetEnumerator Verweise auf dasselbe Objekt zurückgeben - wenn sie unabhängige Iteratoren zurückgeben sollen.

Nun gesagt, der C# -Compiler wird Klassen generieren, die beide Typen implementieren, wenn Sie Iteratorblöcke verwenden ... aber es geht um große Anstrengungen, um sicherzustellen, dass es richtig ankommt. Ich schlage vor, dass Sie sich nicht durch diesen Schmerz bringen :)

Answer 3

Von Implementing IEnumerable:

bieten Sie eine GetEnumerator() -Methode, die eine verschachtelte public struct „Enumerator“ genannt zurück .

By the way, ist diese Webseite von Brad Abrams gepflegt - einer der Autoren von Framework Design Guidelines.