c# - 如何为基于策略模式的应用程序编写清晰优雅的代码？

我正在编写一个应用程序来创作音乐，但我有点担心代码中的类和接口(interface)的结构。这里有我写的一些类的签名:

Interface IGuidoFormattable
Class Note : IGuidoFormattable, ICloneable
Class Pause : IGuidoFormattable, ICloneable
Class Chord : List<Note>, IGuidoFormattable, ICloneable
Class Key : IGuidoFormattable, ICloneable
Class Tempo : IGuidoFormattable, ICloneable
Class Meter : IGuidoFormattable, ICloneable
Class FretPiece : List<IGuidoFormattable>, ICloneable
Class Fret : List<FretPiece>

FretPiece 代表一个音乐短语，一个完整的 freize。它公开为属性 Key、Tempo 和 Meter，它们与它们的类型同义。更多的短语放在一起创建一个 Freize，由 Fret 类表示。单个短语中的每个元素都必须根据 GUIDO 标准表示法进行格式化，因此它必须实现 IGuidoFormattable 接口(interface)。
在另一个命名空间中，定义了一些变异类，它们都继承自两个抽象类之一:

Class FretMutation
Class LambdaFretMutation : FretMutation

最后，存在一个名为 FretMutationGenerator 的类，它的唯一任务是将选定的突变应用于音乐主题，并将整个 freize 作为 Fret 类的实例输出。

FretPiece 必须能够包含几个不同的元素(在这种情况下是音符、停顿和和弦)，但它们必须满足两个约束条件:它们必须可以使用 GUIDO 符号进行格式化，从而转换为有意义的字符串；它们必须是可克隆的。在现在的代码中，每个类都实现了 ICloneable，但是当前代码的语法和语义并不能保证集合的所有成员都是可克隆的。我需要找到一种方法来表达这两个约束，而无需将继承应用于 IGuidoFormattable，并且最好不要在 IGuidoFormattable 接口(interface)中定义 Clone 方法。

其次，也是最重要的问题。 FretMutation 定义了一个抽象方法“Apply”，它必须在每个派生类中被覆盖。因此，任何突变类都定义了自己的此方法版本，它具有以下签名:

FretPiece Apply(FretPiece originalTheme)

它接受 FretPiece 作为输入并输出该对象的副本，根据指定为类成员的所有其他参数进行变异。我认为这是策略模式的一种实现。但是，仅在此方法创建输入副本这一事实的情况下，这意味着参数本身(及其所有成员)必须是可克隆的。
此外，FretPiece 被声明为 IGuidoFormattable 的列表，但每个突变类的行为都与其他类不同，并且可能会作用于音符、停顿或和弦，因此:这意味着我需要检查每个元素的类型，并为每种类型都有“很多”(实际上，最多 3 个)if 语句。这在我看来很少面向对象。

我怎样才能以一种更加面向对象的方式来安排类和接口(interface)，而不那么依赖于假设和类型检查？

最佳答案

第三个选项呢？

public interface ICloneableAndGuidoFormattable : IGuidoFormattable, ICloneable { }

那么你的 FretPiece 是 ICloneableAndGuidoFormattable 的列表

如果不是这样，您可以尝试这样的构造:

public interface ICloneable<T>
{
  T Clone();
}

public class FretPiece : IEnumerable<IFormattable>, ICloneable<FretPiece>
{
    private List<IFormattable> items = new List<IFormattable>();

    public void Add<T>(T value) where T : IFormattable, ICloneable<IFormattable>
    {
        items.Add(value);
    }

    public IEnumerator<IFormattable> GetEnumerator()
    {
        items.GetEnumerator();
    }

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

    public FretPiece Clone()
    {
        return new FretPiece { items = new List<IFormattable>(
            items.Cast<ICloneable<IFormattable>>().Select(c=>c.Clone()))
        };
    }
}

和其他地方，例如在你的突变器上:

public T Apply<T>(T fretPiece) where T : IEnumerable<IFormattable>, ICloneable<T> ( ...)

这将确保您只能添加实现这两个接口(interface)的项目。枚举仅假定返回 IFormattables。这将允许您在强制转换中安全地强制转换为 ICloneable，因为它必须通过“添加”的类型约束。可以看到clone的实现。即使你在那里有 Actor ，它也是安全的，除非有人根据反射摆弄 items ;)

关于c# - 如何为基于策略模式的应用程序编写清晰优雅的代码？，我们在Stack Overflow上找到一个类似的问题：https://stackoverflow.com/questions/4949606/