Checkstyle EmptyLineSeparator 在类减速之前不会在多个空行上失败

Question

为什么 Checkstyle 不将语句之后import但class声明之前的空行识别为违规？

package b.pi.persistence.hibernate;

import java.lang.String;



// ^^^ much space! Why is the above not a violation of EmptyLineSeparator   
public class CheckstyleExample {

  ...
}

声明以下检查：

<module name="EmptyLineSeparator">
  <property name="allowNoEmptyLineBetweenFields" value="true"/>
  <property name="allowMultipleEmptyLines" value="false"/>
</module>

Answer 1

这是因为缺乏测试和代码问题。

通常，此检查会验证某些标记上方是否不存在超过一个空行。import当或上方有多个不必要的空格时，它效果很好package。当检查到达class声明时，它也会验证上面的行，但是......

• 在EmptyLineSeparatorCheck.java#L214行处，为 token 获取了下一个同级class，它是null，因为在文件中与此类处于同一级别上有其他注释
• 因为下一个兄弟是在EmptyLineSeparatorCheck.java#L216null行跳过整个验证

此错误已作为checkstyle/checkstyle#2067提交，希望修复将在 Checkstyle 6.11 中发布。