java - 如何在“if”语句中编写干净的代码？

Question

什么会更清楚，为什么？（使用 Apache Commons 库）

在下面的代码中，逻辑是相同的：

if(!ArrayUtils.isEmpty(someArray)) {
  some code here ...
}

//or this variant
if(ArrayUtils.isNotEmpty(someArray)) {
  some code here ...
}

非常感谢！

Answer 1

1. 第二个（使用 isNotEmpty）读起来像简单的英语，这是它的主要优点。
2. 如果您使用的库提供了您需要的确切功能而不是您应该使用它（在您的示例中就是这种情况）。
3. 在您的 ArrayUtils 的特定情况下，检查数组是否不为空或 null 非常常见，以至于 API 设计人员建议为其添加单独的函数。

任何傻瓜都可以编写计算机可以理解的代码。优秀的程序员编写人类可以理解的代码。——马丁·福勒

Answer 2

我投票给选项#2。它更容易阅读。很明显，Commons 添加了此方法正是为了不使用选项 #1。

Answer 3

我也认为选项 2 更好，因为它更具可读性。我看不出这两个选项有任何性能差异。