¿Está bien que devuelva IEnumerator <T>. GetEnumerator() en IEnumerator.GetEnumerator()?

Question

suponer Básicamente tienes alguna colección:

public class FurCollection : IEnumerable<FurStrand> 
{ 
    public IEnumerator<FurStrand> GetEnumerator() 
    { 
     foreach(var strand in this.Strands) 
     { 
     yield return strand; 
     } 
    } 

    IEnumerator IEnumerable.GetEnumerator() 
    { 
     return this.GetEnumerator(); 
    } 
} 

Es esto aceptable? ¿O es esta propensión a errores o mala práctica? Casi siempre usaré el IEnumerator<T> pero aún quiero que la versión no genérica sea estable y se implemente correctamente.

Answer 1

Esto es completamente estándar y se recomienda para cumplir con DRY y otras preocupaciones.

Tenga en cuenta que

return strand; 

debería ser

yield return strand; 

Además, parece que this.Strands ya implementa IEnumerable<FurStrand> por lo que podría simplemente decir

return this.Strands.GetEnumerator(); 

Answer 2

No, esto es perfectamente aceptable. Recomendado incluso

Answer 3

Esto no sólo es una buena práctica, pero su proyecto no cumplirá sin esto, porque IEnumerable<T> hereda IEnumerable. Mira la definición de IEnumerable<T>:

public interface IEnumerable<out T> : IEnumerable 
    IEnumerator<T> GetEnumerator(); 
} 

Hay que aplicar la versión no genérica o que obtendrá un error "... no implementa miembro de interfaz ..."