0

在我们的项目中，我们有时必须抑制一些警告（例如，“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 的特殊注释的东西，但这只是另一个抑制警告的事情，也需要解释......但是如果附近有任何评论，有没有办法说抑制是可以的（在同一行之前或同一行上）？

4

1 回答 1

1

我建议同时使用 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.

您会注意到有 2 个违规行为，但有 3 个 SuppressWarnings。第一个示例显示了如何正确抑制没有文档。第 2 个仅显示评论，但没有显示有关压制的文档，第 3 个根本没有显示任何评论。

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

这指定了未经检查和未使用的抑制只需要文档。如果您想要除这 2 种以外的所有类型的文档，我推荐使用表达式"^((?!unchecked|unused).)*$"。

于 2019-02-28T12:54:52.070 回答