Class AvoidStarImportCheck
- All Implemented Interfaces:
Configurable,Contextualizable
Checks that there are no import statements that use the * notation.
Rationale: Importing all classes from a package or static members from a class leads to tight coupling between packages or classes and might lead to problems when a new version of a library introduces name clashes.
Note that property excludes is not recursive, subpackages of excluded
packages are not automatically excluded.
-
Property
excludes- Specify packages where star imports are allowed. Type isjava.lang.String[]. Default value is"". -
Property
allowClassImports- Control whether to allow starred class imports likeimport java.util.*;. Type isboolean. Default value isfalse. -
Property
allowStaticMemberImports- Control whether to allow starred static member imports likeimport static org.junit.Assert.*;. Type isboolean. Default value isfalse.
To configure the check:
<module name="AvoidStarImport"/>
Example:
import java.util.Scanner; // OK import java.io.*; // violation import static java.lang.Math.*; // violation import java.util.*; // violation import java.net.*; // violation
To configure the check so that star imports from packages
java.io and java.net as well as static members from class
java.lang.Math are allowed:
<module name="AvoidStarImport"> <property name="excludes" value="java.io,java.net,java.lang.Math"/> </module>
Example:
import java.util.Scanner; // OK import java.io.*; // OK import static java.lang.Math.*; // OK import java.util.*; // violation import java.net.*; // OK
To configure the check so that star imports from all packages are allowed:
<module name="AvoidStarImport"> <property name="allowClassImports" value="true"/> </module>
Example:
import java.util.Scanner; // OK import java.io.*; // OK import static java.lang.Math.*; // violation import java.util.*; // OK import java.net.*; // OK
To configure the check so that starred static member imports from all packages are allowed:
<module name="AvoidStarImport"> <property name="allowStaticMemberImports" value="true"/> </module>
Example:
import java.util.Scanner; // OK import java.io.*; // violation import static java.lang.Math.*; // OK import java.util.*; // violation import java.net.*; // violation
Parent is com.puppycrawl.tools.checkstyle.TreeWalker
Violation Message Keys:
-
import.avoidStar
- Since:
- 3.0
-
Nested Class Summary
Nested classes/interfaces inherited from class com.puppycrawl.tools.checkstyle.api.AutomaticBean
AutomaticBean.OutputStreamOptions -
Field Summary
FieldsModifier and TypeFieldDescriptionprivate booleanControl whether to allow starred class imports likeimport java.util.*;.private booleanControl whether to allow starred static member imports likeimport static org.junit.Assert.*;.Specify packages where star imports are allowed.static final StringA key is pointing to the warning message text in "messages.properties" file.private static final StringSuffix for the star import. -
Constructor Summary
Constructors -
Method Summary
Modifier and TypeMethodDescriptionint[]The configurable token set.int[]Returns the default token a check is interested in.int[]The tokens that this check must be registered for.private voidlogsStarredImportViolation(DetailAST startingDot) Gets the full import identifier.voidsetAllowClassImports(boolean allow) Setter to control whether to allow starred class imports likeimport java.util.*;.voidsetAllowStaticMemberImports(boolean allow) Setter to control whether to allow starred static member imports likeimport static org.junit.Assert.*;.voidsetExcludes(String... excludesParam) Setter to specify packages where star imports are allowed.voidvisitToken(DetailAST ast) Called to process a token.Methods inherited from class com.puppycrawl.tools.checkstyle.api.AbstractCheck
beginTree, clearViolations, destroy, finishTree, getFileContents, getLine, getLineCodePoints, getLines, getTabWidth, getTokenNames, getViolations, init, isCommentNodesRequired, leaveToken, log, log, log, setFileContents, setTabWidth, setTokensMethods inherited from class com.puppycrawl.tools.checkstyle.api.AbstractViolationReporter
finishLocalSetup, getCustomMessages, getId, getMessageBundle, getSeverity, getSeverityLevel, setId, setSeverityMethods inherited from class com.puppycrawl.tools.checkstyle.api.AutomaticBean
configure, contextualize, getConfiguration, setupChild
-
Field Details
-
MSG_KEY
A key is pointing to the warning message text in "messages.properties" file.- See Also:
-
STAR_IMPORT_SUFFIX
Suffix for the star import.- See Also:
-
excludes
Specify packages where star imports are allowed. -
allowClassImports
private boolean allowClassImportsControl whether to allow starred class imports likeimport java.util.*;. -
allowStaticMemberImports
private boolean allowStaticMemberImportsControl whether to allow starred static member imports likeimport static org.junit.Assert.*;.
-
-
Constructor Details
-
AvoidStarImportCheck
public AvoidStarImportCheck()
-
-
Method Details
-
getDefaultTokens
public int[] getDefaultTokens()Description copied from class:AbstractCheckReturns the default token a check is interested in. Only used if the configuration for a check does not define the tokens.- Specified by:
getDefaultTokensin classAbstractCheck- Returns:
- the default tokens
- See Also:
-
getAcceptableTokens
public int[] getAcceptableTokens()Description copied from class:AbstractCheckThe configurable token set. Used to protect Checks against malicious users who specify an unacceptable token set in the configuration file. The default implementation returns the check's default tokens.- Specified by:
getAcceptableTokensin classAbstractCheck- Returns:
- the token set this check is designed for.
- See Also:
-
getRequiredTokens
public int[] getRequiredTokens()Description copied from class:AbstractCheckThe tokens that this check must be registered for.- Specified by:
getRequiredTokensin classAbstractCheck- Returns:
- the token set this must be registered for.
- See Also:
-
setExcludes
Setter to specify packages where star imports are allowed.- Parameters:
excludesParam- a list of package names/fully-qualifies class names where star imports are ok.
-
setAllowClassImports
public void setAllowClassImports(boolean allow) Setter to control whether to allow starred class imports likeimport java.util.*;.- Parameters:
allow- true to allow false to disallow
-
setAllowStaticMemberImports
public void setAllowStaticMemberImports(boolean allow) Setter to control whether to allow starred static member imports likeimport static org.junit.Assert.*;.- Parameters:
allow- true to allow false to disallow
-
visitToken
Description copied from class:AbstractCheckCalled to process a token.- Overrides:
visitTokenin classAbstractCheck- Parameters:
ast- the token to process
-
logsStarredImportViolation
Gets the full import identifier. If the import is a starred import and it's not excluded then a violation is logged.- Parameters:
startingDot- the starting dot for the import statement
-