Since Checkstyle 3.1
Implements Joshua Bloch, Effective Java, Item 17 - Use Interfaces only to define types.
According to Bloch, an interface should describe a type. It is therefore inappropriate to define an interface that does not contain any methods but only constants. The Standard interface javax.swing.SwingConstants is an example of an interface that would be flagged by this check.
The check can be configured to also disallow marker interfaces like
java.io.Serializable
, that do not contain methods or
constants at all.
name | description | type | default value | since |
---|---|---|---|---|
allowMarkerInterfaces | Control whether marker interfaces like Serializable are allowed. | boolean | true |
3.1 |
To configure the check:
<module name="Checker"> <module name="TreeWalker"> <module name="InterfaceIsType"/> </module> </module>
Example:
public interface Test1 { // violation int a = 3; } public interface Test2 { // OK } public interface Test3 { // OK int a = 3; void test(); }
To configure the check to report violation so that it doesn't allow Marker Interfaces:
<module name="Checker"> <module name="TreeWalker"> <module name="InterfaceIsType"> <property name="allowMarkerInterfaces" value="false"/> </module> </module> </module>
Example:
public interface Test1 { // violation int a = 3; } public interface Test2 { // violation }
All messages can be customized if the default message doesn't suit you. Please see the documentation to learn how to.
com.puppycrawl.tools.checkstyle.checks.design