编程语言
首页 > 编程语言> > java-严格省略单行大括号是否正确?

java-严格省略单行大括号是否正确?

作者:互联网

我个人反对在if-else语句中省略花括号,而我完全明白为什么应该避免使用它.

但是,现在我遇到了一个有趣的用例,这里是示例代码:

public <E extends RuntimeException> void throwOnFail(final boolean result, final Supplier<E> exceptionSupplier) throws E {
    Objects.requireNonNull(exceptionSupplier);
    if (result) return;
    throw exceptionSupplier.get();
}

我个人认为这段代码是:

>尽可能简洁,将在下面显示其他变体.
>不容易受到添加行会更改代码逻辑的问题的影响.

我将自己设置为仅在控制流语句中使用的个人规则.
所以实际上这意味着返回,中断和继续.

该代码的两个替代版本如下所示.

选择1

public <E extends RuntimeException> void throwOnFail(final boolean result, final Supplier<E> exceptionSupplier) throws E {
    Objects.requireNonNull(exceptionSupplier);
    if (result) {
        return;
    }
    throw exceptionSupplier.get();
}

选择2

public <E extends RuntimeException> void throwOnFail(final boolean result, final Supplier<E> exceptionSupplier) throws E {
    Objects.requireNonNull(exceptionSupplier);
    if (!result) {
        throw exceptionSupplier.get();
    }
}

我要说的是,它们都使代码看起来更加复杂,没有任何理由.

解决方法:

Is it considered correct to omit curly braces strictly on one-liners?

好吧,这没有硬性规定.但是考虑到可用性的情况,我总是使用大括号.它使代码更具可读性,并且对于初级开发人员来说很容易理解.同样,这纯粹是个人/公司(代码标准)的选择.

再次,在您的替代方案中,我将选择

public <E extends RuntimeException> void throwOnFail(final boolean result, final Supplier<E> exceptionSupplier) throws E {
    Objects.requireNonNull(exceptionSupplier);
    if (!result) {
        throw exceptionSupplier.get();
    }
}

为什么?

>更简洁
>行数少
>逻辑上直截了当

标签:if-statement,coding-style,java
来源: https://codeday.me/bug/20191121/2054955.html