Cómo evitar el switch-case en un método de fábrica de clases secundarias

Question

Digamos que tenemos una familia de clases (tarjetas, por el bien de ellas), y tenemos que crear instancias basadas en algún identificador. Un método de fábrica sería así:

public Card GetCard(int cardNumber) 
{ 
    switch(cardNumber) 
    { 
    case 13: return new King(); 
    case 12: return new Queen(); 
    case 11: return new Jack();   
    } 

    //... 
} 

Lo que yo quiero es evitar esta switch. ¿Por qué? Tal vez quiero reutilizar esta comparación en la función.

Lo que se me ocurrió es algo como esto:

private Dictionary<int, Type> cardTypes = 
{ 
    {13, typeof(King)}, 
    {12, typeof(Queen)}, 
    {11, typeof(Jack)} 
}; 

public Card GetCard(int cardNumber) 
{   
    var cardType = cardTypes[cardNumber]; 
    var instance = Activator.CreateInstance(cardType); 
    return (Card)instance; 
} 

Sin embargo, esta solución utiliza la reflexión, que es caro, y también es problemática cuando se tiene más de un "identificador" (por ejemplo 1 y 14 ambos dan Ace - ¿debo agregar 2 claves al diccionario?).

¿Cuál es la mejor práctica en este escenario?

Answer 1

En lugar de almacenar el tipo en el diccionario, se puede almacenar un Func<Card>:

private Dictionary<int, Func<Card>> cardFactories = 
{ 
    { 13,() => new King() }, 
    // etc 
} 

public Card GetCard(int cardNumber) 
{   
    var factory = cardFactories[cardNumber]; 
    return factory(); 
} 

En el caso de las tarjetas, probablemente me los hago inmutable para comenzar, y acaba de llenar con el diccionario las tarjetas en sí mismas, pero esa es una cuestión diferente :)