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

2019-11-21 22:05:38 阅读：317 来源： 互联网

我个人反对在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

本站声明： 1. iCode9 技术分享网（下文简称本站）提供的所有内容，仅供技术学习、探讨和分享；
2. 关于本站的所有留言、评论、转载及引用，纯属内容发起人的个人观点，与本站观点和立场无关；
3. 关于本站的所有言论和文字，纯属内容发起人的个人观点，与本站观点和立场无关；
4. 本站文章均是网友提供，不完全保证技术分享内容的完整性、准确性、时效性、风险性和版权归属；如您发现该文章侵犯了您的权益，可联系我们第一时间进行删除；
5. 本站为非盈利性的个人网站，所有内容不会用来进行牟利，也不会利用任何形式的广告来间接获益，纯粹是为了广大技术爱好者提供技术内容和技术思想的分享性交流网站。

ICode9

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