The following document contains the results of PMD 6.13.0.
| Rule | Violation | Line |
|---|---|---|
| LocalVariableNamingConventions | The local variable name '_isSwearWords' doesn't match '[a-z][a-zA-Z0-9]*' | 85 |
| LocalVariableNamingConventions | The local variable name '_isSwearWords' doesn't match '[a-z][a-zA-Z0-9]*' | 131 |
| AvoidThrowingNullPointerException | Avoid throwing null pointer exceptions. | 174 |
| Rule | Violation | Line |
|---|---|---|
| LocalVariableNamingConventions | The local variable name '_filterService' doesn't match '[a-z][a-zA-Z0-9]*' | 118 |
| Rule | Violation | Line |
|---|---|---|
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 159 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 184 |
| Rule | Violation | Line |
|---|---|---|
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 155 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 180 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 39–66 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'checkString': the method is declared in an interface type | 48 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'checkStringCounter': the method is declared in an interface type | 57 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'getCounterSwearWords': the method is declared in an interface type | 65 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 50–199 |
| NonThreadSafeSingleton | Singleton is not thread safe | 64–67 |
| ShortVariable | Avoid variables with short names like p | 77 |
| ShortVariable | Avoid variables with short names like p | 112 |
| UnnecessaryCaseChange | Using equalsIgnoreCase() is cleaner than using toUpperCase/toLowerCase().equals(). | 179 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 179 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 179 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 180 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 180 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 39–125 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 41 |
| ImmutableField | Private field '_strWords' could be made final; it is only initialized in the declaration or constructor. | 41 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '_bIsSwearWords' | 43 |
| Rule | Violation | Line |
|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'JSON_WORD_NOT_AUTHORIZED'. | 86 |
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 112 |
| LinguisticNaming | Linguistics Antipattern - The method 'isSwearWord' indicates linguistically it returns a boolean, but it returns 'String' | 113–142 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 123–130 |
| Rule | Violation | Line |
|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 50–92 |
| SingularField | Perhaps '_nDefaultItemsPerPage' could be replaced by a local variable. | 60 |
| AddEmptyString | Do not add empty strings | 86 |
| Rule | Violation | Line |
|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 54–241 |
| UnusedPrivateField | Avoid unused private fields such as 'PROPERTY_DEFAULT_LIST_WORD_PER_PAGE'. | 79 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 109 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 127 |
| Rule | Violation | Line |
|---|---|---|
| UnusedImports | Avoid unused imports such as 'org.hibernate.validator.constraints' | 36 |
| UnusedImports | Avoid unused imports such as 'javax.validation.constraints' | 38 |
| Rule | Violation | Line |
|---|---|---|
| ShortClassName | Avoid short class names like Word | 43–92 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedImports | Avoid unused imports such as 'org.hibernate.validator.constraints' | 4 | 36 |
| UnusedImports | Avoid unused imports such as 'javax.validation.constraints' | 4 | 38 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 159 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 184 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ShortClassName | Avoid short class names like Word | 4 | 43–92 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 155 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 180 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 39–66 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'checkString': the method is declared in an interface type | 3 | 48 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'checkStringCounter': the method is declared in an interface type | 3 | 57 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'getCounterSwearWords': the method is declared in an interface type | 3 | 65 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 50–199 |
| NonThreadSafeSingleton | Singleton is not thread safe | 3 | 64–67 |
| ShortVariable | Avoid variables with short names like p | 3 | 77 |
| LocalVariableNamingConventions | The local variable name '_isSwearWords' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 85 |
| ShortVariable | Avoid variables with short names like p | 3 | 112 |
| LocalVariableNamingConventions | The local variable name '_isSwearWords' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 131 |
| AvoidThrowingNullPointerException | Avoid throwing null pointer exceptions. | 1 | 174 |
| UnnecessaryCaseChange | Using equalsIgnoreCase() is cleaner than using toUpperCase/toLowerCase().equals(). | 3 | 179 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 3 | 179 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 3 | 179 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 3 | 180 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 3 | 180 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 39–125 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 41 |
| ImmutableField | Private field '_strWords' could be made final; it is only initialized in the declaration or constructor. | 3 | 41 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '_bIsSwearWords' | 3 | 43 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'JSON_WORD_NOT_AUTHORIZED'. | 3 | 86 |
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 3 | 112 |
| LinguisticNaming | Linguistics Antipattern - The method 'isSwearWord' indicates linguistically it returns a boolean, but it returns 'String' | 3 | 113–142 |
| LocalVariableNamingConventions | The local variable name '_filterService' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 118 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 123–130 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 3 | 50–92 |
| SingularField | Perhaps '_nDefaultItemsPerPage' could be replaced by a local variable. | 3 | 60 |
| AddEmptyString | Do not add empty strings | 3 | 86 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 3 | 54–241 |
| UnusedPrivateField | Avoid unused private fields such as 'PROPERTY_DEFAULT_LIST_WORD_PER_PAGE'. | 3 | 79 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 109 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 127 |
