Class JavaNCSSCheck
- All Implemented Interfaces:
Configurable,Contextualizable
Roughly said the NCSS metric is calculated by counting the source lines which are not comments, (nearly) equivalent to counting the semicolons and opening curly braces.
The NCSS for a class is summarized from the NCSS of all its methods, the NCSS of its nested classes and the number of member variable declarations.
The NCSS for a file is summarized from the ncss of all its top level classes, the number of imports and the package declaration.
Rationale: Too large methods and classes are hard to read and costly to maintain. A large NCSS number often means that a method or class has too many responsibilities and/or functionalities which should be decomposed into smaller units.
Here is a breakdown of what exactly is counted and not counted:
| Structure | NCSS Count | Notes |
|---|---|---|
| Package declaration | 1 | Counted at the terminating semicolon. |
| Import declaration | 1 | Each single, static, or wildcard import counts as 1. |
Class, Interface, Annotation (@interface) | 1 | Counted at the opening curly brace of the body. |
| Method, Constructor | 1 | Counted at the declaration. |
| Static initializer, Instance initializer | 1 | Both static {} and bare {} initializer blocks count as 1. |
| Annotation type member | 1 | Each method-like member declaration inside @interface counts as 1.
A standalone ; inside @interface also counts as 1. |
| Variable declaration | 1 | 1 per statement regardless of how many variables are declared on that line.
int x, y; counts as 1. |
if | 1 | The if keyword counts as 1. |
else, else if | 1 | The else keyword counts as 1, separate from the if count. |
while, do, for | 1 | The keyword header counts as 1. |
switch | 1 | The switch keyword counts as 1. |
case, default | 1 | Every case and default label adds 1. |
try | 0 | The try keyword itself does not count. |
catch | 1 | Each catch block counts as 1. |
finally | 1 | The finally block counts as 1. |
synchronized | 1 | The synchronized statement counts as 1. |
return, break, continue, throw | 1 | Each counts as 1. |
assert | 1 | Each assert statement counts as 1, with or without a message expression. |
| Labeled statement | 1 | label: statement counts as 1. |
| Explicit constructor invocation | 1 | this() or super() calls inside a constructor body
each count as 1. |
| Expression statements (assignments, method calls) | 1 | Statement-level expressions terminated by ; count as 1.
A method call inside a return does not add an extra count. |
| Empty blocks {} | 0 | Empty curly braces do not increase the count. |
| Empty statements ; | 0 | Standalone semicolons outside of @interface do not increase
the count. |
- Since:
- 3.5
-
Nested Class Summary
Nested ClassesModifier and TypeClassDescriptionprivate static final classClass representing a counter.Nested classes/interfaces inherited from class com.puppycrawl.tools.checkstyle.AbstractAutomaticBean
AbstractAutomaticBean.OutputStreamOptions -
Field Summary
FieldsModifier and TypeFieldDescriptionprivate static final intDefault constant for max file ncss.private intSpecify the maximum allowed number of non commenting lines in a class.private Deque<JavaNCSSCheck.Counter> List containing the stacked counters.private static final intDefault constant for max file ncss.private intSpecify the maximum allowed number of non commenting lines in a file including all top level and nested classes.private static final intDefault constant for max method ncss.private intSpecify the maximum allowed number of non commenting lines in a method.static final StringA key is pointing to the warning message text in "messages.properties" file.static final StringA key is pointing to the warning message text in "messages.properties" file.static final StringA key is pointing to the warning message text in "messages.properties" file.static final StringA key is pointing to the warning message text in "messages.properties" file.private static final intDefault constant for max record ncss.private intSpecify the maximum allowed number of non commenting lines in a record. -
Constructor Summary
Constructors -
Method Summary
Modifier and TypeMethodDescriptionvoidCalled before the starting to process a tree.voidfinishTree(DetailAST rootAST) Called after finished processing a tree.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 static booleanisCountable(DetailAST ast) Checks if a token is countable for the ncss metric.private static booleanChecks if an expression is countable for the ncss metric.private static booleanisMethodOrCtorOrInitDefinition(int tokenType) Checks if a token is a method, constructor, or compact constructor definition.private static booleanChecks if a variable definition is countable.voidleaveToken(DetailAST ast) Called after all the child nodes have been process.voidsetClassMaximum(int classMaximum) Setter to specify the maximum allowed number of non commenting lines in a class.voidsetFileMaximum(int fileMaximum) Setter to specify the maximum allowed number of non commenting lines in a file including all top level and nested classes.voidsetMethodMaximum(int methodMaximum) Setter to specify the maximum allowed number of non commenting lines in a method.voidsetRecordMaximum(int recordMaximum) Setter to specify the maximum allowed number of non commenting lines in a record.voidvisitToken(DetailAST ast) Called to process a token.Methods inherited from class com.puppycrawl.tools.checkstyle.api.AbstractCheck
clearViolations, destroy, 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_METHOD
A key is pointing to the warning message text in "messages.properties" file.- See Also:
-
MSG_CLASS
A key is pointing to the warning message text in "messages.properties" file.- See Also:
-
MSG_RECORD
A key is pointing to the warning message text in "messages.properties" file.- See Also:
-
MSG_FILE
A key is pointing to the warning message text in "messages.properties" file.- See Also:
-
FILE_MAX_NCSS
Default constant for max file ncss.- See Also:
-
CLASS_MAX_NCSS
Default constant for max file ncss.- See Also:
-
RECORD_MAX_NCSS
Default constant for max record ncss.- See Also:
-
METHOD_MAX_NCSS
Default constant for max method ncss.- See Also:
-
fileMaximum
Specify the maximum allowed number of non commenting lines in a file including all top level and nested classes. -
classMaximum
Specify the maximum allowed number of non commenting lines in a class. -
recordMaximum
Specify the maximum allowed number of non commenting lines in a record. -
methodMaximum
Specify the maximum allowed number of non commenting lines in a method. -
counters
List containing the stacked counters.
-
-
Constructor Details
-
JavaNCSSCheck
public JavaNCSSCheck()
-
-
Method Details
-
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:
-
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:
-
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:
-
beginTree
Description copied from class:AbstractCheckCalled before the starting to process a tree. Ideal place to initialize information that is to be collected whilst processing a tree.- Overrides:
beginTreein classAbstractCheck- Parameters:
rootAST- the root of the tree
-
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
-
finishTree
Description copied from class:AbstractCheckCalled after finished processing a tree. Ideal place to report on information collected whilst processing a tree.- Overrides:
finishTreein classAbstractCheck- Parameters:
rootAST- the root of the tree
-
setFileMaximum
Setter to specify the maximum allowed number of non commenting lines in a file including all top level and nested classes.- Parameters:
fileMaximum- the maximum ncss- Since:
- 3.5
-
setClassMaximum
Setter to specify the maximum allowed number of non commenting lines in a class.- Parameters:
classMaximum- the maximum ncss- Since:
- 3.5
-
setRecordMaximum
Setter to specify the maximum allowed number of non commenting lines in a record.- Parameters:
recordMaximum- the maximum ncss- Since:
- 8.36
-
setMethodMaximum
Setter to specify the maximum allowed number of non commenting lines in a method.- Parameters:
methodMaximum- the maximum ncss- Since:
- 3.5
-
isCountable
Checks if a token is countable for the ncss metric.- Parameters:
ast- the AST- Returns:
- true if the token is countable
-
isVariableDefCountable
Checks if a variable definition is countable.- Parameters:
ast- the AST- Returns:
- true if the variable definition is countable, false otherwise
-
isExpressionCountable
Checks if an expression is countable for the ncss metric.- Parameters:
ast- the AST- Returns:
- true if the expression is countable, false otherwise
-
isMethodOrCtorOrInitDefinition
Checks if a token is a method, constructor, or compact constructor definition.- Parameters:
tokenType- the type of token we are checking- Returns:
- true if token type is method or ctor definition, false otherwise
-