The following document contains the results of PMD 6.13.0.
| Rule | Violation | Line |
|---|---|---|
| AbstractClassWithoutAnyMethod | No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. | 40–43 |
| Rule | Violation | Line |
|---|---|---|
| ClassNamingConventions | The utility class name 'Constants' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' | 39–57 |
| Rule | Violation | Line |
|---|---|---|
| MethodNamingConventions | The instance method name 'GET' doesn't match '[a-z][a-zA-Z0-9]*' | 79–83 |
| MethodNamingConventions | The instance method name 'PUT' doesn't match '[a-z][a-zA-Z0-9]*' | 96–99 |
| MethodNamingConventions | The instance method name 'POST' doesn't match '[a-z][a-zA-Z0-9]*' | 112–115 |
| MethodNamingConventions | The instance method name 'DELETE' doesn't match '[a-z][a-zA-Z0-9]*' | 126–129 |
| Rule | Violation | Line |
|---|---|---|
| AbstractClassWithoutAbstractMethod | This abstract class does not have any abstract methods | 40–88 |
| CommentRequired | Header comments are required | 40–88 |
| CommentRequired | Public method and constructor comments are required | 45–48 |
| Rule | Violation | Line |
|---|---|---|
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 47 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 47 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 88 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 101 |
| Rule | Violation | Line |
|---|---|---|
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the _listMust field if you want a default access modifier | 51 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 51 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the _listFilter field if you want a default access modifier | 52 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 52 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the _listShould field if you want a default access modifier | 53 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 53 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 54 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the _listMustNot field if you want a default access modifier | 54 |
| CommentRequired | Public method and constructor comments are required | 73–80 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 77 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 98 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 119 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 140 |
| NPathComplexity | The method 'getNodeContent(JsonNodeFactory)' has an NPath complexity of 256 | 149–226 |
| CyclomaticComplexity | The method 'getNodeContent(JsonNodeFactory)' has a cyclomatic complexity of 13. | 149–226 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 156 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 174 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 192 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 210 |
| Rule | Violation | Line |
|---|---|---|
| RedundantFieldInitializer | Avoid using redundant field initializer for '_searchQuery' | 44 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '_nSize' | 45 |
| Rule | Violation | Line |
|---|---|---|
| TooManyMethods | This class has too many methods, consider refactoring it. | 53–485 |
| ImmutableField | Private field '_connexion' could be made final; it is only initialized in the declaration or constructor. | 55 |
| ImmutableField | Private field '_strServerUrl' could be made final; it is only initialized in the declaration or constructor. | 56 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 462 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 479 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 480 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 481 |
| Rule | Violation | Line |
|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 41–48 |
| CommentRequired | Public method and constructor comments are required | 43–47 |
| Rule | Violation | Line |
|---|---|---|
| ImmutableField | Private field '_clientHttp' could be made final; it is only initialized in the declaration or constructor. | 47 |
| UncommentedEmptyConstructor | Document empty constructor | 67–69 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AbstractClassWithoutAbstractMethod | This abstract class does not have any abstract methods | 3 | 40–88 |
| CommentRequired | Header comments are required | 3 | 40–88 |
| CommentRequired | Public method and constructor comments are required | 3 | 45–48 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 47 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 47 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 88 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 101 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AbstractClassWithoutAnyMethod | No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. | 1 | 40–43 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the _listMust field if you want a default access modifier | 3 | 51 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 51 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the _listFilter field if you want a default access modifier | 3 | 52 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 52 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the _listShould field if you want a default access modifier | 3 | 53 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 53 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 54 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the _listMustNot field if you want a default access modifier | 3 | 54 |
| CommentRequired | Public method and constructor comments are required | 3 | 73–80 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 77 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 98 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 119 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 140 |
| NPathComplexity | The method 'getNodeContent(JsonNodeFactory)' has an NPath complexity of 256 | 3 | 149–226 |
| CyclomaticComplexity | The method 'getNodeContent(JsonNodeFactory)' has a cyclomatic complexity of 13. | 3 | 149–226 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 156 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 174 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 192 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 210 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| RedundantFieldInitializer | Avoid using redundant field initializer for '_searchQuery' | 3 | 44 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '_nSize' | 3 | 45 |
| 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–57 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyMethods | This class has too many methods, consider refactoring it. | 3 | 53–485 |
| ImmutableField | Private field '_connexion' could be made final; it is only initialized in the declaration or constructor. | 3 | 55 |
| ImmutableField | Private field '_strServerUrl' could be made final; it is only initialized in the declaration or constructor. | 3 | 56 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 3 | 462 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 3 | 479 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 3 | 480 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 481 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 3 | 41–48 |
| CommentRequired | Public method and constructor comments are required | 3 | 43–47 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ImmutableField | Private field '_clientHttp' could be made final; it is only initialized in the declaration or constructor. | 3 | 47 |
| UncommentedEmptyConstructor | Document empty constructor | 3 | 67–69 |
| MethodNamingConventions | The instance method name 'GET' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 79–83 |
| MethodNamingConventions | The instance method name 'PUT' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 96–99 |
| MethodNamingConventions | The instance method name 'POST' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 112–115 |
| MethodNamingConventions | The instance method name 'DELETE' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 126–129 |
