c# - 如何在子类的工厂方法中避免切换情况

可以说我们有一系列的类(为了方便起见，使用卡)，我们需要根据一些标识符实例化它们。
工厂方法如下所示:

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

   //...
}

我想要的是避免使用switch。为什么？也许我想在功能中重用此比较。

我想到的是这样的:

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;
 }

但是，此解决方案使用了昂贵的反射，当您有多个“标识符”时(例如1和14都给出Ace-我应该在字典中添加2个键吗？)，反射也很成问题。

在这种情况下，最佳做法是什么？

最佳答案

除了将类型存储在字典中之外，您还可以存储Func<Card>:

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

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

在卡片的情况下，我可能会让它们从一开始就不可变，只是用卡片本身填充字典，但这是另一回事:)