я не понимаю разницу

Question

Я просматривал некоторые Java-код с этой линией

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

Я заметил, что я не вижу, как это не отличается от только что делать

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

он сказал это важно и должно быть первым. Я не понимаю. Может быть, если бы это было что-то вроде

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

и размер должен был быть 1 большую часть времени вы можете использовать оба, если someVeryCostlyFunction() всегда возвращает некоторое положительное число> = 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 аналогичной озабоченности.