Class MethodCountCheck
- All Implemented Interfaces:
Configurable
,Contextualizable
This check can be configured to flag classes that define too many methods
to prevent the class from getting too complex. Counting can be customized
to prevent too many total methods in a type definition (maxTotal
),
or to prevent too many methods of a specific access modifier (private
,
package
, protected
or public
). Each count is completely
separated to customize how many methods of each you want to allow. For example,
specifying a maxTotal
of 10, still means you can prevent more than 0
maxPackage
methods. A violation won't appear for 8 public methods,
but one will appear if there is also 3 private methods or any package-private methods.
Methods defined in anonymous classes are not counted towards any totals. Counts only go towards the main type declaration parent, and are kept separate from it's children's inner types.
public class ExampleClass { public enum Colors { RED, GREEN, YELLOW; public String getRGB() { ... } // NOT counted towards ExampleClass } public void example() { // counted towards ExampleClass Runnable r = (new Runnable() { public void run() { ... } // NOT counted towards ExampleClass, won't produce any violations }); } public static class InnerExampleClass { protected void example2() { ... } // NOT counted towards ExampleClass, // but counted towards InnerExampleClass } }
-
Property
maxPackage
- Specify the maximum number ofpackage
methods allowed. Type isint
. Default value is100
. -
Property
maxPrivate
- Specify the maximum number ofprivate
methods allowed. Type isint
. Default value is100
. -
Property
maxProtected
- Specify the maximum number ofprotected
methods allowed. Type isint
. Default value is100
. -
Property
maxPublic
- Specify the maximum number ofpublic
methods allowed. Type isint
. Default value is100
. -
Property
maxTotal
- Specify the maximum number of methods allowed at all scope levels. Type isint
. Default value is100
. -
Property
tokens
- tokens to check Type isjava.lang.String[]
. Validation type istokenSet
. Default value is: CLASS_DEF, ENUM_CONSTANT_DEF, ENUM_DEF, INTERFACE_DEF, ANNOTATION_DEF, RECORD_DEF.
Parent is com.puppycrawl.tools.checkstyle.TreeWalker
Violation Message Keys:
-
too.many.methods
-
too.many.packageMethods
-
too.many.privateMethods
-
too.many.protectedMethods
-
too.many.publicMethods
- Since:
- 5.3
-
Nested Class Summary
Nested ClassesModifier and TypeClassDescriptionprivate static final class
Marker class used to collect data about the number of methods per class.Nested classes/interfaces inherited from class com.puppycrawl.tools.checkstyle.AbstractAutomaticBean
AbstractAutomaticBean.OutputStreamOptions
-
Field Summary
FieldsModifier and TypeFieldDescriptionprivate final Deque<MethodCountCheck.MethodCounter>
Maintains stack of counters, to support inner types.private static final int
Default maximum number of methods.private int
Specify the maximum number ofpackage
methods allowed.private int
Specify the maximum number ofprivate
methods allowed.private int
Specify the maximum number ofprotected
methods allowed.private int
Specify the maximum number ofpublic
methods allowed.private int
Specify the maximum number of methods allowed at all scope levels.static final String
A key is pointing to the warning message text in "messages.properties" file.static final String
A key is pointing to the warning message text in "messages.properties" file.static final String
A key is pointing to the warning message text in "messages.properties" file.static final String
A key is pointing to the warning message text in "messages.properties" file.static final String
A key is pointing to the warning message text in "messages.properties" file. -
Constructor Summary
Constructors -
Method Summary
Modifier and TypeMethodDescriptionprivate void
checkCounters
(MethodCountCheck.MethodCounter counter, DetailAST ast) Check the counters and report violations.private void
Utility for reporting if a maximum has been exceeded.int[]
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 boolean
isInLatestScopeDefinition
(DetailAST methodDef) Checks if there is a scope definition to check and that the method is found inside that scope (class, enum, etc.).void
leaveToken
(DetailAST ast) Called after all the child nodes have been process.private void
raiseCounter
(DetailAST method) Determine the visibility modifier and raise the corresponding counter.void
setMaxPackage
(int value) Setter to specify the maximum number ofpackage
methods allowed.void
setMaxPrivate
(int value) Setter to specify the maximum number ofprivate
methods allowed.void
setMaxProtected
(int value) Setter to specify the maximum number ofprotected
methods allowed.void
setMaxPublic
(int value) Setter to specify the maximum number ofpublic
methods allowed.void
setMaxTotal
(int value) Setter to specify the maximum number of methods allowed at all scope levels.void
visitToken
(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, setTokens
Methods inherited from class com.puppycrawl.tools.checkstyle.api.AbstractViolationReporter
finishLocalSetup, getCustomMessages, getId, getMessageBundle, getSeverity, getSeverityLevel, setId, setSeverity
Methods inherited from class com.puppycrawl.tools.checkstyle.AbstractAutomaticBean
configure, contextualize, getConfiguration, setupChild
-
Field Details
-
MSG_PRIVATE_METHODS
A key is pointing to the warning message text in "messages.properties" file.- See Also:
-
MSG_PACKAGE_METHODS
A key is pointing to the warning message text in "messages.properties" file.- See Also:
-
MSG_PROTECTED_METHODS
A key is pointing to the warning message text in "messages.properties" file.- See Also:
-
MSG_PUBLIC_METHODS
A key is pointing to the warning message text in "messages.properties" file.- See Also:
-
MSG_MANY_METHODS
A key is pointing to the warning message text in "messages.properties" file.- See Also:
-
DEFAULT_MAX_METHODS
Default maximum number of methods.- See Also:
-
counters
Maintains stack of counters, to support inner types. -
maxPrivate
Specify the maximum number ofprivate
methods allowed. -
maxPackage
Specify the maximum number ofpackage
methods allowed. -
maxProtected
Specify the maximum number ofprotected
methods allowed. -
maxPublic
Specify the maximum number ofpublic
methods allowed. -
maxTotal
Specify the maximum number of methods allowed at all scope levels.
-
-
Constructor Details
-
MethodCountCheck
public MethodCountCheck()
-
-
Method Details
-
getDefaultTokens
Description copied from class:AbstractCheck
Returns the default token a check is interested in. Only used if the configuration for a check does not define the tokens.- Specified by:
getDefaultTokens
in classAbstractCheck
- Returns:
- the default tokens
- See Also:
-
getAcceptableTokens
Description copied from class:AbstractCheck
The 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:
getAcceptableTokens
in classAbstractCheck
- Returns:
- the token set this check is designed for.
- See Also:
-
getRequiredTokens
Description copied from class:AbstractCheck
The tokens that this check must be registered for.- Specified by:
getRequiredTokens
in classAbstractCheck
- Returns:
- the token set this must be registered for.
- See Also:
-
visitToken
Description copied from class:AbstractCheck
Called to process a token.- Overrides:
visitToken
in classAbstractCheck
- Parameters:
ast
- the token to process
-
leaveToken
Description copied from class:AbstractCheck
Called after all the child nodes have been process.- Overrides:
leaveToken
in classAbstractCheck
- Parameters:
ast
- the token leaving
-
isInLatestScopeDefinition
Checks if there is a scope definition to check and that the method is found inside that scope (class, enum, etc.).- Parameters:
methodDef
- The method to analyze.- Returns:
true
if the method is part of the latest scope definition and should be counted.
-
raiseCounter
Determine the visibility modifier and raise the corresponding counter.- Parameters:
method
- The method-subtree from the AbstractSyntaxTree.
-
checkCounters
Check the counters and report violations.- Parameters:
counter
- the method counters to checkast
- to report violations against.
-
checkMax
Utility for reporting if a maximum has been exceeded.- Parameters:
max
- the maximum allowed valuevalue
- the actual valuemsg
- the message to log. Takes two arguments of value and maximum.ast
- the AST to associate with the message.
-
setMaxPrivate
Setter to specify the maximum number ofprivate
methods allowed.- Parameters:
value
- the maximum allowed.- Since:
- 5.3
-
setMaxPackage
Setter to specify the maximum number ofpackage
methods allowed.- Parameters:
value
- the maximum allowed.- Since:
- 5.3
-
setMaxProtected
Setter to specify the maximum number ofprotected
methods allowed.- Parameters:
value
- the maximum allowed.- Since:
- 5.3
-
setMaxPublic
Setter to specify the maximum number ofpublic
methods allowed.- Parameters:
value
- the maximum allowed.- Since:
- 5.3
-
setMaxTotal
Setter to specify the maximum number of methods allowed at all scope levels.- Parameters:
value
- the maximum allowed.- Since:
- 5.3
-