The following document contains the results of PMD 6.13.0.
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The field name 'productService' doesn't match '_[a-z][a-zA-Z0-9]*' | 106 |
| LocalVariableNamingConventions | The local variable name '_productDAO' doesn't match '[a-z][a-zA-Z0-9]*' | 349 |
| Rule | Violation | Line |
|---|---|---|
| LocalVariableNamingConventions | The local variable name '_productService' doesn't match '[a-z][a-zA-Z0-9]*' | 70 |
| Rule | Violation | Line |
|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 34–380 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 99 |
| CommentRequired | Public method and constructor comments are required | 124–127 |
| MissingOverride | The method 'isEnable()' is missing an @Override annotation. | 124–127 |
| MissingOverride | The method 'indexDocuments()' is missing an @Override annotation. | 133–159 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 135 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 151 |
| ExcessiveMethodLength | Avoid really long methods. | 164–264 |
| CyclomaticComplexity | The method 'getItem(Product)' has a cyclomatic complexity of 11. | 164–264 |
| NPathComplexity | The method 'getItem(Product)' has an NPath complexity of 384 | 164–264 |
| UnnecessaryWrapperObjectCreation | Unnecessary wrapper object creation | 169 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 181 |
| AddEmptyString | Do not add empty strings | 205 |
| CollapsibleIfStatements | These nested if statements could be combined | 228–232 |
| MissingOverride | The method 'getName()' is missing an @Override annotation. | 271–274 |
| MissingOverride | The method 'getVersion()' is missing an @Override annotation. | 280–283 |
| MissingOverride | The method 'getDescription()' is missing an @Override annotation. | 288–291 |
| MissingOverride | The method 'getAdditionalFields()' is missing an @Override annotation. | 296–338 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 298 |
| UnnecessaryLocalBeforeReturn | Consider simply returning the value vs storing it in local variable 'lstFields' | 299 |
| MissingOverride | The method 'getDocuments(String)' is missing an @Override annotation. | 344–359 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 354 |
| MissingOverride | The method 'getResourcesName()' is missing an @Override annotation. | 364–367 |
| MissingOverride | The method 'getResourceUid(String, String)' is missing an @Override annotation. | 372–378 |
| ShortVariable | Avoid variables with short names like sb | 374 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 39–42 |
| Rule | Violation | Line |
|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 1–269 |
| CommentRequired | Header comments are required | 38–269 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 43–45 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the _productService field if you want a default access modifier | 45 |
| CommentRequired | Public method and constructor comments are required | 48–54 |
| CommentRequired | Public method and constructor comments are required | 57–63 |
| CommentRequired | Public method and constructor comments are required | 66–72 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 43–87 |
| UseUtilityClass | All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. | 44–87 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 50–97 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 3 | 34–380 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 99 |
| FieldNamingConventions | The field name 'productService' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 106 |
| CommentRequired | Public method and constructor comments are required | 3 | 124–127 |
| MissingOverride | The method 'isEnable()' is missing an @Override annotation. | 3 | 124–127 |
| MissingOverride | The method 'indexDocuments()' is missing an @Override annotation. | 3 | 133–159 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 135 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 151 |
| ExcessiveMethodLength | Avoid really long methods. | 3 | 164–264 |
| CyclomaticComplexity | The method 'getItem(Product)' has a cyclomatic complexity of 11. | 3 | 164–264 |
| NPathComplexity | The method 'getItem(Product)' has an NPath complexity of 384 | 3 | 164–264 |
| UnnecessaryWrapperObjectCreation | Unnecessary wrapper object creation | 3 | 169 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 181 |
| AddEmptyString | Do not add empty strings | 3 | 205 |
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 228–232 |
| MissingOverride | The method 'getName()' is missing an @Override annotation. | 3 | 271–274 |
| MissingOverride | The method 'getVersion()' is missing an @Override annotation. | 3 | 280–283 |
| MissingOverride | The method 'getDescription()' is missing an @Override annotation. | 3 | 288–291 |
| MissingOverride | The method 'getAdditionalFields()' is missing an @Override annotation. | 3 | 296–338 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 298 |
| UnnecessaryLocalBeforeReturn | Consider simply returning the value vs storing it in local variable 'lstFields' | 3 | 299 |
| MissingOverride | The method 'getDocuments(String)' is missing an @Override annotation. | 3 | 344–359 |
| LocalVariableNamingConventions | The local variable name '_productDAO' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 349 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 354 |
| MissingOverride | The method 'getResourcesName()' is missing an @Override annotation. | 3 | 364–367 |
| MissingOverride | The method 'getResourceUid(String, String)' is missing an @Override annotation. | 3 | 372–378 |
| ShortVariable | Avoid variables with short names like sb | 3 | 374 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 39–42 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 3 | 1–269 |
| CommentRequired | Header comments are required | 3 | 38–269 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 43–45 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the _productService field if you want a default access modifier | 3 | 45 |
| CommentRequired | Public method and constructor comments are required | 3 | 48–54 |
| CommentRequired | Public method and constructor comments are required | 3 | 57–63 |
| CommentRequired | Public method and constructor comments are required | 3 | 66–72 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 43–87 |
| UseUtilityClass | All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. | 3 | 44–87 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 50–97 |
| LocalVariableNamingConventions | The local variable name '_productService' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 70 |
