重构两个包含一半相关函数和一半不相关函数的类时应使用哪种设计模式？

Question

我有两个对象，鸟和狗。 Bird 和 Dog 大约 50% 的方法/属性具有相同的实现，但对于其他 50% 的方法/属性具有不同的不相关方法和属性。

我正在重构，但我不知道最好的策略是什么。

我尝试定义一个超类 Animal 来实现超类中的通用方法，并让子类定义自己的方法/属性。

我编写了一个返回其中一个的工厂 - 这一切似乎都是正确的...但是在编写调用代码时我很困惑..例如

public class Animal{
   public string Talk(){ return "yak yak yak";
}
public class Dog:Animal{
   public string Walk(){ return "walk walk walk"; }
}
public class Bird:Animal{
   public string Fly(){ return "flap flap flap"; }
}
...
Animal thing = CreatureFactory.GetCreature(modifier);

当我想使用 thing 来 Talk< /strong> 没有问题，

Debug.Print(thing.Talk());

但是作为程序员，当我知道我希望它飞时，我是否可以将其投射到鸟呢？这似乎是错误的......但在 Dog 上定义 Fly 方法似乎也是错误的。

Answer 1

你要么把它扔给一只鸟：

Debug.Print(((Bird)thing).Fly());

要么你一直把它当作一只鸟：

// depending on how the factory works, might not need the cast
Bird thing = (Bird) CreatureFactory.GetCreature(modifier);
Debug.Print(bird.Fly());

Answer 2

给定您的示例，当您希望您的 Animal 飞行时，执行飞行的类正在使用错误的类 - 它应该使用 Bird 本身或某些 ICanFly 界面。

虽然其他答案说您可以通过强制转换来实现，但代码的可读性会因此受到影响。当您使用工厂创建对象时，绝对没有理由将这些对象转换为另一种类型。还有一个有力的论据表明，当您开始转换为其他类型时，您的类违反了单一职责原则。

Answer 3

我处理它的方法是让工厂方法返回具体类型而不是超类。您仍然可以将其传递给可以对超类进行操作的方法，但是您可以在需要时使用具体类的一项，并且不需要强制转换它。

Bird bird = CreatureFactory.GetBird();

或者

Dog dog = CreatureFactory.GetDog();

现在，您仍然可以将它们用作动物。

public class Trainer
{
     public void TeachToSpeak( Animal animal )
     {
          ...
          animal.Talk();
     }
}

但由于它们是针对具体类的类型，因此您可以适当地利用它们不共享的方法。

Answer 4

我想说：
如果你的狗也有一个给出“运动”概念的方法，我会将“fly and walk”的名称更改为“move”，然后调用它。
如果你的狗没有这样的东西，开发者不应该能够在任何物体上调用它，因为并不是所有的动物都能飞:)

Answer 5

一方面，我会在基类上放置一个名为 Move() 的虚拟方法，并在派生类中重写它。

（以下是 C#）

public abstract class Animal {
   public string Talk() { return "yak yak yak"; }
   public virtual string Move();
}
public class Dog : Animal {
   public override string Move() { return "walk walk walk"; }
}
public class Bird : Animal {
   public override string Move() { return "flap flap flap"; }
}

但是为了回答你的问题，如果你希望动物在（且仅当）它可以飞时移动，你可以定义一个 IFlyingAnimal 接口并使用Bird 来实现它。然后您可以测试 Animal 是否实现该接口。如果是，则将其转换为 IFlyingAnimal 并调用其 Fly() 方法。

public interface IFlyingAnimal {
   string Fly();
}
public class Bird : Animal, IFlyingAnimal {
   public string Fly(){ return "flap flap flap"; }
}

//later, in your main program
public string FlyIfYouCan(Animal animal) {
    if (animal is IFlyingAnimal)
        return ((IFlyingAnimal)animal).Fly();

    return "I can't fly!";
}

您不必使用界面；您可以只使用 if (animal is Bird) 来代替。但这样做是更好的做法；鸟类并不是唯一会飞的动物，因此您要根据您的物品用途来做出决定，而不是它是什么。这就是接口的用途。