The following document contains the results of PMD 6.13.0.
| Rule | Violation | Line |
|---|---|---|
| FormalParameterNamingConventions | The method parameter name '_bMandatory' doesn't match '[a-z][a-zA-Z0-9]*' | 198 |
| Rule | Violation | Line |
|---|---|---|
| FormalParameterNamingConventions | The method parameter name '_lastUpdateDate' doesn't match '[a-z][a-zA-Z0-9]*' | 474 |
| Rule | Violation | Line |
|---|---|---|
| ClassWithOnlyPrivateConstructorsShouldBeFinal | A class which only has private constructors should be final | 73–643 |
| Rule | Violation | Line |
|---|---|---|
| ClassNamingConventions | The utility class name 'Constants' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' | 43–202 |
| Rule | Violation | Line |
|---|---|---|
| ClassWithOnlyPrivateConstructorsShouldBeFinal | A class which only has private constructors should be final | 82–1005 |
| Rule | Violation | Line |
|---|---|---|
| UncommentedEmptyConstructor | Document empty constructor | 53–56 |
| CommentRequired | Public method and constructor comments are required | 53–56 |
| CommentRequired | Public method and constructor comments are required | 58–67 |
| CommentRequired | Public method and constructor comments are required | 69–76 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 73 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 74 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 75 |
| Rule | Violation | Line |
|---|---|---|
| ExcessivePublicCount | This class has a bunch of public methods and attributes | 34–528 |
| ImmutableField | Private field '_mapAttributes' could be made final; it is only initialized in the declaration or constructor. | 49 |
| CommentRequired | Public method and constructor comments are required | 53–56 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 55 |
| Rule | Violation | Line |
|---|---|---|
| NonThreadSafeSingleton | Singleton is not thread safe | 113–140 |
| ShortVariable | Avoid variables with short names like i | 120 |
| AvoidUncheckedExceptionsInSignatures | A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause | 154 |
| AvoidUncheckedExceptionsInSignatures | A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause | 165 |
| ExcessiveMethodLength | Avoid really long methods. | 228–365 |
| NPathComplexity | The method 'checkDashboardIdentityFields(DashboardIdentity, HttpServletRequest, boolean, AttributeCategory)' has an NPath complexity of 2147483647 | 228–365 |
| CyclomaticComplexity | The method 'checkDashboardIdentityFields(DashboardIdentity, HttpServletRequest, boolean, AttributeCategory)' has a cyclomatic complexity of 66. | 228–365 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 231 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 231 |
| CyclomaticComplexity | The method 'checkDashboardIdentityFieldsFromServiceContract(DashboardIdentity, HttpServletRequest, boolean, String)' has a cyclomatic complexity of 13. | 371–408 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 373 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 373 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 388–393 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 398–401 |
| AvoidUncheckedExceptionsInSignatures | A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause | 414 |
| AvoidUncheckedExceptionsInSignatures | A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause | 424 |
| CyclomaticComplexity | The method 'getErrorValidation(HttpServletRequest, String, String, String, boolean, AttributeCategory)' has a cyclomatic complexity of 14. | 492–533 |
| CyclomaticComplexity | The method 'getErrorValidation(DashboardAttribute, String, String)' has a cyclomatic complexity of 13. | 543–577 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 609 |
| Rule | Violation | Line |
|---|---|---|
| AvoidUncheckedExceptionsInSignatures | A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause | 61 |
| AvoidUncheckedExceptionsInSignatures | A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause | 71 |
| AvoidUncheckedExceptionsInSignatures | A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause | 108 |
| AvoidUncheckedExceptionsInSignatures | A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause | 119 |
| AvoidUncheckedExceptionsInSignatures | A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause | 167 |
| Rule | Violation | Line |
|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 34–1005 |
| CyclomaticComplexity | The class 'DashboardIdentityUtils' has a total cyclomatic complexity of 141 (highest 15). | 82–1005 |
| GodClass | Possible God Class (WMC=141, ATFD=123, TCC=2.429%) | 82–1005 |
| TooManyMethods | This class has too many methods, consider refactoring it. | 83–1005 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 102 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 103 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 104 |
| CommentRequired | Public method and constructor comments are required | 146–176 |
| NonThreadSafeSingleton | Singleton is not thread safe | 148–174 |
| ShortVariable | Avoid variables with short names like i | 154 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 223 |
| CommentRequired | Public method and constructor comments are required | 243–269 |
| CyclomaticComplexity | The method 'needCertification(DashboardIdentity, ServiceContractSearchResponse, List, int, int)' has a cyclomatic complexity of 12. | 243–269 |
| CyclomaticComplexity | The method 'convertToIdentityDto(DashboardIdentity, boolean, AttributeCategory)' has a cyclomatic complexity of 10. | 299–332 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 304 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 305 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 391–409 |
| UseCollectionIsEmpty | Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() | 418 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 515 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 516 |
| AvoidUncheckedExceptionsInSignatures | A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause | 579 |
| CyclomaticComplexity | The method 'updateIdentity(IdentityDto)' has a cyclomatic complexity of 14. | 579–611 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 585–605 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 590 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 600 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 670–677 |
| NPathComplexity | The method 'updateDashboardIdentityInSession(DashboardIdentity, DashboardIdentity)' has an NPath complexity of 2187 | 857–895 |
| CyclomaticComplexity | The method 'updateDashboardIdentityInSession(DashboardIdentity, DashboardIdentity)' has a cyclomatic complexity of 15. | 857–895 |
| ShortVariable | Avoid variables with short names like o | 971 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 55–88 |
| SingularField | Perhaps '_dashboardIdentity' could be replaced by a local variable. | 66 |
| CommentRequired | Public method and constructor comments are required | 68–87 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 77 |
| Rule | Violation | Line |
|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 34–370 |
| ImmutableField | Private field '_securityTokenService' could be made final; it is only initialized in the declaration or constructor. | 115 |
| ImmutableField | Private field '_dashboardPropertiesGroup' could be made final; it is only initialized in the declaration or constructor. | 116 |
| ImmutableField | Private field '_lstGenderList' could be made final; it is only initialized in the declaration or constructor. | 117 |
| ShortVariable | Avoid variables with short names like i | 128 |
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 204 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 285 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 296 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 343 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 364 |
| Rule | Violation | Line |
|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 34–978 |
| TooManyMethods | This class has too many methods, consider refactoring it. | 80–978 |
| UnusedPrivateField | Avoid unused private fields such as 'PROPERTY_FC_CERTIFIER_CODE'. | 150 |
| ImmutableField | Private field '_lstContactModeList' could be made final; it is only initialized in the declaration or constructor. | 158 |
| ImmutableField | Private field '_lstGenderList' could be made final; it is only initialized in the declaration or constructor. | 159 |
| SingularField | Perhaps '_identityService' could be replaced by a local variable. | 168 |
| ImmutableField | Private field '_identityService' could be made final; it is only initialized in the declaration or constructor. | 168 |
| UnusedPrivateField | Avoid unused private fields such as '_identityService'. | 168 |
| UnusedPrivateField | Avoid unused private fields such as '_bReInitAppCode'. | 169 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '_bReInitAppCode' | 169 |
| ImmutableField | Private field '_securityTokenService' could be made final; it is only initialized in the declaration or constructor. | 171 |
| ShortVariable | Avoid variables with short names like i | 183 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 248 |
| CyclomaticComplexity | The method 'getModifyIdentity(HttpServletRequest)' has a cyclomatic complexity of 10. | 270–317 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 307 |
| ExcessiveMethodLength | Avoid really long methods. | 346–449 |
| NPathComplexity | The method 'getCheckIdentity(HttpServletRequest)' has an NPath complexity of 2976 | 346–449 |
| CyclomaticComplexity | The method 'getCheckIdentity(HttpServletRequest)' has a cyclomatic complexity of 19. | 346–449 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 409–419 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 411–414 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 512 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 523 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 524 |
| CyclomaticComplexity | The method 'doCheckIdentity(HttpServletRequest)' has a cyclomatic complexity of 11. | 543–601 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 588 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 595 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 596 |
| CommentRequired | Public method and constructor comments are required | 912–938 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 914 |
| CommentRequired | Public method and constructor comments are required | 941–976 |
| Rule | Violation | Line |
|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 58–129 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 81 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 81 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 42–71 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 42–71 |
| Rule | Violation | Line |
|---|---|---|
| UnusedImports | Avoid unused imports such as 'fr.paris.lutece.portal.service.security.UserNotSignedException' | 49 |
| UnusedImports | Avoid unused imports such as 'fr.paris.lutece.portal.service.util.AppLogService' | 51 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UncommentedEmptyConstructor | Document empty constructor | 3 | 53–56 |
| CommentRequired | Public method and constructor comments are required | 3 | 53–56 |
| CommentRequired | Public method and constructor comments are required | 3 | 58–67 |
| CommentRequired | Public method and constructor comments are required | 3 | 69–76 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 73 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 74 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 75 |
| FormalParameterNamingConventions | The method parameter name '_bMandatory' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 198 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ExcessivePublicCount | This class has a bunch of public methods and attributes | 3 | 34–528 |
| ImmutableField | Private field '_mapAttributes' could be made final; it is only initialized in the declaration or constructor. | 3 | 49 |
| CommentRequired | Public method and constructor comments are required | 3 | 53–56 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 55 |
| FormalParameterNamingConventions | The method parameter name '_lastUpdateDate' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 474 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ClassWithOnlyPrivateConstructorsShouldBeFinal | A class which only has private constructors should be final | 1 | 73–643 |
| NonThreadSafeSingleton | Singleton is not thread safe | 3 | 113–140 |
| ShortVariable | Avoid variables with short names like i | 3 | 120 |
| AvoidUncheckedExceptionsInSignatures | A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause | 3 | 154 |
| AvoidUncheckedExceptionsInSignatures | A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause | 3 | 165 |
| ExcessiveMethodLength | Avoid really long methods. | 3 | 228–365 |
| NPathComplexity | The method 'checkDashboardIdentityFields(DashboardIdentity, HttpServletRequest, boolean, AttributeCategory)' has an NPath complexity of 2147483647 | 3 | 228–365 |
| CyclomaticComplexity | The method 'checkDashboardIdentityFields(DashboardIdentity, HttpServletRequest, boolean, AttributeCategory)' has a cyclomatic complexity of 66. | 3 | 228–365 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 231 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 231 |
| CyclomaticComplexity | The method 'checkDashboardIdentityFieldsFromServiceContract(DashboardIdentity, HttpServletRequest, boolean, String)' has a cyclomatic complexity of 13. | 3 | 371–408 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 373 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 373 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 3 | 388–393 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 3 | 398–401 |
| AvoidUncheckedExceptionsInSignatures | A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause | 3 | 414 |
| AvoidUncheckedExceptionsInSignatures | A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause | 3 | 424 |
| CyclomaticComplexity | The method 'getErrorValidation(HttpServletRequest, String, String, String, boolean, AttributeCategory)' has a cyclomatic complexity of 14. | 3 | 492–533 |
| CyclomaticComplexity | The method 'getErrorValidation(DashboardAttribute, String, String)' has a cyclomatic complexity of 13. | 3 | 543–577 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 609 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AvoidUncheckedExceptionsInSignatures | A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause | 3 | 61 |
| AvoidUncheckedExceptionsInSignatures | A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause | 3 | 71 |
| AvoidUncheckedExceptionsInSignatures | A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause | 3 | 108 |
| AvoidUncheckedExceptionsInSignatures | A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause | 3 | 119 |
| AvoidUncheckedExceptionsInSignatures | A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause | 3 | 167 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ClassNamingConventions | The utility class name 'Constants' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' | 1 | 43–202 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 3 | 34–1005 |
| ClassWithOnlyPrivateConstructorsShouldBeFinal | A class which only has private constructors should be final | 1 | 82–1005 |
| CyclomaticComplexity | The class 'DashboardIdentityUtils' has a total cyclomatic complexity of 141 (highest 15). | 3 | 82–1005 |
| GodClass | Possible God Class (WMC=141, ATFD=123, TCC=2.429%) | 3 | 82–1005 |
| TooManyMethods | This class has too many methods, consider refactoring it. | 3 | 83–1005 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 102 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 103 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 104 |
| CommentRequired | Public method and constructor comments are required | 3 | 146–176 |
| NonThreadSafeSingleton | Singleton is not thread safe | 3 | 148–174 |
| ShortVariable | Avoid variables with short names like i | 3 | 154 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 223 |
| CommentRequired | Public method and constructor comments are required | 3 | 243–269 |
| CyclomaticComplexity | The method 'needCertification(DashboardIdentity, ServiceContractSearchResponse, List, int, int)' has a cyclomatic complexity of 12. | 3 | 243–269 |
| CyclomaticComplexity | The method 'convertToIdentityDto(DashboardIdentity, boolean, AttributeCategory)' has a cyclomatic complexity of 10. | 3 | 299–332 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 304 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 305 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 391–409 |
| UseCollectionIsEmpty | Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() | 3 | 418 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 515 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 516 |
| AvoidUncheckedExceptionsInSignatures | A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause | 3 | 579 |
| CyclomaticComplexity | The method 'updateIdentity(IdentityDto)' has a cyclomatic complexity of 14. | 3 | 579–611 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 585–605 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 590 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 600 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 670–677 |
| NPathComplexity | The method 'updateDashboardIdentityInSession(DashboardIdentity, DashboardIdentity)' has an NPath complexity of 2187 | 3 | 857–895 |
| CyclomaticComplexity | The method 'updateDashboardIdentityInSession(DashboardIdentity, DashboardIdentity)' has a cyclomatic complexity of 15. | 3 | 857–895 |
| ShortVariable | Avoid variables with short names like o | 3 | 971 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedImports | Avoid unused imports such as 'fr.paris.lutece.portal.service.security.UserNotSignedException' | 4 | 49 |
| UnusedImports | Avoid unused imports such as 'fr.paris.lutece.portal.service.util.AppLogService' | 4 | 51 |
| CommentRequired | Header comments are required | 3 | 55–88 |
| SingularField | Perhaps '_dashboardIdentity' could be replaced by a local variable. | 3 | 66 |
| CommentRequired | Public method and constructor comments are required | 3 | 68–87 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 77 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 3 | 34–370 |
| ImmutableField | Private field '_securityTokenService' could be made final; it is only initialized in the declaration or constructor. | 3 | 115 |
| ImmutableField | Private field '_dashboardPropertiesGroup' could be made final; it is only initialized in the declaration or constructor. | 3 | 116 |
| ImmutableField | Private field '_lstGenderList' could be made final; it is only initialized in the declaration or constructor. | 3 | 117 |
| ShortVariable | Avoid variables with short names like i | 3 | 128 |
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 3 | 204 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 285 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 296 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 343 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 364 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 3 | 34–978 |
| TooManyMethods | This class has too many methods, consider refactoring it. | 3 | 80–978 |
| UnusedPrivateField | Avoid unused private fields such as 'PROPERTY_FC_CERTIFIER_CODE'. | 3 | 150 |
| ImmutableField | Private field '_lstContactModeList' could be made final; it is only initialized in the declaration or constructor. | 3 | 158 |
| ImmutableField | Private field '_lstGenderList' could be made final; it is only initialized in the declaration or constructor. | 3 | 159 |
| SingularField | Perhaps '_identityService' could be replaced by a local variable. | 3 | 168 |
| ImmutableField | Private field '_identityService' could be made final; it is only initialized in the declaration or constructor. | 3 | 168 |
| UnusedPrivateField | Avoid unused private fields such as '_identityService'. | 3 | 168 |
| UnusedPrivateField | Avoid unused private fields such as '_bReInitAppCode'. | 3 | 169 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '_bReInitAppCode' | 3 | 169 |
| ImmutableField | Private field '_securityTokenService' could be made final; it is only initialized in the declaration or constructor. | 3 | 171 |
| ShortVariable | Avoid variables with short names like i | 3 | 183 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 248 |
| CyclomaticComplexity | The method 'getModifyIdentity(HttpServletRequest)' has a cyclomatic complexity of 10. | 3 | 270–317 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 307 |
| ExcessiveMethodLength | Avoid really long methods. | 3 | 346–449 |
| NPathComplexity | The method 'getCheckIdentity(HttpServletRequest)' has an NPath complexity of 2976 | 3 | 346–449 |
| CyclomaticComplexity | The method 'getCheckIdentity(HttpServletRequest)' has a cyclomatic complexity of 19. | 3 | 346–449 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 409–419 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 3 | 411–414 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 512 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 523 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 524 |
| CyclomaticComplexity | The method 'doCheckIdentity(HttpServletRequest)' has a cyclomatic complexity of 11. | 3 | 543–601 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 588 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 595 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 596 |
| CommentRequired | Public method and constructor comments are required | 3 | 912–938 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 914 |
| CommentRequired | Public method and constructor comments are required | 3 | 941–976 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 3 | 58–129 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 81 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 81 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 42–71 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 42–71 |
