checkstyle запретить аннотацию SuppressWarnings, если рядом нет комментария - PullRequest
Новая
       63

checkstyle запретить аннотацию SuppressWarnings, если рядом нет комментария

0 голосов
/ 27 февраля 2019

В нашем проекте нам иногда приходится подавлять некоторые предупреждения (например, «WeakerAccess» может быть подавлен, так как проект также используется как библиотека в другом проекте, или «выражение всегда ложно» для instanceof проверенное исключение, котороесоздается из библиотеки, которая маскирует факт выдачи этого исключения).

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

Но я не могу найти способ сделать это.Есть этот блок: 

<module name="SuppressWarnings">
  <property name="format"
      value="^unchecked$|^unused$"/>
  <property name="tokens"
    value="
    CLASS_DEF,INTERFACE_DEF,ENUM_DEF,
    ANNOTATION_DEF,ANNOTATION_FIELD_DEF,
    ENUM_CONSTANT_DEF,METHOD_DEF,CTOR_DEF
    "/>
</module>

и кое-что о специальных комментариях, чтобы отключить checkstyler для строки, но это просто еще одна вещь подавления предупреждений, которая также нуждается в объяснении ... Но есть лиспособ сказать, что подавление в порядке, если рядом есть какой-либо комментарий (в строке до или в той же строке)?

1 Ответ

0 голосов
/ 28 февраля 2019

Я рекомендую использовать 2 чека в унисон.Используйте SuppressWarningsCheck , чтобы пометить методы, которые вы хотите документировать, и отобразить сообщение об ошибке, в котором говорится, что это нарушение, поскольку оно не задокументировано.Затем используйте SuppressWithNearbyCommentFilter , чтобы подавить нарушения другой проверки при добавлении документации.Чтобы фильтр работал, документация должна начинаться с определенного текста, чтобы он не ложно подавлял SuppressWarnings, у которого на самом деле нет документации.

Пример: 

$ cat TestClass.java
public class TestClass {
    //SuppressWarnings: this is my reason for the suppression
    @SuppressWarnings("unchecked")
    void method() {
    }

    //this is just a comment and not a reason
    @SuppressWarnings("unused")
    void method2() {
    }

    @SuppressWarnings("unused")
    void noComment() {
    }
}

$ cat TestConfig.xml
<?xml version="1.0"?>
<!DOCTYPE module PUBLIC
          "-//Puppy Crawl//DTD Check Configuration 1.3//EN"
          "http://www.puppycrawl.com/dtds/configuration_1_3.dtd">

<module name="Checker">
    <property name="charset" value="UTF-8"/>

    <module name="TreeWalker">
    <module name="SuppressWarnings">
        <property name="format" value="^(unchecked|unused)$"/>
        <message key="suppressed.warning.not.allowed"
             value="The warning ''{0}'' cannot be suppressed at this location unless a comment is given for the reason for the suppression." />
        <property name="tokens" value="CLASS_DEF,INTERFACE_DEF,ENUM_DEF,ANNOTATION_DEF,ANNOTATION_FIELD_DEF,ENUM_CONSTANT_DEF,METHOD_DEF,CTOR_DEF"/>
    </module>
    <module name="SuppressWithNearbyCommentFilter">
      <property name="commentFormat"
                value="SuppressWarnings: .{10,}"/>
      <property name="checkFormat" value="SuppressWarnings"/>
      <property name="influenceFormat" value="3"/>
    </module>
    </module>
</module>

$ java -jar checkstyle-8.18-all.jar -c TestConfig.xml TestClass.java
Starting audit...
[ERROR] TestClass.java:8:23: The warning 'unused' cannot be suppressed at this location unless a comment is given for the reason for the suppression. [SuppressWarnings]
[ERROR] TestClass.java:12:23: The warning 'unused' cannot be suppressed at this location unless a comment is given for the reason for the suppression. [SuppressWarnings]
Audit done.
Checkstyle ends with 2 errors.

You 'Вы заметите, что есть 2 нарушения, но 3 подавления предупреждений.Первый пример показывает, как правильно подавить отсутствие документации.2-й показывает только комментарий, но не документацию по подавлению, а 3-й показывает отсутствие комментариев вообще.

<property name="format" value="^(unchecked|unused)$"/>

Это указывает, что только документация будеттребуется для неконтролируемых и неиспользованных подавлений.Если вам нужна документация для всех типов, кроме этих 2, я рекомендую выражение "^((?!unchecked|unused).)*$".

...