我不明白区别

Question

我正在用这一行检查一些 Java 代码

if ( list.size() == 1 || list.size() <= 4) {...}

我评论说我看不出这与做有什么不同

if (list.size() <= 4) {...}

他说这很重要，需要成为第一个。我不明白。也许如果它是类似的东西

if (list.size() == 1 || list.size() <= someVeryCostlyFunction() ) {...}

如果 someVeryCostlyFunction() 总是返回某个正数 >= 1，那么在大多数情况下，大小预计为 1。否则我看不出区别。我们应该检查效率。

Answer 1

代码smells满足这两个条件：

if (list.size() == 1 || list.size() <= 4)

也许作者是这么想的：

如果列表中有 4 个或更少的元素。

即使列表有零个元素，当前代码的更多内容也允许满足条件 - 这很可能是错误的。

这个条件的另一个问题是使用magic number 4？
它有什么如此重要，为什么它不是5？ 它应该是自我记录的，并与可能出现在代码中的其他 4 区分开来：

int MAX_HANDLED = 4;
if ( list.size() > 0 && list.size() <= MAX_HANDLED )
:
:
int ALL_TIRES = 4;
if (car.getTires() < ALL_TIRES) {
  car.stop();
}

至于性能，我认为现有条件应该比您提出的条件更快（即使是第二个条件在list.size > 1 时也会更快），我看不出有任何重要原因。请参阅此 question 类似问题。