The following document contains the results of PMD 6.13.0.
| Rule | Violation | Line |
|---|---|---|
| AvoidThrowingRawExceptionTypes | Avoid throwing raw exception types. | 150 |
| AvoidThrowingRawExceptionTypes | Avoid throwing raw exception types. | 154 |
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The constant name '_objectMapper' doesn't match '[A-Z][A-Z_0-9]*' | 64 |
| Rule | Violation | Line |
|---|---|---|
| ClassNamingConventions | The utility class name 'Constants' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' | 3–44 |
| ClassWithOnlyPrivateConstructorsShouldBeFinal | A class which only has private constructors should be final | 3–44 |
| Rule | Violation | Line |
|---|---|---|
| ClassWithOnlyPrivateConstructorsShouldBeFinal | A class which only has private constructors should be final | 38–60 |
| FieldNamingConventions | The static field name 'instance' doesn't match '_[a-z][a-zA-Z0-9]*' | 40 |
| Rule | Violation | Line |
|---|---|---|
| LoggerIsNotStaticFinal | The Logger variable declaration does not contain the static and final modifiers | 57 |
| GuardLogStatement | Logger calls should be surrounded by log level guards. | 89 |
| GuardLogStatement | Logger calls should be surrounded by log level guards. | 104 |
| GuardLogStatement | Logger calls should be surrounded by log level guards. | 120 |
| GuardLogStatement | Logger calls should be surrounded by log level guards. | 135 |
| Rule | Violation | Line |
|---|---|---|
| LoggerIsNotStaticFinal | The Logger variable declaration does not contain the static and final modifiers | 59 |
| Rule | Violation | Line |
|---|---|---|
| LoggerIsNotStaticFinal | The Logger variable declaration does not contain the static and final modifiers | 63 |
| GuardLogStatement | Logger calls should be surrounded by log level guards. | 102 |
| GuardLogStatement | Logger calls should be surrounded by log level guards. | 139 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 11–224 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 11–103 |
| Rule | Violation | Line |
|---|---|---|
| AbstractClassWithoutAbstractMethod | This abstract class does not have any abstract methods | 44–89 |
| Rule | Violation | Line |
|---|---|---|
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 90 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 105 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 121 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 136 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Public method and constructor comments are required | 64–67 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 92 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 123 |
| ShortVariable | Avoid variables with short names like e | 141 |
| SignatureDeclareThrowsException | A method/constructor should not explicitly throw java.lang.Exception | 141 |
| SignatureDeclareThrowsException | A method/constructor should not explicitly throw java.lang.Exception | 165 |
| MissingOverride | The method 'setApiEndPointUrl(String)' is missing an @Override annotation. | 175–179 |
| MissingOverride | The method 'getApiEndPointUrl()' is missing an @Override annotation. | 186–190 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 214 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 245 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 254 |
| Rule | Violation | Line |
|---|---|---|
| SignatureDeclareThrowsException | A method/constructor should not explicitly throw java.lang.Exception | 98 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 104 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 104 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 105 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 105 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 113 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 120 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 3–44 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 38–60 |
| CommentRequired | Public method and constructor comments are required | 42–49 |
| NonThreadSafeSingleton | Singleton is not thread safe | 44–47 |
| ShortVariable | Avoid variables with short names like i | 56 |
| Rule | Violation | Line |
|---|---|---|
| SignatureDeclareThrowsException | A method/constructor should not explicitly throw java.lang.Exception | 91 |
| SignatureDeclareThrowsException | A method/constructor should not explicitly throw java.lang.Exception | 106 |
| SignatureDeclareThrowsException | A method/constructor should not explicitly throw java.lang.Exception | 121 |
| Rule | Violation | Line |
|---|---|---|
| SignatureDeclareThrowsException | A method/constructor should not explicitly throw java.lang.Exception | 58 |
| SignatureDeclareThrowsException | A method/constructor should not explicitly throw java.lang.Exception | 70 |
| SignatureDeclareThrowsException | A method/constructor should not explicitly throw java.lang.Exception | 82 |
| SignatureDeclareThrowsException | A method/constructor should not explicitly throw java.lang.Exception | 94 |
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'setApiEndPointUrl': the method is declared in an interface type | 52 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'getApiEndPointUrl': the method is declared in an interface type | 59 |
| CommentRequired | Public method and constructor comments are required | 61–63 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'doGet': the method is declared in an interface type | 61–63 |
| SignatureDeclareThrowsException | A method/constructor should not explicitly throw java.lang.Exception | 63 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'doGetList': the method is declared in an interface type | 66–67 |
| CommentRequired | Public method and constructor comments are required | 66–67 |
| SignatureDeclareThrowsException | A method/constructor should not explicitly throw java.lang.Exception | 67 |
| Rule | Violation | Line |
|---|---|---|
| ShortClassName | Avoid short class names like City | 11–224 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ShortClassName | Avoid short class names like City | 4 | 11–224 |
| CommentRequired | Header comments are required | 3 | 11–224 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 11–103 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AbstractClassWithoutAbstractMethod | This abstract class does not have any abstract methods | 3 | 44–89 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| LoggerIsNotStaticFinal | The Logger variable declaration does not contain the static and final modifiers | 2 | 57 |
| GuardLogStatement | Logger calls should be surrounded by log level guards. | 2 | 89 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 90 |
| GuardLogStatement | Logger calls should be surrounded by log level guards. | 2 | 104 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 105 |
| GuardLogStatement | Logger calls should be surrounded by log level guards. | 2 | 120 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 121 |
| GuardLogStatement | Logger calls should be surrounded by log level guards. | 2 | 135 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 136 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| LoggerIsNotStaticFinal | The Logger variable declaration does not contain the static and final modifiers | 2 | 59 |
| CommentRequired | Public method and constructor comments are required | 3 | 64–67 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 92 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 123 |
| ShortVariable | Avoid variables with short names like e | 3 | 141 |
| SignatureDeclareThrowsException | A method/constructor should not explicitly throw java.lang.Exception | 3 | 141 |
| AvoidThrowingRawExceptionTypes | Avoid throwing raw exception types. | 1 | 150 |
| AvoidThrowingRawExceptionTypes | Avoid throwing raw exception types. | 1 | 154 |
| SignatureDeclareThrowsException | A method/constructor should not explicitly throw java.lang.Exception | 3 | 165 |
| MissingOverride | The method 'setApiEndPointUrl(String)' is missing an @Override annotation. | 3 | 175–179 |
| MissingOverride | The method 'getApiEndPointUrl()' is missing an @Override annotation. | 3 | 186–190 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 214 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 245 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 254 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| LoggerIsNotStaticFinal | The Logger variable declaration does not contain the static and final modifiers | 2 | 63 |
| FieldNamingConventions | The constant name '_objectMapper' doesn't match '[A-Z][A-Z_0-9]*' | 1 | 64 |
| SignatureDeclareThrowsException | A method/constructor should not explicitly throw java.lang.Exception | 3 | 98 |
| GuardLogStatement | Logger calls should be surrounded by log level guards. | 2 | 102 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 104 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 104 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 105 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 105 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 113 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 120 |
| GuardLogStatement | Logger calls should be surrounded by log level guards. | 2 | 139 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ClassNamingConventions | The utility class name 'Constants' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' | 1 | 3–44 |
| ClassWithOnlyPrivateConstructorsShouldBeFinal | A class which only has private constructors should be final | 1 | 3–44 |
| CommentRequired | Header comments are required | 3 | 3–44 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 38–60 |
| ClassWithOnlyPrivateConstructorsShouldBeFinal | A class which only has private constructors should be final | 1 | 38–60 |
| FieldNamingConventions | The static field name 'instance' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 40 |
| CommentRequired | Public method and constructor comments are required | 3 | 42–49 |
| NonThreadSafeSingleton | Singleton is not thread safe | 3 | 44–47 |
| ShortVariable | Avoid variables with short names like i | 3 | 56 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| SignatureDeclareThrowsException | A method/constructor should not explicitly throw java.lang.Exception | 3 | 91 |
| SignatureDeclareThrowsException | A method/constructor should not explicitly throw java.lang.Exception | 3 | 106 |
| SignatureDeclareThrowsException | A method/constructor should not explicitly throw java.lang.Exception | 3 | 121 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| SignatureDeclareThrowsException | A method/constructor should not explicitly throw java.lang.Exception | 3 | 58 |
| SignatureDeclareThrowsException | A method/constructor should not explicitly throw java.lang.Exception | 3 | 70 |
| SignatureDeclareThrowsException | A method/constructor should not explicitly throw java.lang.Exception | 3 | 82 |
| SignatureDeclareThrowsException | A method/constructor should not explicitly throw java.lang.Exception | 3 | 94 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'setApiEndPointUrl': the method is declared in an interface type | 3 | 52 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'getApiEndPointUrl': the method is declared in an interface type | 3 | 59 |
| CommentRequired | Public method and constructor comments are required | 3 | 61–63 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'doGet': the method is declared in an interface type | 3 | 61–63 |
| SignatureDeclareThrowsException | A method/constructor should not explicitly throw java.lang.Exception | 3 | 63 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'doGetList': the method is declared in an interface type | 3 | 66–67 |
| CommentRequired | Public method and constructor comments are required | 3 | 66–67 |
| SignatureDeclareThrowsException | A method/constructor should not explicitly throw java.lang.Exception | 3 | 67 |
