问题 为角色扮演游戏中的“角色”设计一个干净/灵活的方式来施放不同的法术

特别清楚为什么你希望它是一个两阶段的过程，除非它将在UI中公开（即如果用户将设置“加载的咒语”并且可以在以后改变他们的想法）。

另外，如果你 是 将拥有一个属性而不仅仅是wizard.Cast（新的 Spell.MageArmor(), wizard)有一个SetSpell方法有点奇怪 - 为什么不只是做 LoadedSpell 财产公众？

最后，法术实际上有任何可变状态吗？你能拥有一组固定的实例（flyweight / enum模式）吗？我不是在考虑这里的内存使用（这是flyweight模式的正常原因），而只是它的概念性。感觉就像你想要的东西真的就像一个Java枚举 - 一组具有自定义行为的值。在C＃中更难做到这一点，因为没有直接的语言支持，但它仍然是可能的。

法术中的实际模式（有施法者和目标）似乎是合理的，但是如果你想要能够拥有区域效果法术（使用目标位置而不是特定生物）或诅咒法术，你会发现它变得不灵活/祝福物品等。您可能还需要传递游戏世界的其他状态 - 例如如果你有一个咒语来创建小兵。

我可能不会在这里为每个法术使用子类。我会尝试使用XML或JSON将其放在磁盘上并动态创建它们。

- 想澄清（希望） -

这种方法需要尽可能提前计划。您必须将属性定义为：

• 名称
• 描述
• 持续时间
• 目标（自我，区域，其他）
• 类型（奖金，伤害，诅咒）
• 效果（例如：1d6冰霜伤害，+ 2护甲等级，-5伤害抗性）

在通用法术类中包含所有这些行为应该使它非常灵活，更直接地进行测试。

用命令模式封装“法术”是很自然的（这基本上就是你所做的）。但是你遇到两个问题： -

1）你必须重新编译才能添加更多法术

• 你可以列举所有可能的 行动有可能是一个咒语 拿，然后定义一些法术 外部格式（XML，Database） 加载到您的应用程序中 启动。西方角色扮演游戏往往被编码为 这 - “咒语”包括“申请” 法术效果＃1234，参数1000“， “播放动画＃2345”等

• 您可以将您的游戏状态暴露给脚本 语言和脚本你的法术（你也可以 将此与第一个想法相结合，以便在大多数情况下 你的脚本法术只是在代码中调用预定义的效果。 Planeswalkers的决斗 （X-Box 360上的M：TG游戏）写得很广泛 这种方法

• 或者你可以忍受它（我...）

2）当你的法术目标不是生物时会发生什么？

• 如果您将游戏状态暴露给您的拼写脚本，这不是问题，因为您的脚本可以在您所暴露的内容中执行他们喜欢的任何操作。

• 否则，你最好制作一个通用类型。

我通常做类似下面的事情（不仅仅是在游戏中，我一直使用这种模式来表示mutli-agent-systems中的行为）： -

public interface IEffect<TContext>
{
  public void Apply(TContext context);
}

public class SingleTargetContext
{
  public Creature Target { get; set; }
}
public class AoEContext
{
  public Point Target { get; set; }
}
// etc.

这种模式的优势在于它可以非常灵活地执行那些你经常期望法术能够完成的更多固定模型无法做到的“奇怪”事情。你可以把它们连在一起做。你可以有一个效果，它可以为你的目标添加一个TriggeredEffect - 这对于做一些像Thorns Aura这样的事情很有帮助。您可以使用IReversibleEffect（使用额外的Unapply方法）来表示增益效果。

关于Planeswalkers决斗的那篇文章虽然非常出色。太好了，我会连接两次！

出于某种原因，“咒语”对我来说更像是一种命令模式。但我从来没有设计过这样的游戏......

我看到这个模式的最大问题是所有法术必须记住减去他们的法术力费用。怎么样：

public abstract class Spell
{
   public string Name { get; set; }
   public int ManaCost { get; set; }
   public Spell(string name, int manaCost)
   {
      Name = name;
      ManaCost = manaCost;
   }

   public void Cast(Creature caster, Creature targetCreature)
   {
       caster.SubtractMana(ManaCost); //might throw NotEnoughManaException? 
       ApplySpell(caster, targetCreature);
   }

   protected abstract void ApplySpell(Creature caster, Creature targetCreature);
}

另外，应该向导扩展PlayerCharacter，这将扩展Creature？

我觉得你的设计很好看。由于每个Spell类基本上都是一个函数的包装器（这更适合Command模式，而不是策略），你可以完全摆脱法术类，只需使用一些反射函数来找到拼写方法并添加一些他们的元数据。喜欢：

public delegate void Spell(Creature caster, Creature targetCreature);

public static class Spells
{
    [Spell("Mage Armor", 4)]
    public static void MageArmor(Creature caster, Creature targetCreature)
    {
        targetCreature.AddAC(4);
    }

    [Spell("Fire Ball", 5)]
    public static void FireBall(Creature caster, Creature targetCreature)
    {
        targetCreature.SubtractHealth(7);
    }
}

首先：对于一切事物总是有更好/更清洁/更容易的方法。

但是在我看来，你已经对你的挑战做了很好的抽象，这可以成为进一步改进的坚实基础。

我可能会遗漏一些东西，但三重奏WizardSpells，LoadedSpell，SetSpell似乎可以澄清一下。具体来说，到目前为止，我没有看到您的代码中使用的列表。我可能会使用LearnNewSpell（Spell newSpell）将向导可用的法术添加到列表中，并检查LoadSpell是否使用该列表中的一个咒语。
此外，如果您要使用多种类型的脚轮，您可能会考虑在某些时候在法术上添加一些关于施法者类型的额外信息。

你的单元测试是什么样的？

设计是否使您可以轻松编写所需的测试？