在不违反 SOLID 原则的情况下使用具有不同方法名称的子类

Question

//接口

public interface Singer{
    void sing();
}
public interface SongWriter{
    void writeSong();
}

//实现

public class PureSinger implements Singer{
    void sing(){}
}
public class SingerSongWriter implements Singer, SongWriter{
    void sing(){}
    void writeSong(){}
}

//客户端代码

void methodA(){
    Singer objPureSinger = new PureSinger();
    Singer objSingerSWer = new SingerSongWriter();

    doSomething(objPureSinger);
    doSomething(objSingerSWer);
}

public void doSomething(Singer obj){
    obj.sing();
    obj.writeSong();    //<--- this does not work.
}

为了实现这种类型的代码，我应该如何设计类结构？

Answer 1

为了实现这种类型的代码，我应该如何设计类结构？

首先定义一个名为Artist 的interface，它有一个名为perform 的方法。接下来，定义两个子类，即Singer 和SongWriter，每个子类都实现perform，并分别包含唱歌和写歌的代码。

以下两种设计模式可以很好地支持您的用例：

1. Decorator 设计模式：在您的情况下，Artist 是 组件，Singer 和 SongWriter 是 具体组件，ArtistDecorator 是你的 Abstract Decorator 类。使用ArtistDecorator 将Singer 包装成SongWriter。 doSomething 方法接受一个 Artist 参数，并传递最终装饰的 Artist 对象，该对象将 Singer 包装到 SongWriter 中。
2. Composite 设计模式：在您的情况下，Arist 是 组件。 CompositeArtist 是 Composite，Singer 和 SongWriter 是 Leaf 或 Concrete Component 类。 doSomething 方法采用 Artist 参数。将CompositeArtist 的实例传递给它（IS-A Artist）和doSomething 只是调用perform，它在内部迭代所有Artist 实例调用它们的perform 方法逐个。

Answer 2

要快速回答问题doSomething 调用sing() 和writeSong() 方法。只有你的 SingerSongWritter 类定义了这两个，所以是唯一可以同时做这两个的类。 objPureSinger 没有定义 writeSong 是什么，因此不知道调用时要做什么。

如果你想保持这个结构，你可以做一件事来解决这个问题，即在两个类中实现一个名为do 的方法。在您的PureSinger 类中，do 只是简单地调用sing()，但在您的SingerSongWriter 类中它同时调用sing() 和writeSong()。最后在你的doSomething 方法中调用obj.do();

Answer 3

您可以根据自己的业务需要设计课程。而且，很明显会在

处得到编译时错误

public void doSomething(Singer obj){
    obj.sing();
    obj.writeSong();    //<--- compiler will complain here
}

因为，Singer 没有 writeSong 方法。根据您当前的设计：

1. 所有 Singer 仅 CAN sing。
2. 所有 SongWriter 仅限 CAN writeSong。

但是，以上都不能同时执行。所以，很明显doSomething 直到

doSomething (SingerSongWriter obj)

因为，只有SingerSongWriter 可以同时执行。

Answer 4

您不应该假设在 doSomething() 中使用 .writeSong()，因为您将参数作为 Singer 传递，而且我们知道 Singer 只有一个名为 .sing() 的方法。这里有一些选项。

1. 您应该考虑 doSomething() 在业务逻辑意义上的实际作用。也许我们可以不用调用 .writeSong()，因为传入的参数并不总是可以转换为 SongWriter。
2. 考虑使用基类（例如抽象类或具体类）来实现 Singer 和 SongWriter 以将其作为参数数据类型传入。
3. 如果您正在寻找快速修复以使您的代码运行，请参见下文。

快速修复： 您需要首先查看 obj 是否为 instanceOf SongWriter，因为传入的参数是 Singer 类型，它没有名为 .writeSong() 的方法。

public void doSomething(Singer obj){
    obj.sing();
    //obj.writeSong();    //<--- this does not work.
    if (obj instanceOf SongWriter) songWriter.writeSong();
}

这是一个快速修复。您可以从下面的帖子中看到，您应该最后使用 instanceOf。

• Java generic to avoid ClassCastException
• Is instanceof considered bad practice? If so, under what circumstances is instanceof still preferable?