Casting y herencia de interfaz

Question

Cada elemento tiene una interfaz, IItem. Además de esto, existe una interfaz conocida como IDrawableItem que hereda de Item.

El siguiente código intenta dibujar un elemento dibujable, pero no puede, ya que la colección almacena esta clase acepta solo IItem. Puede agregar cualquier cosa que herede de IItem a esta clase, pero usar otros métodos solo se puede lograr mediante casting.

foreach (var item in Items) { 
    item.Draw();    // The casting would go here. 
} 

sé cómo lanzar, as etc ... pero es esto aceptable? ¿Es una mejor práctica?

Solo me pregunto si hay otras formas de manejar estos escenarios.

Answer 1

Uso Enumerable.OfType para extraer sólo aquellos elementos de Items que implementan IDrawableItem:

foreach(var item in Items.OfType<IDrawableItem>()) { 
    item.Draw(); 
} 

Para abordar la cuestión de que Nanook pidió en los comentarios del código anterior, probablemente se traduce en un código equivalente a la siguiente:

foreach(var item in Items) { 
    if(item is IDrawableItem) { 
     ((IDrawable)item).Draw(); 
    } 
} 

Por supuesto, en realidad no es un iterador detrás de las escenas que se ve algo como esto:

public static IEnumerable<T> OfType<T>(this IEnumerable<TSource> source) { 
    if(source == null) { 
     throw new ArgumentNullException("source"); 
    } 
    foreach(TSource item in source) { 
     if(item is T) { 
      yield return (T)item; 
     } 
    } 
} 

Entonces, lo que esto muestra es que probablemente solo iteremos a través de Items una vez. Por supuesto, no hay ningún requisito de que OfType se implemente como el anterior, pero esto es lo más sensato.

Answer 2

dos soluciones alternativas:

• mantener el dibujables en una recogida selectiva.
• Agregue el método DrawIfPossible() al IItem. IDrawableItem debe anularlo para llamar al Draw(), otros implementadores IItem() deben tener una implementación vacía.

Las consultas de tipo explícito se consideran un signo de que puede haber algún problema con el diseño.