如何在不违反Liskov替换原则的情况下有效地在类之间共享函数

Question

我有一个代码库，它最初是用许多不同的选项创建的，这些选项允许您让代码以稍微不同的方式执行相同的过程，如下所示：

public class MainFunction {
  public void main(String option){
     if (option.equals("vanilla mode")){
        this.firstFunction();
     }else{
        this.differentVersionOfFirstFunction();
     }
     this.secondFunction();
  }

  public void firstFunction(){
    //First functions code
  }

  public void secondFunction(){
     //Second functions code
  }

  public void differentVersionOfFirstFunction(){
     // different code from First functions code that produces the same type of end result using a slightly different method
  }
}

随着各种不同选项的添加和代码变得越来越复杂，这变得越来越复杂。

为了解决这个问题，我最初计划创建一个父对象，然后在需要时，可以在父对象方法上有不同细微变化的子对象。问题是，我知道这会违反Liskov替代原则，事实上，我可能有孩子应该使用他们父母不使用的相同方法。

所以我只剩下在同一个类对象中使用不同的方法，以稍微不同的方式完成相同的工作。

public class MainFunction {
  public void main1(){
    this.firstFunction();
    this.secondFunction();
  }

  public void main2(){
    this.differentVersionOfFirstFunction();
    this.secondFunction();
  }

  public void firstFunction(){
    //First functions code
  }

  public void secondFunction(){
    //Second functions code
  }

  public void differentVersionOfFirstFunction(){
    // different code from First functions code that produces the same type of end result using a slightly different method
  }
}

我想我可以创建一个单独的实用程序类来保存所有的函数，但我不确定是否有更优雅的解决方案？

Answer 1

也许您可以尝试使用策略模式，在main函数中注入每次都需要使用的策略对象。看看这里

Answer 2

我看不出你的例子是如何违反Liskov替换原则的。然而，我看到的是，它们可能违反了打开/关闭原则，这意味着每次需要修改程序时，都要编辑一些main函数。java文件，有可能影响运行程序的所有可能情况。一个更好的解决方案是拥有大量解耦组件，这样当您需要修改某些内容时，您只修改一小部分内容，而这不太可能影响程序可能遇到的所有场景。这就是单一责任原则。

正如在另一个答案中提到的，您的场景似乎非常适合应用策略模式。这可能如下所示：

>

使用void main（）方法创建接口main函数，不带任何选项

使用public abstract void main（）成员创建一个抽象策略类，例如AbstractMainFunction，不带任何选项。此类将实现main函数接口

根据需要创建abstractmain函数的单独实现，例如vanillamodemain函数和differentmain函数。您可能会发现在abstractmain函数中保留一些共享代码非常有用

创建策略切换器类，例如MainFunctionService。它将有一个方法，public void main（String选项），如第一个示例所示，并且可能会有这样一个switch语句：

MainFunction strategy = defaultFunction;
switch(option) {
    case "vanilla":
         strategy = vanillaFunction;
         break;
    case "different":
         strategy = differentFunction;
         break;
}
strategy.main();

看起来有很多事情要做，但最后您将看到这是如何真正简化维护和进一步开发的。