如何优化返回布尔值或抛异常的Switch Case？代码实践及优化方案咨询

这段代码属于合格但有优化空间的实践：

• 优点：严格遵循fail-fast原则，对未定义的状态（如4）主动抛出异常，避免隐性逻辑错误；分支逻辑清晰，明确覆盖了所有预期的状态值。
• 不足：硬编码的数字缺乏语义，后续维护时需对照规则才能理解每个数字的含义；如果状态值有增减，需要修改分支代码，扩展性一般。

你提到的if (1,2,3) {true}, else {false}是伪代码，实际Java中无法直接这样写，即使改成合法的条件判断（比如if (status ==1 || status ==2 || status ==3)），也容易漏掉对未知状态的校验，而原switch的default分支正好补上了这个关键校验，这也是原写法的核心优势之一。

1. 用枚举类封装状态（推荐） #

把状态值和对应的逻辑结果封装成枚举，赋予每个状态明确的语义，同时统一处理状态查找和异常抛出：

enum Status {
    ACTIVE(1, true),
    PENDING(2, true),
    PROCESSING(3, true),
    CANCELLED(5, false),
    REJECTED(6, false),
    EXPIRED(7, false);

    private final int code;
    private final boolean isBlahTrue;

    Status(int code, boolean isBlahTrue) {
        this.code = code;
        this.isBlahTrue = isBlahTrue;
    }

    public static Status fromCode(int code) {
        for (Status status : values()) {
            if (status.code == code) {
                return status;
            }
        }
        throw new MyAppRuntimeException("Status unknown!");
    }
}

private boolean isBlahTrue(final BigDecimal status) {
    return Status.fromCode(status.intValue()).isBlahTrue;
}

这种写法完全消除了硬编码的魔法数字，语义清晰，后续新增状态只需要在枚举中添加即可，维护成本极低。

2. 使用命名清晰的常量集合 #

把对应true/false的状态值放到命名明确的Set中，通过集合的contains方法判断，同时保留未知状态的校验：

private static final Set<Integer> BLAH_TRUE_STATUSES = Set.of(1, 2, 3);
private static final Set<Integer> BLAH_FALSE_STATUSES = Set.of(5, 6, 7);

private boolean isBlahTrue(final BigDecimal status) {
    int statusCode = status.intValue();
    if (BLAH_TRUE_STATUSES.contains(statusCode)) {
        return true;
    }
    if (BLAH_FALSE_STATUSES.contains(statusCode)) {
        return false;
    }
    throw new MyAppRuntimeException("Status unknown!");
}

这种写法比原switch更简洁，集合的命名也能传递语义，适合状态值较少且不需要额外属性的场景。

3. 用Java 12+的Switch表达式简化写法 #

如果使用较新的Java版本，可以用switch表达式替代传统switch，让代码更紧凑：

private boolean isBlahTrue(final BigDecimal status) {
    return switch (status.intValue()) {
        case 1, 2, 3 -> true;
        case 5, 6, 7 -> false;
        default -> throw new MyAppRuntimeException("Status unknown!");
    };
}

这种写法保留了原switch的清晰分支，同时语法更简洁，适合不想引入枚举或集合的轻量优化场景。

内容的提问来源于stack exchange，提问作者user20206699