The following document contains the results of PMD 6.13.0.
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The field name 'canNotify' doesn't match '_[a-z][a-zA-Z0-9]*' | 6 |
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The field name 'suspiciousIdentity' doesn't match '_[a-z][a-zA-Z0-9]*' | 41 |
| FieldNamingConventions | The field name 'bestIdentity' doesn't match '_[a-z][a-zA-Z0-9]*' | 42 |
| FieldNamingConventions | The field name 'duplicatesToMergeAttributes' doesn't match '_[a-z][a-zA-Z0-9]*' | 43 |
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The final field name '_referentialService' doesn't match '[a-z][a-zA-Z0-9]*' | 55 |
| FieldNamingConventions | The final field name '_currentClientCode' doesn't match '[a-z][a-zA-Z0-9]*' | 56 |
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The final field name '_serviceContractService' doesn't match '[a-z][a-zA-Z0-9]*' | 56 |
| FieldNamingConventions | The final field name '_sortedAttributeKeyList' doesn't match '[a-z][a-zA-Z0-9]*' | 57 |
| FieldNamingConventions | The final field name '_attributeKeyToShowList' doesn't match '[a-z][a-zA-Z0-9]*' | 58 |
| Rule | Violation | Line |
|---|---|---|
| ClassWithOnlyPrivateConstructorsShouldBeFinal | A class which only has private constructors should be final | 19–146 |
| FieldNamingConventions | The final field name '_referentialCache' doesn't match '[a-z][a-zA-Z0-9]*' | 20 |
| FieldNamingConventions | The final field name '_minCertificationCode' doesn't match '[a-z][a-zA-Z0-9]*' | 21 |
| FieldNamingConventions | The static field name 'instance' doesn't match '_[a-z][a-zA-Z0-9]*' | 22 |
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The constant name '_serviceQuality' doesn't match '[A-Z][A-Z_0-9]*' | 66 |
| FieldNamingConventions | The constant name '_serviceIdentity' doesn't match '[A-Z][A-Z_0-9]*' | 67 |
| FieldNamingConventions | The constant name '_mediationService' doesn't match '[A-Z][A-Z_0-9]*' | 68 |
| FieldNamingConventions | The constant name '_serviceContractCache' doesn't match '[A-Z][A-Z_0-9]*' | 69 |
| FieldNamingConventions | The final field name 'PARAMETERS_DUPLICATE_SEARCH' doesn't match '[a-z][a-zA-Z0-9]*' | 77 |
| FieldNamingConventions | The final field name 'PARAMETER_PAGE' doesn't match '[a-z][a-zA-Z0-9]*' | 80 |
| FieldNamingConventions | The final field name 'PARAMETER_CLIENT_CODE' doesn't match '[a-z][a-zA-Z0-9]*' | 81 |
| FieldNamingConventions | The final field name '_rulePriorityMin' doesn't match '[a-z][a-zA-Z0-9]*' | 85 |
| FieldNamingConventions | The final field name '_duplicateRules' doesn't match '[a-z][a-zA-Z0-9]*' | 91 |
| FieldNamingConventions | The final field name '_mediationIdentities' doesn't match '[a-z][a-zA-Z0-9]*' | 92 |
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The final field name 'PARAMETER_CUID_PINNED' doesn't match '[a-z][a-zA-Z0-9]*' | 123 |
| FieldNamingConventions | The final field name 'PARAMETER_CUID_TO_EXCLUDE' doesn't match '[a-z][a-zA-Z0-9]*' | 124 |
| FieldNamingConventions | The final field name 'PARAMETER_CUID' doesn't match '[a-z][a-zA-Z0-9]*' | 125 |
| LocalVariableNamingConventions | The final local variable name '_suspiciousCuid' doesn't match '[a-z][a-zA-Z0-9]*' | 353 |
| Rule | Violation | Line |
|---|---|---|
| GuardLogStatement | Logger calls should be surrounded by log level guards. | 84 |
| GuardLogStatement | Logger calls should be surrounded by log level guards. | 111 |
| GuardLogStatement | Logger calls should be surrounded by log level guards. | 138 |
| Rule | Violation | Line |
|---|---|---|
| GuardLogStatement | Logger calls should be surrounded by log level guards. | 73 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 5–37 |
| CommentRequired | Public method and constructor comments are required | 18–36 |
| Rule | Violation | Line |
|---|---|---|
| DataClass | The class 'MediationIdentity' is suspected to be a Data Class (WOC=0.000%, NOPA=0, NOAM=6, WMC=6) | 39–74 |
| CommentRequired | Header comments are required | 39–74 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 49–163 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 58–62 |
| CommentRequired | Public method and constructor comments are required | 58–62 |
| CommentRequired | Public method and constructor comments are required | 64–75 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 71 |
| CommentRequired | Public method and constructor comments are required | 87–102 |
| CommentRequired | Public method and constructor comments are required | 114–129 |
| CommentRequired | Public method and constructor comments are required | 141–156 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 51–166 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 60–64 |
| CommentRequired | Public method and constructor comments are required | 60–64 |
| CommentRequired | Public method and constructor comments are required | 76–85 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 115 |
| ShortVariable | Avoid variables with short names like i1 | 135 |
| ShortVariable | Avoid variables with short names like i2 | 136 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 5–23 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 11–58 |
| ShortVariable | Avoid variables with short names like rt | 20 |
| ShortVariable | Avoid variables with short names like s | 53 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 19–146 |
| RedundantFieldInitializer | Avoid using redundant field initializer for 'instance' | 22 |
| AvoidFieldNameMatchingMethodName | Field instance has the same name as a method | 22 |
| CommentRequired | Public method and constructor comments are required | 29–35 |
| NonThreadSafeSingleton | Singleton is not thread safe | 30–33 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 101 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 118–137 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 121–132 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 141 |
| Rule | Violation | Line |
|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 1–695 |
| TooManyFields | Too many fields | 58–695 |
| GodClass | Possible God Class (WMC=86, ATFD=121, TCC=23.684%) | 58–695 |
| TooManyMethods | This class has too many methods, consider refactoring it. | 58–695 |
| CyclomaticComplexity | The class 'AbstractIdentityDuplicateJspBean' has a total cyclomatic complexity of 86 (highest 11). | 58–695 |
| CommentRequired | Header comments are required | 58–695 |
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 58–695 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 77–79 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the PARAMETERS_DUPLICATE_SEARCH field if you want a default access modifier | 77–79 |
| FinalFieldCouldBeStatic | This final field could be made static | 80 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 80 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the PARAMETER_PAGE field if you want a default access modifier | 80 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the PARAMETER_CLIENT_CODE field if you want a default access modifier | 81 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 81 |
| FinalFieldCouldBeStatic | This final field could be made static | 81 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '_totalRecords' | 96 |
| CyclomaticComplexity | The method 'init(HttpServletRequest, boolean)' has a cyclomatic complexity of 11. | 109–179 |
| NPathComplexity | The method 'init(HttpServletRequest, boolean)' has an NPath complexity of 512 | 109–179 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 120 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 129 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 131 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 133 |
| CommentRequired | Protected method constructor comments are required | 181–196 |
| LinguisticNaming | Linguistics Antipattern - The getter 'getSuspiciousSummary' should not return void linguistically | 181–196 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 221 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 237–244 |
| CompareObjectsWithEquals | Use equals() to compare object references. | 413 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 430 |
| CommentRequired | Protected method constructor comments are required | 468–498 |
| AvoidDuplicateLiterals | The String literal "override-" appears 6 times in this file; the first occurrence is on line 519 | 519 |
| CommentRequired | Protected method constructor comments are required | 543–552 |
| CommentRequired | Protected method constructor comments are required | 554–563 |
| ControlStatementBraces | This statement should have braces | 594 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 629 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 637 |
| Rule | Violation | Line |
|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 34–665 |
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 78–665 |
| FinalFieldCouldBeStatic | This final field could be made static | 123 |
| FinalFieldCouldBeStatic | This final field could be made static | 124 |
| FinalFieldCouldBeStatic | This final field could be made static | 125 |
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 153 |
| AvoidDuplicateLiterals | The String literal "You don't have the right to read duplicates" appears 4 times in this file; the first occurrence is on line 157 | 157 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 159 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 182 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 185 |
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 211 |
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 258 |
| CyclomaticComplexity | The method 'getSelectIdentities(HttpServletRequest)' has a cyclomatic complexity of 11. | 259–339 |
| NPathComplexity | The method 'getSelectIdentities(HttpServletRequest)' has an NPath complexity of 216 | 259–339 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 264 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 321 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 324 |
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 348 |
| NPathComplexity | The method 'getResolveDuplicates(HttpServletRequest)' has an NPath complexity of 1080 | 349–424 |
| CyclomaticComplexity | The method 'getResolveDuplicates(HttpServletRequest)' has a cyclomatic complexity of 15. | 349–424 |
| AvoidDuplicateLiterals | The String literal "You don't have the right to write duplicates" appears 4 times in this file; the first occurrence is on line 351 | 351 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 353 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 462 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 463 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 484 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 485 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 493 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 494 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 508 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 509 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 587 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 588 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 611 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 617 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 5–37 |
| FieldNamingConventions | The field name 'canNotify' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 6 |
| CommentRequired | Public method and constructor comments are required | 3 | 18–36 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| DataClass | The class 'MediationIdentity' is suspected to be a Data Class (WOC=0.000%, NOPA=0, NOAM=6, WMC=6) | 3 | 39–74 |
| CommentRequired | Header comments are required | 3 | 39–74 |
| FieldNamingConventions | The field name 'suspiciousIdentity' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 41 |
| FieldNamingConventions | The field name 'bestIdentity' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 42 |
| FieldNamingConventions | The field name 'duplicatesToMergeAttributes' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 43 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 49–163 |
| FieldNamingConventions | The final field name '_referentialService' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 55 |
| FieldNamingConventions | The final field name '_currentClientCode' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 56 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 58–62 |
| CommentRequired | Public method and constructor comments are required | 3 | 58–62 |
| CommentRequired | Public method and constructor comments are required | 3 | 64–75 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 71 |
| GuardLogStatement | Logger calls should be surrounded by log level guards. | 2 | 84 |
| CommentRequired | Public method and constructor comments are required | 3 | 87–102 |
| GuardLogStatement | Logger calls should be surrounded by log level guards. | 2 | 111 |
| CommentRequired | Public method and constructor comments are required | 3 | 114–129 |
| GuardLogStatement | Logger calls should be surrounded by log level guards. | 2 | 138 |
| CommentRequired | Public method and constructor comments are required | 3 | 141–156 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 51–166 |
| FieldNamingConventions | The final field name '_serviceContractService' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 56 |
| FieldNamingConventions | The final field name '_sortedAttributeKeyList' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 57 |
| FieldNamingConventions | The final field name '_attributeKeyToShowList' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 58 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 60–64 |
| CommentRequired | Public method and constructor comments are required | 3 | 60–64 |
| GuardLogStatement | Logger calls should be surrounded by log level guards. | 2 | 73 |
| CommentRequired | Public method and constructor comments are required | 3 | 76–85 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 115 |
| ShortVariable | Avoid variables with short names like i1 | 3 | 135 |
| ShortVariable | Avoid variables with short names like i2 | 3 | 136 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 5–23 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 11–58 |
| ShortVariable | Avoid variables with short names like rt | 3 | 20 |
| ShortVariable | Avoid variables with short names like s | 3 | 53 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ClassWithOnlyPrivateConstructorsShouldBeFinal | A class which only has private constructors should be final | 1 | 19–146 |
| CommentRequired | Header comments are required | 3 | 19–146 |
| FieldNamingConventions | The final field name '_referentialCache' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 20 |
| FieldNamingConventions | The final field name '_minCertificationCode' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 21 |
| RedundantFieldInitializer | Avoid using redundant field initializer for 'instance' | 3 | 22 |
| FieldNamingConventions | The static field name 'instance' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 22 |
| AvoidFieldNameMatchingMethodName | Field instance has the same name as a method | 3 | 22 |
| CommentRequired | Public method and constructor comments are required | 3 | 29–35 |
| NonThreadSafeSingleton | Singleton is not thread safe | 3 | 30–33 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 101 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 118–137 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 121–132 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 141 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 3 | 1–695 |
| TooManyFields | Too many fields | 3 | 58–695 |
| GodClass | Possible God Class (WMC=86, ATFD=121, TCC=23.684%) | 3 | 58–695 |
| TooManyMethods | This class has too many methods, consider refactoring it. | 3 | 58–695 |
| CyclomaticComplexity | The class 'AbstractIdentityDuplicateJspBean' has a total cyclomatic complexity of 86 (highest 11). | 3 | 58–695 |
| CommentRequired | Header comments are required | 3 | 58–695 |
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 3 | 58–695 |
| FieldNamingConventions | The constant name '_serviceQuality' doesn't match '[A-Z][A-Z_0-9]*' | 1 | 66 |
| FieldNamingConventions | The constant name '_serviceIdentity' doesn't match '[A-Z][A-Z_0-9]*' | 1 | 67 |
| FieldNamingConventions | The constant name '_mediationService' doesn't match '[A-Z][A-Z_0-9]*' | 1 | 68 |
| FieldNamingConventions | The constant name '_serviceContractCache' doesn't match '[A-Z][A-Z_0-9]*' | 1 | 69 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 77–79 |
| FieldNamingConventions | The final field name 'PARAMETERS_DUPLICATE_SEARCH' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 77 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the PARAMETERS_DUPLICATE_SEARCH field if you want a default access modifier | 3 | 77–79 |
| FinalFieldCouldBeStatic | This final field could be made static | 3 | 80 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 80 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the PARAMETER_PAGE field if you want a default access modifier | 3 | 80 |
| FieldNamingConventions | The final field name 'PARAMETER_PAGE' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 80 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the PARAMETER_CLIENT_CODE field if you want a default access modifier | 3 | 81 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 81 |
| FinalFieldCouldBeStatic | This final field could be made static | 3 | 81 |
| FieldNamingConventions | The final field name 'PARAMETER_CLIENT_CODE' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 81 |
| FieldNamingConventions | The final field name '_rulePriorityMin' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 85 |
| FieldNamingConventions | The final field name '_duplicateRules' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 91 |
| FieldNamingConventions | The final field name '_mediationIdentities' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 92 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '_totalRecords' | 3 | 96 |
| CyclomaticComplexity | The method 'init(HttpServletRequest, boolean)' has a cyclomatic complexity of 11. | 3 | 109–179 |
| NPathComplexity | The method 'init(HttpServletRequest, boolean)' has an NPath complexity of 512 | 3 | 109–179 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 120 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 129 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 131 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 133 |
| CommentRequired | Protected method constructor comments are required | 3 | 181–196 |
| LinguisticNaming | Linguistics Antipattern - The getter 'getSuspiciousSummary' should not return void linguistically | 3 | 181–196 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 221 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 237–244 |
| CompareObjectsWithEquals | Use equals() to compare object references. | 3 | 413 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 430 |
| CommentRequired | Protected method constructor comments are required | 3 | 468–498 |
| AvoidDuplicateLiterals | The String literal "override-" appears 6 times in this file; the first occurrence is on line 519 | 3 | 519 |
| CommentRequired | Protected method constructor comments are required | 3 | 543–552 |
| CommentRequired | Protected method constructor comments are required | 3 | 554–563 |
| ControlStatementBraces | This statement should have braces | 3 | 594 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 629 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 637 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 3 | 34–665 |
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 3 | 78–665 |
| FieldNamingConventions | The final field name 'PARAMETER_CUID_PINNED' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 123 |
| FinalFieldCouldBeStatic | This final field could be made static | 3 | 123 |
| FieldNamingConventions | The final field name 'PARAMETER_CUID_TO_EXCLUDE' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 124 |
| FinalFieldCouldBeStatic | This final field could be made static | 3 | 124 |
| FieldNamingConventions | The final field name 'PARAMETER_CUID' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 125 |
| FinalFieldCouldBeStatic | This final field could be made static | 3 | 125 |
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 3 | 153 |
| AvoidDuplicateLiterals | The String literal "You don't have the right to read duplicates" appears 4 times in this file; the first occurrence is on line 157 | 3 | 157 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 159 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 182 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 185 |
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 3 | 211 |
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 3 | 258 |
| CyclomaticComplexity | The method 'getSelectIdentities(HttpServletRequest)' has a cyclomatic complexity of 11. | 3 | 259–339 |
| NPathComplexity | The method 'getSelectIdentities(HttpServletRequest)' has an NPath complexity of 216 | 3 | 259–339 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 264 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 321 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 324 |
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 3 | 348 |
| NPathComplexity | The method 'getResolveDuplicates(HttpServletRequest)' has an NPath complexity of 1080 | 3 | 349–424 |
| CyclomaticComplexity | The method 'getResolveDuplicates(HttpServletRequest)' has a cyclomatic complexity of 15. | 3 | 349–424 |
| AvoidDuplicateLiterals | The String literal "You don't have the right to write duplicates" appears 4 times in this file; the first occurrence is on line 351 | 3 | 351 |
| LocalVariableNamingConventions | The final local variable name '_suspiciousCuid' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 353 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 353 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 462 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 463 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 484 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 485 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 493 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 494 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 508 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 509 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 587 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 588 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 611 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 617 |
