避免简单的 if/else 条件 [重复]

Question

在我的程序中，我需要检查变量是否等于1、2或3，并根据结果执行不同的方法：

if (phase.equals("1")) {
    PhaseOne.performPhase(inputParser.getSource(), inputParser.getTarget());
} else if (phase.equals("2")) {
    PhaseTwo.performPhase(inputParser.getSource(), inputParser.getTarget());
} else {
    PhaseThree.performPhase(inputParser.getSource(), inputParser.getTarget());
}

这段代码非常简单和基本，但我真的不喜欢它。当然，我可以使用开关条件，但以我的拙见，它只会以不同的方式显示相同的基本功能。

我的问题是:有没有一种优雅的、可扩展的方式来实现功能？

仅供参考，我已经红了这篇文章，但我没有找到适合我问题的答案。

Answer 1

另一种选择是为每个阶段创建一个类，并为它们创建一个I阶段接口以供实现。创建一个列表

public interface IPhase {
    public void performPhase();
    public String getId();
}

for (IPhase phase : phasesList){
    if (phase.equals(phase.getId())){
        phase.performPhase();
        // either break or continue the loop
    }
}

Answer 2

如果您的PhaseOne/PhaseTwo/PhaseThree类都实现了相同的接口（比方说 performPhase，那么您可以执行以下操作：

final Phase targetPhase;
switch(phase) {
    case "1": targetPhase = myInstanceOfPhaseOne; break;
    case "2": targetPhase = myInstanceOfPhaseTwo; break;
    case "3": targetPhase = myInstanceOfPhaseThree; break;
    default: throw new IllegalStateException("Unrecognised phase "+phase);
}
targetPhase.performPhase(inputParser.getSource(), inputParser.getTarget()));

Answer 3

在我看来，关于您的链接问题的公认答案非常适合您。在地图中存储对函数的引用：

Map<String,BiConsumer<T,U>> map = new HashMap<>();
map.put("1",PhaseOne::performPhase);
map.put("2",PhaseTwo::performPhase);
map.put("3",PhaseThree::performPhase);
map.get(phase).accept(inputParser.getSource(), inputParser.getTarget());

将 T 和 U 替换为 inputParser.getSource（） 和 inputParser.getTarget（） 的类型。

使用这种方法，Phase…类不需要公共的超类或接口。