Как написать чистый код в инструкции «if»?

Question

Что будет более понятно и почему? (С использованием библиотеки Apache Commons)

В коде ниже логики тот же:

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

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

Большое спасибо!

Answer 1

Я голосую за вариант №2. Его легче читать. Понятно, что Commons добавила этот метод точно для того, чтобы не использовать параметр №1.

Answer 2

Я также думаю, что вариант 2 лучше, так как он более читабельен. Я не вижу разницы в производительности в этих двух вариантах.

Answer 3

1. Второй (с использованием isNotEmpty) читается как простой английский, что является главным его преимуществом.
2. Если библиотека, которую вы используете, обеспечивает точную функциональность, которая вам нужна, чем вы должны ее использовать (это в вашем примере).
3. В вашем конкретном случае ArrayUtils проверка того, что массив не пуст или null, является настолько обычным случаем, что дизайнер API предложил добавить к нему отдельную функцию.

Любой дурак может написать код, который может понять компьютер. Хорошие программисты пишут код, который люди могут понять. - Martin Fowler