The following document contains the results of PMD 6.13.0.
| Rule | Violation | Line |
|---|---|---|
| FormalParameterNamingConventions | The method parameter name '_plugin' doesn't match '[a-z][a-zA-Z0-9]*' | 127 |
| FormalParameterNamingConventions | The method parameter name '_plugin' doesn't match '[a-z][a-zA-Z0-9]*' | 129 |
| Rule | Violation | Line |
|---|---|---|
| FormalParameterNamingConventions | The method parameter name '_plugin' doesn't match '[a-z][a-zA-Z0-9]*' | 90 |
| Rule | Violation | Line |
|---|---|---|
| FormalParameterNamingConventions | The method parameter name '_strLibelle' doesn't match '[a-z][a-zA-Z0-9]*' | 122 |
| Rule | Violation | Line |
|---|---|---|
| FormalParameterNamingConventions | The method parameter name '_computeDate' doesn't match '[a-z][a-zA-Z0-9]*' | 97 |
| FormalParameterNamingConventions | The method parameter name '_validUntil' doesn't match '[a-z][a-zA-Z0-9]*' | 107 |
| FormalParameterNamingConventions | The method parameter name '_file' doesn't match '[a-z][a-zA-Z0-9]*' | 117 |
| Rule | Violation | Line |
|---|---|---|
| FormalParameterNamingConventions | The method parameter name '_plugin' doesn't match '[a-z][a-zA-Z0-9]*' | 129 |
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The field name 'formatter' doesn't match '_[a-z][a-zA-Z0-9]*' | 52 |
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The final field name '_fWidth' doesn't match '[a-z][a-zA-Z0-9]*' | 56 |
| FieldNamingConventions | The final field name '_fHeight' doesn't match '[a-z][a-zA-Z0-9]*' | 57 |
| FieldNamingConventions | The field name 'absolutePosition' doesn't match '_[a-z][a-zA-Z0-9]*' | 58 |
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The final field name '_fWidth' doesn't match '[a-z][a-zA-Z0-9]*' | 65 |
| FieldNamingConventions | The final field name '_fHeight' doesn't match '[a-z][a-zA-Z0-9]*' | 66 |
| FieldNamingConventions | The final field name '_foregroundColor' doesn't match '[a-z][a-zA-Z0-9]*' | 67 |
| FieldNamingConventions | The final field name '_matrix' doesn't match '[a-z][a-zA-Z0-9]*' | 68 |
| FieldNamingConventions | The field name 'absolutePosition' doesn't match '_[a-z][a-zA-Z0-9]*' | 69 |
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The public constant name 'formatter' doesn't match '[A-Z][A-Z_0-9]*' | 117 |
| AvoidThrowingRawExceptionTypes | Avoid throwing raw exception types. | 320 |
| AvoidThrowingRawExceptionTypes | Avoid throwing raw exception types. | 402 |
| Rule | Violation | Line |
|---|---|---|
| ClassWithOnlyPrivateConstructorsShouldBeFinal | A class which only has private constructors should be final | 62–146 |
| Rule | Violation | Line |
|---|---|---|
| ClassWithOnlyPrivateConstructorsShouldBeFinal | A class which only has private constructors should be final | 43–90 |
| Rule | Violation | Line |
|---|---|---|
| AbstractClassWithoutAnyMethod | No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. | 42–47 |
| Rule | Violation | Line |
|---|---|---|
| FormalParameterNamingConventions | The method parameter name '_allocataireCaf' doesn't match '[a-z][a-zA-Z0-9]*' | 51 |
| FormalParameterNamingConventions | The method parameter name '_numAllocataire' doesn't match '[a-z][a-zA-Z0-9]*' | 61 |
| FormalParameterNamingConventions | The method parameter name '_codePostal' doesn't match '[a-z][a-zA-Z0-9]*' | 71 |
| FormalParameterNamingConventions | The method parameter name '_attestationNonCaf' doesn't match '[a-z][a-zA-Z0-9]*' | 81 |
| FormalParameterNamingConventions | The method parameter name '_numFiscal' doesn't match '[a-z][a-zA-Z0-9]*' | 91 |
| FormalParameterNamingConventions | The method parameter name '_referenceAvisImposition' doesn't match '[a-z][a-zA-Z0-9]*' | 101 |
| FormalParameterNamingConventions | The method parameter name '_fromJS' doesn't match '[a-z][a-zA-Z0-9]*' | 111 |
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The final field name '_userDataHome' doesn't match '[a-z][a-zA-Z0-9]*' | 82 |
| Rule | Violation | Line |
|---|---|---|
| AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 320 |
| Rule | Violation | Line |
|---|---|---|
| GuardLogStatement | Logger calls should be surrounded by log level guards. | 290 |
| Rule | Violation | Line |
|---|---|---|
| AvoidReassigningParameters | Avoid reassigning parameters such as 'bytes' | 60 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 43–80 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 65 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Public method and constructor comments are required | 129 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Public method and constructor comments are required | 117–120 |
| CommentRequired | Public method and constructor comments are required | 122–125 |
| Rule | Violation | Line |
|---|---|---|
| ForLoopCanBeForeach | This for loop can be replaced by a foreach loop | 249–252 |
| Rule | Violation | Line |
|---|---|---|
| DataClass | The class 'UserData' is suspected to be a Data Class (WOC=20.000%, NOPA=0, NOAM=12, WMC=16) | 44–140 |
| UnnecessaryModifier | Unnecessary modifier 'static' on enum 'Source': nested enums are implicitly static | 48–52 |
| CommentRequired | Enum comments are required | 48–52 |
| CommentRequired | Public method and constructor comments are required | 82–85 |
| CommentRequired | Public method and constructor comments are required | 87–90 |
| MethodReturnsInternalArray | Returning '_file' may expose an internal array. | 114 |
| ArrayIsStoredDirectly | The user-supplied array '_file' is stored directly. | 117 |
| CommentRequired | Public method and constructor comments are required | 132–139 |
| Rule | Violation | Line |
|---|---|---|
| DefaultPackage | Use explicit scoping instead of the default package private level | 60–61 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the _encryptionService field if you want a default access modifier | 61 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 46–74 |
| ImmutableField | Private field 'formatter' could be made final; it is only initialized in the declaration or constructor. | 52 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the _userDataHome field if you want a default access modifier | 54 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 54 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 70 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 42–49 |
| CommentRequired | Public method and constructor comments are required | 45 |
| CommentRequired | Public method and constructor comments are required | 47 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 52–118 |
| CommentRequired | Public method and constructor comments are required | 60–65 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 59–159 |
| UseUnderscoresInNumericLiterals | Number 0.029375f should separate every third digit with an underscore | 63 |
| CommentRequired | Public method and constructor comments are required | 71–90 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 77 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 77 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 87 |
| Rule | Violation | Line |
|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 34–550 |
| CommentRequired | Header comments are required | 83–550 |
| UnnecessaryModifier | Unnecessary modifier 'static' on enum 'Fonts': nested enums are implicitly static | 130–137 |
| CommentRequired | Enum comments are required | 130–137 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 224 |
| LooseCoupling | Avoid using implementation types like 'HashMap'; use the interface instead | 288 |
| ShortVariable | Avoid variables with short names like tt | 332 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 337–346 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 415 |
| LooseCoupling | Avoid using implementation types like 'HashMap'; use the interface instead | 465 |
| ShortVariable | Avoid variables with short names like qf | 506 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 509 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 510 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 511 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 512 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 513 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 514 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 515 |
| TooFewBranchesForASwitchStatement | A switch with less than three branches is inefficient, use a if statement instead. | 527–533 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 46–92 |
| CommentRequired | Public method and constructor comments are required | 63–66 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 36–47 |
| CommentRequired | Public method and constructor comments are required | 39 |
| CommentRequired | Public method and constructor comments are required | 41 |
| CommentRequired | Public method and constructor comments are required | 43 |
| CommentRequired | Public method and constructor comments are required | 45 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 62–146 |
| ShortVariable | Avoid variables with short names like qf | 74 |
| CommentRequired | Public method and constructor comments are required | 74–80 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. | 76 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 76 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 77 |
| ShortVariable | Avoid variables with short names like iv | 84 |
| CommentRequired | Public method and constructor comments are required | 89–96 |
| UseUnderscoresInNumericLiterals | Number 65536 should separate every third digit with an underscore | 93 |
| UnnecessaryLocalBeforeReturn | Consider simply returning the value vs storing it in local variable 'secret' | 95 |
| CommentRequired | Public method and constructor comments are required | 98–113 |
| ShortVariable | Avoid variables with short names like iv | 103 |
| CommentRequired | Public method and constructor comments are required | 115–129 |
| ShortVariable | Avoid variables with short names like iv | 120 |
| CommentRequired | Public method and constructor comments are required | 131–144 |
| ShortVariable | Avoid variables with short names like md | 135 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 43–90 |
| ShortVariable | Avoid variables with short names like ci | 66 |
| Rule | Violation | Line |
|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 50–117 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the getItemsFromIds method if you want a default access modifier | 116 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 116 |
| Rule | Violation | Line |
|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 42–47 |
| Rule | Violation | Line |
|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 34–341 |
| CommentRequired | Header comments are required | 79–341 |
| CommentRequired | Public method and constructor comments are required | 117–131 |
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 173 |
| CommentRequired | Public method and constructor comments are required | 174–187 |
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 189 |
| NPathComplexity | The method 'getInfos(HttpServletRequest)' has an NPath complexity of 240 | 190–233 |
| CyclomaticComplexity | The method 'getInfos(HttpServletRequest)' has a cyclomatic complexity of 12. | 190–233 |
| CommentRequired | Public method and constructor comments are required | 190–233 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 192 |
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 317 |
| CommentRequired | Public method and constructor comments are required | 318–340 |
| Rule | Violation | Line |
|---|---|---|
| DataClass | The class 'FormBean' is suspected to be a Data Class (WOC=0.000%, NOPA=0, NOAM=14, WMC=14) | 36–115 |
| CommentRequired | Header comments are required | 36–115 |
| Rule | Violation | Line |
|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 66–330 |
| ImmutableField | Private field '_trancheHome' could be made final; it is only initialized in the declaration or constructor. | 114 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 130 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 148–155 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the getItemsFromIds method if you want a default access modifier | 149–155 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 175 |
| Rule | Violation | Line |
|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 56–137 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 43–80 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 65 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| FormalParameterNamingConventions | The method parameter name '_plugin' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 127 |
| CommentRequired | Public method and constructor comments are required | 3 | 129 |
| FormalParameterNamingConventions | The method parameter name '_plugin' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 129 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| FormalParameterNamingConventions | The method parameter name '_plugin' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 90 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Public method and constructor comments are required | 3 | 117–120 |
| CommentRequired | Public method and constructor comments are required | 3 | 122–125 |
| FormalParameterNamingConventions | The method parameter name '_strLibelle' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 122 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ForLoopCanBeForeach | This for loop can be replaced by a foreach loop | 3 | 249–252 |
| AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 2 | 320 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| DataClass | The class 'UserData' is suspected to be a Data Class (WOC=20.000%, NOPA=0, NOAM=12, WMC=16) | 3 | 44–140 |
| UnnecessaryModifier | Unnecessary modifier 'static' on enum 'Source': nested enums are implicitly static | 3 | 48–52 |
| CommentRequired | Enum comments are required | 3 | 48–52 |
| CommentRequired | Public method and constructor comments are required | 3 | 82–85 |
| CommentRequired | Public method and constructor comments are required | 3 | 87–90 |
| FormalParameterNamingConventions | The method parameter name '_computeDate' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 97 |
| FormalParameterNamingConventions | The method parameter name '_validUntil' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 107 |
| MethodReturnsInternalArray | Returning '_file' may expose an internal array. | 3 | 114 |
| ArrayIsStoredDirectly | The user-supplied array '_file' is stored directly. | 3 | 117 |
| FormalParameterNamingConventions | The method parameter name '_file' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 117 |
| CommentRequired | Public method and constructor comments are required | 3 | 132–139 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 60–61 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the _encryptionService field if you want a default access modifier | 3 | 61 |
| FormalParameterNamingConventions | The method parameter name '_plugin' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 129 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 46–74 |
| FieldNamingConventions | The field name 'formatter' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 52 |
| ImmutableField | Private field 'formatter' could be made final; it is only initialized in the declaration or constructor. | 3 | 52 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the _userDataHome field if you want a default access modifier | 3 | 54 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 54 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 3 | 70 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 42–49 |
| CommentRequired | Public method and constructor comments are required | 3 | 45 |
| CommentRequired | Public method and constructor comments are required | 3 | 47 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 52–118 |
| FieldNamingConventions | The final field name '_fWidth' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 56 |
| FieldNamingConventions | The final field name '_fHeight' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 57 |
| FieldNamingConventions | The field name 'absolutePosition' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 58 |
| CommentRequired | Public method and constructor comments are required | 3 | 60–65 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 59–159 |
| UseUnderscoresInNumericLiterals | Number 0.029375f should separate every third digit with an underscore | 3 | 63 |
| FieldNamingConventions | The final field name '_fWidth' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 65 |
| FieldNamingConventions | The final field name '_fHeight' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 66 |
| FieldNamingConventions | The final field name '_foregroundColor' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 67 |
| FieldNamingConventions | The final field name '_matrix' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 68 |
| FieldNamingConventions | The field name 'absolutePosition' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 69 |
| CommentRequired | Public method and constructor comments are required | 3 | 71–90 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 77 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 77 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 87 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 3 | 34–550 |
| CommentRequired | Header comments are required | 3 | 83–550 |
| FieldNamingConventions | The public constant name 'formatter' doesn't match '[A-Z][A-Z_0-9]*' | 1 | 117 |
| UnnecessaryModifier | Unnecessary modifier 'static' on enum 'Fonts': nested enums are implicitly static | 3 | 130–137 |
| CommentRequired | Enum comments are required | 3 | 130–137 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 224 |
| LooseCoupling | Avoid using implementation types like 'HashMap'; use the interface instead | 3 | 288 |
| GuardLogStatement | Logger calls should be surrounded by log level guards. | 2 | 290 |
| AvoidThrowingRawExceptionTypes | Avoid throwing raw exception types. | 1 | 320 |
| ShortVariable | Avoid variables with short names like tt | 3 | 332 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 337–346 |
| AvoidThrowingRawExceptionTypes | Avoid throwing raw exception types. | 1 | 402 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 415 |
| LooseCoupling | Avoid using implementation types like 'HashMap'; use the interface instead | 3 | 465 |
| ShortVariable | Avoid variables with short names like qf | 3 | 506 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 3 | 509 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 3 | 510 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 3 | 511 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 3 | 512 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 3 | 513 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 3 | 514 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 3 | 515 |
| TooFewBranchesForASwitchStatement | A switch with less than three branches is inefficient, use a if statement instead. | 3 | 527–533 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 46–92 |
| CommentRequired | Public method and constructor comments are required | 3 | 63–66 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 36–47 |
| CommentRequired | Public method and constructor comments are required | 3 | 39 |
| CommentRequired | Public method and constructor comments are required | 3 | 41 |
| CommentRequired | Public method and constructor comments are required | 3 | 43 |
| CommentRequired | Public method and constructor comments are required | 3 | 45 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ClassWithOnlyPrivateConstructorsShouldBeFinal | A class which only has private constructors should be final | 1 | 62–146 |
| CommentRequired | Header comments are required | 3 | 62–146 |
| ShortVariable | Avoid variables with short names like qf | 3 | 74 |
| CommentRequired | Public method and constructor comments are required | 3 | 74–80 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. | 3 | 76 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 3 | 76 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 3 | 77 |
| ShortVariable | Avoid variables with short names like iv | 3 | 84 |
| CommentRequired | Public method and constructor comments are required | 3 | 89–96 |
| UseUnderscoresInNumericLiterals | Number 65536 should separate every third digit with an underscore | 3 | 93 |
| UnnecessaryLocalBeforeReturn | Consider simply returning the value vs storing it in local variable 'secret' | 3 | 95 |
| CommentRequired | Public method and constructor comments are required | 3 | 98–113 |
| ShortVariable | Avoid variables with short names like iv | 3 | 103 |
| CommentRequired | Public method and constructor comments are required | 3 | 115–129 |
| ShortVariable | Avoid variables with short names like iv | 3 | 120 |
| CommentRequired | Public method and constructor comments are required | 3 | 131–144 |
| ShortVariable | Avoid variables with short names like md | 3 | 135 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 43–90 |
| ClassWithOnlyPrivateConstructorsShouldBeFinal | A class which only has private constructors should be final | 1 | 43–90 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'bytes' | 2 | 60 |
| ShortVariable | Avoid variables with short names like ci | 3 | 66 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 3 | 50–117 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the getItemsFromIds method if you want a default access modifier | 3 | 116 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 116 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 3 | 42–47 |
| AbstractClassWithoutAnyMethod | No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. | 1 | 42–47 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 3 | 34–341 |
| CommentRequired | Header comments are required | 3 | 79–341 |
| CommentRequired | Public method and constructor comments are required | 3 | 117–131 |
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 3 | 173 |
| CommentRequired | Public method and constructor comments are required | 3 | 174–187 |
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 3 | 189 |
| NPathComplexity | The method 'getInfos(HttpServletRequest)' has an NPath complexity of 240 | 3 | 190–233 |
| CyclomaticComplexity | The method 'getInfos(HttpServletRequest)' has a cyclomatic complexity of 12. | 3 | 190–233 |
| CommentRequired | Public method and constructor comments are required | 3 | 190–233 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 192 |
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 3 | 317 |
| CommentRequired | Public method and constructor comments are required | 3 | 318–340 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| DataClass | The class 'FormBean' is suspected to be a Data Class (WOC=0.000%, NOPA=0, NOAM=14, WMC=14) | 3 | 36–115 |
| CommentRequired | Header comments are required | 3 | 36–115 |
| FormalParameterNamingConventions | The method parameter name '_allocataireCaf' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 51 |
| FormalParameterNamingConventions | The method parameter name '_numAllocataire' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 61 |
| FormalParameterNamingConventions | The method parameter name '_codePostal' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 71 |
| FormalParameterNamingConventions | The method parameter name '_attestationNonCaf' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 81 |
| FormalParameterNamingConventions | The method parameter name '_numFiscal' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 91 |
| FormalParameterNamingConventions | The method parameter name '_referenceAvisImposition' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 101 |
| FormalParameterNamingConventions | The method parameter name '_fromJS' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 111 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 3 | 66–330 |
| ImmutableField | Private field '_trancheHome' could be made final; it is only initialized in the declaration or constructor. | 3 | 114 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 130 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 148–155 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the getItemsFromIds method if you want a default access modifier | 3 | 149–155 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 175 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 3 | 56–137 |
| FieldNamingConventions | The final field name '_userDataHome' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 82 |
