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 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 AbstractCheck
beginTree, clearViolations, destroy, finishTree, getFileContents, getLine, getLineCodePoints, getLines, getTabWidth, getTokenNames, getViolations, init, isCommentNodesRequired, leaveToken, log, log, log, setFileContents, setTabWidth, setTokensMethods inherited from class AbstractViolationReporter
finishLocalSetup, getCustomMessages, getId, getMessageBundle, getSeverity, getSeverityLevel, setId, setSeverityMethods inherited from class 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
-
excludes
-
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
-