The following document contains the results of PMD 6.13.0.
Rule | Violation | Line |
---|---|---|
FormalParameterNamingConventions | The method parameter name '_nIdAttribute' doesn't match '[a-z][a-zA-Z0-9]*' | 143 |
FormalParameterNamingConventions | The method parameter name '_strCertifierCode' doesn't match '[a-z][a-zA-Z0-9]*' | 153 |
FormalParameterNamingConventions | The method parameter name '_certificateDate' doesn't match '[a-z][a-zA-Z0-9]*' | 164 |
FormalParameterNamingConventions | The method parameter name '_certificateExpirationDate' doesn't match '[a-z][a-zA-Z0-9]*' | 175 |
FormalParameterNamingConventions | The method parameter name '_strCertificateLevel' doesn't match '[a-z][a-zA-Z0-9]*' | 185 |
Rule | Violation | Line |
---|---|---|
FormalParameterNamingConventions | The method parameter name '_dateLastUpdate' doesn't match '[a-z][a-zA-Z0-9]*' | 77 |
FormalParameterNamingConventions | The method parameter name '_mapAttributes' doesn't match '[a-z][a-zA-Z0-9]*' | 87 |
FormalParameterNamingConventions | The method parameter name '_dateCreation' doesn't match '[a-z][a-zA-Z0-9]*' | 98 |
FormalParameterNamingConventions | The method parameter name '_dateDeleted' doesn't match '[a-z][a-zA-Z0-9]*' | 109 |
FormalParameterNamingConventions | The method parameter name '_strConnectionId' doesn't match '[a-z][a-zA-Z0-9]*' | 119 |
FormalParameterNamingConventions | The method parameter name '_strCustomerId' doesn't match '[a-z][a-zA-Z0-9]*' | 129 |
Rule | Violation | Line |
---|---|---|
LocalVariableNamingConventions | The local variable name 'ListIdentityAttributes' doesn't match '[a-z][a-zA-Z0-9]*' | 68 |
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifiers 'public static final' on field 'BEAN_NAME': the field is declared in an interface type | 47 |
CommentRequired | Public method and constructor comments are required | 49 |
CommentRequired | Public method and constructor comments are required | 51 |
CommentRequired | Public method and constructor comments are required | 53 |
CommentRequired | Public method and constructor comments are required | 55 |
Rule | Violation | Line |
---|---|---|
CommentRequired | Public method and constructor comments are required | 138–141 |
CommentRequired | Public method and constructor comments are required | 143–146 |
CommentRequired | Public method and constructor comments are required | 148–151 |
CommentRequired | Public method and constructor comments are required | 153–156 |
CommentRequired | Public method and constructor comments are required | 180–183 |
CommentRequired | Public method and constructor comments are required | 185–188 |
Rule | Violation | Line |
---|---|---|
CommentRequired | Public method and constructor comments are required | 75–79 |
CommentRequired | Public method and constructor comments are required | 81–85 |
CommentRequired | Public method and constructor comments are required | 87–91 |
Rule | Violation | Line |
---|---|---|
CommentRequired | Header comments are required | 45–134 |
CallSuperInConstructor | It is a good practice to call super() in a constructor | 57–69 |
CommentRequired | Public method and constructor comments are required | 57–69 |
CommentRequired | Public method and constructor comments are required | 72–75 |
CommentRequired | Public method and constructor comments are required | 77–80 |
CommentRequired | Public method and constructor comments are required | 82–85 |
CommentRequired | Public method and constructor comments are required | 87–90 |
CommentRequired | Public method and constructor comments are required | 114–117 |
CommentRequired | Public method and constructor comments are required | 119–122 |
CommentRequired | Public method and constructor comments are required | 124–127 |
CommentRequired | Public method and constructor comments are required | 129–132 |
Rule | Violation | Line |
---|---|---|
UnusedLocalVariable | Avoid unused local variables such as 'id'. | 72 |
ControlStatementBraces | This statement should have braces | 72–75 |
AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 77 |
UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 161 |
Rule | Violation | Line |
---|---|---|
CommentRequired | Header comments are required | 49–118 |
DefaultPackage | Use explicit scoping instead of the default package private level | 51 |
CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the _mapAttributes field if you want a default access modifier | 51 |
RedundantFieldInitializer | Avoid using redundant field initializer for '_mapAttributes' | 51 |
AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 62–97 |
UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 101 |
LinguisticNaming | Linguistics Antipattern - The getter 'getAttributesList' should not return void linguistically | 110–116 |
Rule | Violation | Line |
---|---|---|
UnusedImports | Avoid unused imports such as 'java.util.List' | 38 |
Rule | Violation | Line |
---|---|---|
UnusedImports | Avoid unused imports such as 'java.sql.Date' | 36 |
Rule | Violation | Line |
---|---|---|
UnusedImports | Avoid unused imports such as 'org.apache.commons.lang3.StringUtils' | 38 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifiers 'public static final' on field 'BEAN_NAME': the field is declared in an interface type | 3 | 47 |
CommentRequired | Public method and constructor comments are required | 3 | 49 |
CommentRequired | Public method and constructor comments are required | 3 | 51 |
CommentRequired | Public method and constructor comments are required | 3 | 53 |
CommentRequired | Public method and constructor comments are required | 3 | 55 |
Rule | Violation | Priority | Line |
---|---|---|---|
CommentRequired | Public method and constructor comments are required | 3 | 138–141 |
CommentRequired | Public method and constructor comments are required | 3 | 143–146 |
FormalParameterNamingConventions | The method parameter name '_nIdAttribute' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 143 |
CommentRequired | Public method and constructor comments are required | 3 | 148–151 |
CommentRequired | Public method and constructor comments are required | 3 | 153–156 |
FormalParameterNamingConventions | The method parameter name '_strCertifierCode' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 153 |
FormalParameterNamingConventions | The method parameter name '_certificateDate' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 164 |
FormalParameterNamingConventions | The method parameter name '_certificateExpirationDate' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 175 |
CommentRequired | Public method and constructor comments are required | 3 | 180–183 |
CommentRequired | Public method and constructor comments are required | 3 | 185–188 |
FormalParameterNamingConventions | The method parameter name '_strCertificateLevel' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 185 |
Rule | Violation | Priority | Line |
---|---|---|---|
CommentRequired | Public method and constructor comments are required | 3 | 75–79 |
CommentRequired | Public method and constructor comments are required | 3 | 81–85 |
CommentRequired | Public method and constructor comments are required | 3 | 87–91 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnusedImports | Avoid unused imports such as 'java.util.List' | 4 | 38 |
CommentRequired | Header comments are required | 3 | 45–134 |
CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 57–69 |
CommentRequired | Public method and constructor comments are required | 3 | 57–69 |
CommentRequired | Public method and constructor comments are required | 3 | 72–75 |
FormalParameterNamingConventions | The method parameter name '_dateLastUpdate' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 77 |
CommentRequired | Public method and constructor comments are required | 3 | 77–80 |
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 '_mapAttributes' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 87 |
FormalParameterNamingConventions | The method parameter name '_dateCreation' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 98 |
FormalParameterNamingConventions | The method parameter name '_dateDeleted' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 109 |
CommentRequired | Public method and constructor comments are required | 3 | 114–117 |
FormalParameterNamingConventions | The method parameter name '_strConnectionId' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 119 |
CommentRequired | Public method and constructor comments are required | 3 | 119–122 |
CommentRequired | Public method and constructor comments are required | 3 | 124–127 |
FormalParameterNamingConventions | The method parameter name '_strCustomerId' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 129 |
CommentRequired | Public method and constructor comments are required | 3 | 129–132 |
Rule | Violation | Priority | Line |
---|---|---|---|
LocalVariableNamingConventions | The local variable name 'ListIdentityAttributes' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 68 |
UnusedLocalVariable | Avoid unused local variables such as 'id'. | 3 | 72 |
ControlStatementBraces | This statement should have braces | 3 | 72–75 |
AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 77 |
UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 161 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnusedImports | Avoid unused imports such as 'java.sql.Date' | 4 | 36 |
CommentRequired | Header comments are required | 3 | 49–118 |
DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 51 |
CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the _mapAttributes field if you want a default access modifier | 3 | 51 |
RedundantFieldInitializer | Avoid using redundant field initializer for '_mapAttributes' | 3 | 51 |
AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 62–97 |
UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 101 |
LinguisticNaming | Linguistics Antipattern - The getter 'getAttributesList' should not return void linguistically | 3 | 110–116 |
Rule | Violation | Priority | Line |
---|---|---|---|
UnusedImports | Avoid unused imports such as 'org.apache.commons.lang3.StringUtils' | 4 | 38 |