The following document contains the results of PMD 6.13.0.
| Rule | Violation | Line |
|---|---|---|
| ClassNamingConventions | The utility class name 'Constants' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' | 39–73 |
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The final field name '_logger' doesn't match '[a-z][a-zA-Z0-9]*' | 57 |
| LocalVariableNamingConventions | The local variable name '_service' doesn't match '[a-z][a-zA-Z0-9]*' | 90 |
| LocalVariableNamingConventions | The local variable name '_service' doesn't match '[a-z][a-zA-Z0-9]*' | 134 |
| LocalVariableNamingConventions | The local variable name '_service' doesn't match '[a-z][a-zA-Z0-9]*' | 178 |
| LocalVariableNamingConventions | The local variable name '_service' doesn't match '[a-z][a-zA-Z0-9]*' | 222 |
| Rule | Violation | Line |
|---|---|---|
| LoggerIsNotStaticFinal | The Logger variable declaration does not contain the static and final modifiers | 57 |
| Rule | Violation | Line |
|---|---|---|
| RedundantFieldInitializer | Avoid using redundant field initializer for '_nNbItemTotal' | 51 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '_nNbItemSuccess' | 53 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '_nNbItemFailure' | 55 |
| ImmutableField | Private field '_report' could be made final; it is only initialized in the declaration or constructor. | 57 |
| UseCollectionIsEmpty | Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() | 179 |
| Rule | Violation | Line |
|---|---|---|
| ShortVariable | Avoid variables with short names like ID | 45 |
| Rule | Violation | Line |
|---|---|---|
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 69–77 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 86 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 140–146 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 154–160 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 260–268 |
| Rule | Violation | Line |
|---|---|---|
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 85 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| RedundantFieldInitializer | Avoid using redundant field initializer for '_nNbItemTotal' | 3 | 51 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '_nNbItemSuccess' | 3 | 53 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '_nNbItemFailure' | 3 | 55 |
| ImmutableField | Private field '_report' could be made final; it is only initialized in the declaration or constructor. | 3 | 57 |
| UseCollectionIsEmpty | Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() | 3 | 179 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ClassNamingConventions | The utility class name 'Constants' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' | 1 | 39–73 |
| ShortVariable | Avoid variables with short names like ID | 3 | 45 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| LoggerIsNotStaticFinal | The Logger variable declaration does not contain the static and final modifiers | 2 | 57 |
| FieldNamingConventions | The final field name '_logger' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 57 |
| LocalVariableNamingConventions | The local variable name '_service' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 90 |
| LocalVariableNamingConventions | The local variable name '_service' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 134 |
| LocalVariableNamingConventions | The local variable name '_service' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 178 |
| LocalVariableNamingConventions | The local variable name '_service' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 222 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 69–77 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 86 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 140–146 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 154–160 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 260–268 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 85 |
