重写一个if语句，以更干净的方式抛出异常

Question

我们假设我们有一个这样的if语句：

public A save(A a) {
    if (isValid.test(a)) {
        return aRepository.save(a);
    }
    throw new ANotValidException("A is not valid");
}

isValid是一个谓词，它可能看起来像：

private Predicate<A> isValid = (a) -> (a != null);

你怎么看？我可以以某种方式使它更干净吗？我的意思是，例如使用Optional用.orElseThrow();将其减少到1行

Answer 1

使用Optional和throwing自定义Exception的更精确版本应为：

public A save(A a) throws ANotValidException { // throws the custom exception
    return Optional.ofNullable(a) // since your predicate is to check for not null 
                   .map(aRepository::save)
                   .orElseThrow(() -> new ANotValidException(a + "A is not valid"));
}

Answer 2

Optional可以使代码更具可读性，尤其是在使用谓词对象时：

public A save(A a) {
    return Optional.ofNullable(a)
            .filter(isValid)
            .map(aRepository::save)
            .orElseThrow(() -> new ANotValidException("A is not valid"));
}

你也可以完全摆脱谓词，因为它很简单，可以使用Objects::nonNull（除非你的真实谓词的测试更复杂）。在这种情况下，保持当前的状态检查可能更有意义（在我看来）。

Answer 3

有人可能会争辩说，以相反的顺序读取它会更自然，首先处理验证及其结果，然后继续保存对象。

public A save(A a) {
    if (!isValid.test(a)) {
        throw new ANotValidException("A is not valid");
    }

    return aRepository.save(a);
}