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

Что будет более понятно и почему? (С использованием библиотеки Apache Commons)Как написать чистый код в инструкции «if»?

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

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

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

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

источник

2013-06-10 Dima Zelinskyi

Задайте это на CodeReview. – mwerschy

Второй вариант - более чистый. Чистый код не должен содержать '!' Или '! ='. – darijan

Да для меня это также более понятно с использованием второго варианта, но в соответствии с правом названия названия java без имени. http://geosoft.no/development/javastyle.html#Specific –

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

источник

2013-06-10 09:56:17

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

источник

2013-06-10 10:06:13

вопрос был не о производительности, а о чистоте i.e. удобочитаемости;) –

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

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

источник

2013-06-10 10:11:35 ybodnar

Продолжить: Код будет читаться еще много раз, чем написано. Напишите для удобочитаемости. –

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

ответ

Смежные вопросы