Почему правила PMD

В каждом случае правило может быть делом конкретных обстоятельств или просто «вкусом».

Следует избегать создания экземпляра объекта в цикле, если существует большое количество итераций, а создание экземпляра дорого. Если вы можете вывести код из цикла, вы избежите множества экземпляров объектов и, следовательно, улучшите производительность. Сказав это, это не всегда возможно, а в некоторых случаях это просто не имеет значения для общей производительности кода. В этих случаях делайте то, что яснее.

У OnlyOneReturn есть несколько способов взглянуть на это (за каждым из них стоят яростные сторонники), но все они в основном сводятся к вкусу.

В вашем примере сторонникам OnlyOneReturn нужен такой код:

public int performAction(String input) {
    int result;
    if (input.equals("bob")) {
        result = 1;
    } else {
        result = 2;
    }
    return result;
}

Скорее, чем:

public int performAction(String input) {
    if (input.equals("bob")) {
        return 1;
    } else {
        return 2;
    }
}

Как видите, дополнительную ясность ReturnOnlyOnce можно обсуждать.

Также см. этот вопрос SO, который относится к создание экземпляров внутри циклов.

Эта статья Сравнение инструментов поиска ошибок для Java, "Ник Рутар, Кристиан Алмазан и Джефф Фостер, сравнивает несколько средств проверки ошибок для Java..." — Документы и публикации FindBugs. PMD выглядит более многословным.

Приложение: Как предполагают авторы,

«Все инструменты выбирают разные компромиссы между генерированием ложных срабатываний и ложных отрицаний».

В частности, AvoidInstantiatingObjectsInLoops может вообще не быть ошибкой, если такова цель. Он включен, чтобы помочь избежать создания ненужных объектов< /а>. Точно так же OnlyOneReturn носит наводящий характер. Множественные возвраты представляют собой форму перехода, иногда считающуюся вредной, но разумно используется для улучшения читаемости.

Меня больше всего раздражают люди, которые предписывают использование таких инструментов, не понимая, что такое ложные срабатывания.

Как отмечено здесь, более поздние версии PMD поддерживает улучшенную настройку при интеграции в процесс сборки.

Вы можете заглянуть на домашнюю страницу PMD, правила подробно объясняются здесь и часто с пояснениями. Сайт структурирован для групп правил, здесь ссылка на основные правила: http://pmd.sourceforge.net/rules/basic.html

Каждое правило находится в наборе правил PMD, который может дать вам подсказку к обоснованию правила (если оно не объяснено подробно на самой странице набора правил).

В случае AgainstInstantiatingObjectsInLoops повторное создание экземпляра одного и того же объекта может быть дорогостоящим. Однако часто это необходимо. В моем собственном проекте я отключил это правило, так как оно помечает слишком много ложных срабатываний.

В случае OnlyOneReturn обратите внимание, что он находится в наборе правил под названием спорный, который намек на то, что эти правила спорны и зависят от случая. Я также отключил весь этот набор правил.