The following document contains the results of PMD 6.13.0.
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The final field name '_strExtension' doesn't match '[a-z][a-zA-Z0-9]*' | 47 |
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The constant name '_manageConfigProducerService' doesn't match '[A-Z][A-Z_0-9]*' | 145 |
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The constant name '_manageConfigProducerService' doesn't match '[A-Z][A-Z_0-9]*' | 66 |
| Rule | Violation | Line |
|---|---|---|
| AvoidReassigningParameters | Avoid reassigning parameters such as 'listIdQuestionConfig' | 370 |
| Rule | Violation | Line |
|---|---|---|
| ShortVariable | Avoid variables with short names like id | 64 |
| Rule | Violation | Line |
|---|---|---|
| ShortVariable | Avoid variables with short names like id | 71 |
| Rule | Violation | Line |
|---|---|---|
| TooManyMethods | This class has too many methods, consider refactoring it. | 49–430 |
| Rule | Violation | Line |
|---|---|---|
| TooManyMethods | This class has too many methods, consider refactoring it. | 47–257 |
| Rule | Violation | Line |
|---|---|---|
| SwitchStmtsShouldHaveDefault | Switch statements should have a default label | 75–81 |
| TooFewBranchesForASwitchStatement | A switch with less than three branches is inefficient, use a if statement instead. | 75–81 |
| SwitchStmtsShouldHaveDefault | Switch statements should have a default label | 93–99 |
| TooFewBranchesForASwitchStatement | A switch with less than three branches is inefficient, use a if statement instead. | 93–99 |
| Rule | Violation | Line |
|---|---|---|
| TooManyMethods | This class has too many methods, consider refactoring it. | 45–204 |
| Rule | Violation | Line |
|---|---|---|
| TooManyMethods | This class has too many methods, consider refactoring it. | 50–229 |
| Rule | Violation | Line |
|---|---|---|
| ShortVariable | Avoid variables with short names like rt | 86 |
| ShortVariable | Avoid variables with short names like p | 97 |
| Rule | Violation | Line |
|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 34–447 |
| SimpleDateFormatNeedsLocale | When instantiating a SimpleDateFormat object, specify a Locale | 169 |
| CyclomaticComplexity | The method 'builderPDFWithFormQuestionResponse(Document, FormResponse, List)' has a cyclomatic complexity of 12. | 251–352 |
| ExcessiveMethodLength | Avoid really long methods. | 251–352 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 318 |
| ShortVariable | Avoid variables with short names like os | 385 |
| Rule | Violation | Line |
|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 34–558 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the _configProducer field if you want a default access modifier | 150 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 150 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 151 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the _form field if you want a default access modifier | 151 |
| CommentRequired | Public method and constructor comments are required | 154–169 |
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 179 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 200 |
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 216 |
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 245 |
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 273 |
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 294 |
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 311 |
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 340 |
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 368 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 394–401 |
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 413 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 424 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 425 |
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 464 |
| Rule | Violation | Line |
|---|---|---|
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 80 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ShortVariable | Avoid variables with short names like id | 3 | 64 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ShortVariable | Avoid variables with short names like id | 3 | 71 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyMethods | This class has too many methods, consider refactoring it. | 3 | 49–430 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyMethods | This class has too many methods, consider refactoring it. | 3 | 47–257 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| FieldNamingConventions | The final field name '_strExtension' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 47 |
| SwitchStmtsShouldHaveDefault | Switch statements should have a default label | 3 | 75–81 |
| TooFewBranchesForASwitchStatement | A switch with less than three branches is inefficient, use a if statement instead. | 3 | 75–81 |
| SwitchStmtsShouldHaveDefault | Switch statements should have a default label | 3 | 93–99 |
| TooFewBranchesForASwitchStatement | A switch with less than three branches is inefficient, use a if statement instead. | 3 | 93–99 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyMethods | This class has too many methods, consider refactoring it. | 3 | 45–204 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyMethods | This class has too many methods, consider refactoring it. | 3 | 50–229 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ShortVariable | Avoid variables with short names like rt | 3 | 86 |
| ShortVariable | Avoid variables with short names like p | 3 | 97 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 3 | 34–447 |
| SimpleDateFormatNeedsLocale | When instantiating a SimpleDateFormat object, specify a Locale | 3 | 169 |
| CyclomaticComplexity | The method 'builderPDFWithFormQuestionResponse(Document, FormResponse, List)' has a cyclomatic complexity of 12. | 3 | 251–352 |
| ExcessiveMethodLength | Avoid really long methods. | 3 | 251–352 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 318 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'listIdQuestionConfig' | 2 | 370 |
| ShortVariable | Avoid variables with short names like os | 3 | 385 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 3 | 34–558 |
| FieldNamingConventions | The constant name '_manageConfigProducerService' doesn't match '[A-Z][A-Z_0-9]*' | 1 | 145 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the _configProducer field if you want a default access modifier | 3 | 150 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 150 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 151 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the _form field if you want a default access modifier | 3 | 151 |
| CommentRequired | Public method and constructor comments are required | 3 | 154–169 |
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 3 | 179 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 200 |
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 3 | 216 |
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 3 | 245 |
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 3 | 273 |
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 3 | 294 |
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 3 | 311 |
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 3 | 340 |
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 3 | 368 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 394–401 |
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 3 | 413 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 424 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 425 |
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 3 | 464 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| FieldNamingConventions | The constant name '_manageConfigProducerService' doesn't match '[A-Z][A-Z_0-9]*' | 1 | 66 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 80 |
