The following document contains the results of PMD 6.13.0.
| Rule | Violation | Line |
|---|---|---|
| LocalVariableNamingConventions | The local variable name 'FormResponseHistoryDataObject' doesn't match '[a-z][a-zA-Z0-9]*' | 258 |
| Rule | Violation | Line |
|---|---|---|
| MissingOverride | The method 'getId()' is missing an @Override annotation. | 66–69 |
| MissingOverride | The method 'setId(String)' is missing an @Override annotation. | 77–80 |
| MissingOverride | The method 'getParentId()' is missing an @Override annotation. | 108–111 |
| MissingOverride | The method 'setParentId(String)' is missing an @Override annotation. | 119–122 |
| MissingOverride | The method 'getParentName()' is missing an @Override annotation. | 129–132 |
| MissingOverride | The method 'setParentName(String)' is missing an @Override annotation. | 140–143 |
| MissingOverride | The method 'getDocumentTypeName()' is missing an @Override annotation. | 150–153 |
| MissingOverride | The method 'setDocumentTypeName(String)' is missing an @Override annotation. | 161–164 |
| Rule | Violation | Line |
|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 34–459 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the _mapFields field if you want a default access modifier | 95 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 95 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 95 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 101 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 110–136 |
| UseArrayListInsteadOfVector | Use ArrayList instead of Vector | 112 |
| EmptyStatementNotInLoop | An empty statement (semicolon) not part of a loop | 196 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 225–229 |
| EmptyStatementNotInLoop | An empty statement (semicolon) not part of a loop | 285 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 350 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 383–390 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 447 |
| Rule | Violation | Line |
|---|---|---|
| MissingOverride | The method 'loadByQuestionId(int, Plugin)' is missing an @Override annotation. | 102–119 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 157 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 180 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 204 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 43–109 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 46 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the _formsDataSource field if you want a default access modifier | 46 |
| DoNotUseThreads | To be compliant to J2EE, a webapp should not use any thread. | 60 |
| DoNotUseThreads | To be compliant to J2EE, a webapp should not use any thread. | 71 |
| DoNotUseThreads | To be compliant to J2EE, a webapp should not use any thread. | 82 |
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 111 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 176–192 |
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.sql.Timestamp' due to existing import 'java.sql.Timestamp' | 426 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.sql.Timestamp' due to existing import 'java.sql.Timestamp' | 426 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingOverride | The method 'getId()' is missing an @Override annotation. | 3 | 66–69 |
| MissingOverride | The method 'setId(String)' is missing an @Override annotation. | 3 | 77–80 |
| MissingOverride | The method 'getParentId()' is missing an @Override annotation. | 3 | 108–111 |
| MissingOverride | The method 'setParentId(String)' is missing an @Override annotation. | 3 | 119–122 |
| MissingOverride | The method 'getParentName()' is missing an @Override annotation. | 3 | 129–132 |
| MissingOverride | The method 'setParentName(String)' is missing an @Override annotation. | 3 | 140–143 |
| MissingOverride | The method 'getDocumentTypeName()' is missing an @Override annotation. | 3 | 150–153 |
| MissingOverride | The method 'setDocumentTypeName(String)' is missing an @Override annotation. | 3 | 161–164 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 3 | 34–459 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the _mapFields field if you want a default access modifier | 3 | 95 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 95 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 95 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 101 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 110–136 |
| UseArrayListInsteadOfVector | Use ArrayList instead of Vector | 3 | 112 |
| EmptyStatementNotInLoop | An empty statement (semicolon) not part of a loop | 3 | 196 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 3 | 225–229 |
| LocalVariableNamingConventions | The local variable name 'FormResponseHistoryDataObject' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 258 |
| EmptyStatementNotInLoop | An empty statement (semicolon) not part of a loop | 3 | 285 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 350 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 383–390 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.sql.Timestamp' due to existing import 'java.sql.Timestamp' | 4 | 426 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.sql.Timestamp' due to existing import 'java.sql.Timestamp' | 4 | 426 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 447 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingOverride | The method 'loadByQuestionId(int, Plugin)' is missing an @Override annotation. | 3 | 102–119 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 157 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 180 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 204 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 43–109 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 46 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the _formsDataSource field if you want a default access modifier | 3 | 46 |
| DoNotUseThreads | To be compliant to J2EE, a webapp should not use any thread. | 3 | 60 |
| DoNotUseThreads | To be compliant to J2EE, a webapp should not use any thread. | 3 | 71 |
| DoNotUseThreads | To be compliant to J2EE, a webapp should not use any thread. | 3 | 82 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 3 | 111 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 176–192 |
