避免Java中不同枚举值的多if-else条件

Question

我在枚举中有6个值，使用6 if else是一种很糟糕的做法。我们能以更好的方式实现这一点吗？下面是我的场景：

ExampleEnum value = getEnumValue();

if(ExampleEnum.A == value){
   doA();
}else if(ExampleEnum.B == value){
   doB();
}else if(ExampleEnum.C == value){
   doC();
}else if(ExampleEnum.D == value){
   doD();
}else if(ExampleEnum.E == value){
   doE();
}else if(ExampleEnum.F == value){
   doF();
}

我在考虑switch，但并没有太大的区别，我还需要根据某些参数在doA（）内返回一个布尔值。

提前谢谢。

Answer 1

可以在枚举内添加do方法。

public enum ExampleEnum {
    A {
        public void doIt() { ... }
    },
    B {
        public void doIt() { ... }
    },
    ...

    abstract public void doIt();

}

ExampleEnum value = getEnumValue();
if (value != null) {
    value.doIt();
}

Answer 2

如果您想使用开关语句，并且您在Java12或更新版本，请考虑使用扩展开关表达式来避免中断语句的陷阱：

java prettyprint-override">switch (value) {
    case A -> doA();
    case B -> doB();
    case C -> doC();
    case D -> doD();
    case E -> doE();
    case F -> doF();
}

Answer 3

您有几个选择：

保持代码不变。读写困难。

switch (value) {
    case A:
        doA();
        break;
    case B:
        doB();
        break;
    case C:
        doC();
        break;
    case D:
        doD();
        break;
    case E:
        doE();
        break;
    case F:
        doF();
        break;
}

请注意，这是经典开关。如果您可以访问较新的Java版本，则可能可以摆脱中断s。

您还可以创建EnumMap：

EnumMap<ExampleEnum, Runnable> enumMap = new EnumMap<>(Map.<ExampleEnum, Runnable>of(
    ExampleEnum.A, Main::doA, // 'Main', or wherever your do* methods are.
    ExampleEnum.B, Main::doB,
    ExampleEnum.C, Main::doC, // I'm using method references. But you could
    ExampleEnum.D, Main::doD, // also use lambda expressions: '() -> doD()'.
    ExampleEnum.E, Main::doE,
    ExampleEnum.F, Main::doF
));

ExampleEnum value = getEnumValue();
enumMap.get(value).run();