Class FinalLocalVariableCheck
java.lang.Object
com.puppycrawl.tools.checkstyle.AbstractAutomaticBean
com.puppycrawl.tools.checkstyle.api.AbstractViolationReporter
com.puppycrawl.tools.checkstyle.api.AbstractCheck
com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck
- All Implemented Interfaces:
Configurable,Contextualizable
Checks that local variables that never have their values changed are declared final.
The check can be configured to also check that unchanged parameters are declared final.
When configured to check parameters, the check ignores parameters of interface methods and abstract methods.
-
Property
validateEnhancedForLoopVariable- Control whether to check enhanced for-loop variable. Type isboolean. Default value isfalse. -
Property
validateUnnamedVariables- Control whether to check unnamed variables. Type isboolean. Default value isfalse. -
Property
tokens- tokens to check Type isjava.lang.String[]. Validation type istokenSet. Default value is: VARIABLE_DEF.
Parent is com.puppycrawl.tools.checkstyle.TreeWalker
Violation Message Keys:
-
final.variable
- Since:
- 3.2
-
Nested Class Summary
Nested ClassesModifier and TypeClassDescriptionprivate static final classRepresents information about final local variable candidate.private static final classHolder for the scope data.Nested classes/interfaces inherited from class com.puppycrawl.tools.checkstyle.AbstractAutomaticBean
AbstractAutomaticBean.OutputStreamOptions -
Field Summary
FieldsModifier and TypeFieldDescriptionprivate static final BitSetAssign operator types.Assigned variables of current scope.private static final BitSetLoop types.static final StringA key is pointing to the warning message text in "messages.properties" file.private final Deque<FinalLocalVariableCheck.ScopeData>Scope Deque.private booleanControl whether to check enhanced for-loop variable.private booleanControl whether to check unnamed variables. -
Constructor Summary
Constructors -
Method Summary
Modifier and TypeMethodDescriptionprivate static voiddetermineAssignmentConditions(DetailAST ident, FinalLocalVariableCheck.FinalVariableCandidate candidate) Determines identifier assignment conditions (assigned or already assigned).private static DetailASTFind the Class, Constructor, Enum, Method, or Field in which it is defined.private static DetailASTfindLastCaseGroupWhichContainsSlist(DetailAST literalSwitchAst) Returns the last token of typeTokenTypes.CASE_GROUPwhich containsTokenTypes.SLIST.int[]The configurable token set.int[]Returns the default token a check is interested in.Gets final variable candidate for ast.private static DetailASTgetParentLoop(DetailAST ast) Get the ast node of typeLOOP_TYPESthat is the ancestor of the current ast node, if there is no such node, null is returned.int[]The tokens that this check must be registered for.private voidinsertParameter(DetailAST ast) Insert a parameter at the topmost scope stack.private voidinsertVariable(DetailAST ast) Insert a variable at the topmost scope stack.private static booleanisAssignOperator(int parentType) Is Arithmetic operator.private static booleanIf token is CASE_GROUP or SWITCH_RULE and there is anothercasefollowing.private static booleanisFirstChild(DetailAST ast) Whether the ast is the first child of its parent.private static booleanisInitialized(DetailAST ast) Check if VARIABLE_DEF is initialized or not.private static booleanisInLambda(DetailAST paramDef) Check if current param is lambda's param.private static booleanisInMethodWithoutBody(DetailAST parameterDefAst) Checks if a parameter is within a method that has no implementation body.private static booleanisInSpecificCodeBlocks(DetailAST node, int... blockTypes) Checks whether the scope of a node is restricted to a specific code blocks.private static booleanisLoopAst(int ast) Checks whether the ast is a loop.private static booleanisMultipleTypeCatch(DetailAST parameterDefAst) Check if given parameter definition is a multiple type catch.private static booleanisSameVariables(DetailAST ast1, DetailAST ast2) Check if both the Variables are same.private static booleanisVariableInForInit(DetailAST variableDef) Checks if current variable is defined infor-loop init, e.g.:voidleaveToken(DetailAST ast) Called after all the child nodes have been process.private voidRemoves the final variable candidate from the Stack.final voidsetValidateEnhancedForLoopVariable(boolean validateEnhancedForLoopVariable) Setter to control whether to check enhanced for-loop variable.final voidsetValidateUnnamedVariables(boolean validateUnnamedVariables) Setter to control whether to check unnamed variables.private booleanDetermines whether enhanced for-loop variable should be checked or not.private booleanDetermines whether unnamed variable should be checked or not.private static booleanWhether the final variable candidate should be removed from the list of final local variable candidates.private static booleanIf there is anelsefollowing or token is CASE_GROUP or SWITCH_RULE and there is anothercasefollowing, then update the uninitialized variables.private voidStore un-initialized variables in a temporary stack for future use.private voidUpdate current scope data uninitialized variable according to the whole scope data.private voidUpdate assigned variables in a temporary stack.private voidupdateUninitializedVariables(Deque<DetailAST> scopeUninitializedVariableData) Update current scope data uninitialized variable according to the specific scope data.voidvisitToken(DetailAST ast) Called to process a token.Methods inherited from class com.puppycrawl.tools.checkstyle.api.AbstractCheck
beginTree, clearViolations, destroy, finishTree, getFileContents, getFilePath, getLine, getLineCodePoints, getLines, getTabWidth, getTokenNames, getViolations, init, isCommentNodesRequired, 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.AbstractAutomaticBean
configure, contextualize, getConfiguration, setupChild
-
Field Details
-
MSG_KEY
A key is pointing to the warning message text in "messages.properties" file.- See Also:
-
ASSIGN_OPERATOR_TYPES
Assign operator types. -
LOOP_TYPES
Loop types. -
scopeStack
Scope Deque. -
currentScopeAssignedVariables
Assigned variables of current scope. -
validateEnhancedForLoopVariable
Control whether to check enhanced for-loop variable. -
validateUnnamedVariables
Control whether to check unnamed variables.
-
-
Constructor Details
-
FinalLocalVariableCheck
public FinalLocalVariableCheck()
-
-
Method Details
-
setValidateEnhancedForLoopVariable
Setter to control whether to check enhanced for-loop variable.- Parameters:
validateEnhancedForLoopVariable- whether to check for-loop variable- Since:
- 6.5
-
setValidateUnnamedVariables
Setter to control whether to check unnamed variables.- Parameters:
validateUnnamedVariables- whether to check unnamed variables- Since:
- 10.18.0
-
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:
-
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
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:
-
visitToken
Description copied from class:AbstractCheckCalled to process a token.- Overrides:
visitTokenin classAbstractCheck- Parameters:
ast- the token to process
-
leaveToken
Description copied from class:AbstractCheckCalled after all the child nodes have been process.- Overrides:
leaveTokenin classAbstractCheck- Parameters:
ast- the token leaving
-
updateCurrentScopeAssignedVariables
Update assigned variables in a temporary stack. -
determineAssignmentConditions
private static void determineAssignmentConditions(DetailAST ident, FinalLocalVariableCheck.FinalVariableCandidate candidate) Determines identifier assignment conditions (assigned or already assigned).- Parameters:
ident- identifier.candidate- final local variable candidate.
-
isInSpecificCodeBlocks
Checks whether the scope of a node is restricted to a specific code blocks.- Parameters:
node- node.blockTypes- int array of all block types to check.- Returns:
- true if the scope of a node is restricted to specific code block types.
-
getFinalCandidate
Gets final variable candidate for ast.- Parameters:
ast- ast.- Returns:
- Optional of
FinalLocalVariableCheck.FinalVariableCandidatefor ast from scopeStack.
-
storePrevScopeUninitializedVariableData
Store un-initialized variables in a temporary stack for future use. -
updateAllUninitializedVariables
Update current scope data uninitialized variable according to the whole scope data. -
updateUninitializedVariables
Update current scope data uninitialized variable according to the specific scope data.- Parameters:
scopeUninitializedVariableData- variable for specific stack of uninitialized variables
-
shouldUpdateUninitializedVariables
If there is anelsefollowing or token is CASE_GROUP or SWITCH_RULE and there is anothercasefollowing, then update the uninitialized variables.- Parameters:
ast- token to be checked- Returns:
- true if should be updated, else false
-
isCaseTokenWithAnotherCaseFollowing
If token is CASE_GROUP or SWITCH_RULE and there is anothercasefollowing.- Parameters:
ast- token to be checked- Returns:
- true if token is CASE_GROUP or SWITCH_RULE and there is another
casefollowing, else false
-
findLastCaseGroupWhichContainsSlist
Returns the last token of typeTokenTypes.CASE_GROUPwhich containsTokenTypes.SLIST.- Parameters:
literalSwitchAst- ast node of typeTokenTypes.LITERAL_SWITCH- Returns:
- the matching token, or null if no match
-
shouldCheckEnhancedForLoopVariable
Determines whether enhanced for-loop variable should be checked or not.- Parameters:
ast- The ast to compare.- Returns:
- true if enhanced for-loop variable should be checked.
-
shouldCheckUnnamedVariable
Determines whether unnamed variable should be checked or not.- Parameters:
ast- The ast to compare.- Returns:
- true if unnamed variable should be checked.
-
insertParameter
Insert a parameter at the topmost scope stack.- Parameters:
ast- the variable to insert.
-
insertVariable
Insert a variable at the topmost scope stack.- Parameters:
ast- the variable to insert.
-
isInitialized
Check if VARIABLE_DEF is initialized or not.- Parameters:
ast- VARIABLE_DEF to be checked- Returns:
- true if initialized
-
isFirstChild
Whether the ast is the first child of its parent.- Parameters:
ast- the ast to check.- Returns:
- true if the ast is the first child of its parent.
-
removeFinalVariableCandidateFromStack
Removes the final variable candidate from the Stack.- Parameters:
ast- variable to remove.
-
isMultipleTypeCatch
Check if given parameter definition is a multiple type catch.- Parameters:
parameterDefAst- parameter definition- Returns:
- true if it is a multiple type catch, false otherwise
-
shouldRemoveFinalVariableCandidate
private static boolean shouldRemoveFinalVariableCandidate(FinalLocalVariableCheck.ScopeData scopeData, DetailAST ast) Whether the final variable candidate should be removed from the list of final local variable candidates.- Parameters:
scopeData- the scope data of the variable.ast- the variable ast.- Returns:
- true, if the variable should be removed.
-
getParentLoop
Get the ast node of typeLOOP_TYPESthat is the ancestor of the current ast node, if there is no such node, null is returned.- Parameters:
ast- ast node- Returns:
- ast node of type
LOOP_TYPESthat is the ancestor of the current ast node, null if no such node exists
-
isAssignOperator
Is Arithmetic operator.- Parameters:
parentType- token AST- Returns:
- true is token type is in arithmetic operator
-
isVariableInForInit
Checks if current variable is defined infor-loop init, e.g.:for (int i = 0, j = 0; i < j; i++) { . . . }i, jare defined infor-loop init- Parameters:
variableDef- variable definition node.- Returns:
- true if variable is defined in
for-loop init
-
isInMethodWithoutBody
Checks if a parameter is within a method that has no implementation body.- Parameters:
parameterDefAst- the AST node representing the parameter definition- Returns:
- true if the parameter is in a method without a body
-
isInLambda
Check if current param is lambda's param.- Parameters:
paramDef-parameter def.- Returns:
- true if current param is lambda's param.
-
findFirstUpperNamedBlock
Find the Class, Constructor, Enum, Method, or Field in which it is defined.- Parameters:
ast- Variable for which we want to find the scope in which it is defined- Returns:
- ast The Class or Constructor or Method in which it is defined.
-
isSameVariables
Check if both the Variables are same.- Parameters:
ast1- Variable to compareast2- Variable to compare- Returns:
- true if both the variables are same, otherwise false
-
isLoopAst
Checks whether the ast is a loop.- Parameters:
ast- the ast to check.- Returns:
- true if the ast is a loop.
-