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–198 |
| Rule | Violation | Line |
|---|---|---|
| ClassWithOnlyPrivateConstructorsShouldBeFinal | A class which only has private constructors should be final | 84–976 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Public method and constructor comments are required | 53–56 |
| UncommentedEmptyConstructor | Document empty constructor | 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 |
| CyclomaticComplexity | The method 'checkDashboardIdentityFields(DashboardIdentity, HttpServletRequest, boolean, AttributeCategory)' has a cyclomatic complexity of 66. | 228–365 |
| NPathComplexity | The method 'checkDashboardIdentityFields(DashboardIdentity, HttpServletRequest, boolean, AttributeCategory)' has an NPath complexity of 2147483647 | 228–365 |
| ExcessiveMethodLength | Avoid really long methods. | 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 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 373 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 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–976 |
| GodClass | Possible God Class (WMC=132, ATFD=111, TCC=1.991%) | 84–976 |
| CyclomaticComplexity | The class 'DashboardIdentityUtils' has a total cyclomatic complexity of 132 (highest 15). | 84–976 |
| TooManyMethods | This class has too many methods, consider refactoring it. | 85–976 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 104 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 105 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 106 |
| CommentRequired | Public method and constructor comments are required | 148–178 |
| NonThreadSafeSingleton | Singleton is not thread safe | 150–176 |
| ShortVariable | Avoid variables with short names like i | 156 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 225 |
| CommentRequired | Public method and constructor comments are required | 245–267 |
| CyclomaticComplexity | The method 'convertToIdentityDto(DashboardIdentity, boolean, AttributeCategory)' has a cyclomatic complexity of 10. | 297–329 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 302 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 303 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 388–406 |
| UseCollectionIsEmpty | Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() | 415 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 512 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 513 |
| AvoidUncheckedExceptionsInSignatures | A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause | 577 |
| CyclomaticComplexity | The method 'updateIdentity(IdentityDto)' has a cyclomatic complexity of 14. | 577–609 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 583–603 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 588 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 598 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 668–675 |
| CyclomaticComplexity | The method 'updateDashboardIdentityInSession(DashboardIdentity, DashboardIdentity)' has a cyclomatic complexity of 15. | 855–893 |
| NPathComplexity | The method 'updateDashboardIdentityInSession(DashboardIdentity, DashboardIdentity)' has an NPath complexity of 2187 | 855–893 |
| ShortVariable | Avoid variables with short names like o | 942 |
| 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 |
|---|---|---|
| ImmutableField | Private field '_securityTokenService' could be made final; it is only initialized in the declaration or constructor. | 114 |
| ImmutableField | Private field '_dashboardPropertiesGroup' could be made final; it is only initialized in the declaration or constructor. | 115 |
| ImmutableField | Private field '_lstGenderList' could be made final; it is only initialized in the declaration or constructor. | 116 |
| ShortVariable | Avoid variables with short names like i | 127 |
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 206 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 287 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 298 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 345 |
| 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–964 |
| TooManyMethods | This class has too many methods, consider refactoring it. | 80–964 |
| UnusedPrivateField | Avoid unused private fields such as 'PROPERTY_FC_CERTIFIER_CODE'. | 148 |
| ImmutableField | Private field '_lstContactModeList' could be made final; it is only initialized in the declaration or constructor. | 153 |
| ImmutableField | Private field '_lstGenderList' could be made final; it is only initialized in the declaration or constructor. | 154 |
| SingularField | Perhaps '_identityService' could be replaced by a local variable. | 163 |
| ImmutableField | Private field '_identityService' could be made final; it is only initialized in the declaration or constructor. | 163 |
| UnusedPrivateField | Avoid unused private fields such as '_identityService'. | 163 |
| UnusedPrivateField | Avoid unused private fields such as '_bReInitAppCode'. | 164 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '_bReInitAppCode' | 164 |
| ImmutableField | Private field '_securityTokenService' could be made final; it is only initialized in the declaration or constructor. | 166 |
| ShortVariable | Avoid variables with short names like i | 178 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 243 |
| CyclomaticComplexity | The method 'getModifyIdentity(HttpServletRequest)' has a cyclomatic complexity of 10. | 265–312 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 302 |
| ExcessiveMethodLength | Avoid really long methods. | 341–442 |
| NPathComplexity | The method 'getCheckIdentity(HttpServletRequest)' has an NPath complexity of 2976 | 341–442 |
| CyclomaticComplexity | The method 'getCheckIdentity(HttpServletRequest)' has a cyclomatic complexity of 19. | 341–442 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 403–413 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 405–408 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 498 |
| 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. | 510 |
| CyclomaticComplexity | The method 'doCheckIdentity(HttpServletRequest)' has a cyclomatic complexity of 10. | 529–587 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 574 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 581 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 582 |
| CommentRequired | Public method and constructor comments are required | 898–924 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 900 |
| CommentRequired | Public method and constructor comments are required | 927–962 |
| 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' | 69 |
| 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 |
|---|---|---|---|
| CommentRequired | Public method and constructor comments are required | 3 | 53–56 |
| UncommentedEmptyConstructor | Document empty constructor | 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 |
| CyclomaticComplexity | The method 'checkDashboardIdentityFields(DashboardIdentity, HttpServletRequest, boolean, AttributeCategory)' has a cyclomatic complexity of 66. | 3 | 228–365 |
| NPathComplexity | The method 'checkDashboardIdentityFields(DashboardIdentity, HttpServletRequest, boolean, AttributeCategory)' has an NPath complexity of 2147483647 | 3 | 228–365 |
| ExcessiveMethodLength | Avoid really long methods. | 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 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 373 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 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–198 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 3 | 34–976 |
| UnusedImports | Avoid unused imports such as 'fr.paris.lutece.portal.service.security.UserNotSignedException' | 4 | 69 |
| GodClass | Possible God Class (WMC=132, ATFD=111, TCC=1.991%) | 3 | 84–976 |
| ClassWithOnlyPrivateConstructorsShouldBeFinal | A class which only has private constructors should be final | 1 | 84–976 |
| CyclomaticComplexity | The class 'DashboardIdentityUtils' has a total cyclomatic complexity of 132 (highest 15). | 3 | 84–976 |
| TooManyMethods | This class has too many methods, consider refactoring it. | 3 | 85–976 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 104 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 105 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 106 |
| CommentRequired | Public method and constructor comments are required | 3 | 148–178 |
| NonThreadSafeSingleton | Singleton is not thread safe | 3 | 150–176 |
| ShortVariable | Avoid variables with short names like i | 3 | 156 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 225 |
| CommentRequired | Public method and constructor comments are required | 3 | 245–267 |
| CyclomaticComplexity | The method 'convertToIdentityDto(DashboardIdentity, boolean, AttributeCategory)' has a cyclomatic complexity of 10. | 3 | 297–329 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 302 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 303 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 388–406 |
| UseCollectionIsEmpty | Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() | 3 | 415 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 512 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 513 |
| AvoidUncheckedExceptionsInSignatures | A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause | 3 | 577 |
| CyclomaticComplexity | The method 'updateIdentity(IdentityDto)' has a cyclomatic complexity of 14. | 3 | 577–609 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 583–603 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 588 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 598 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 668–675 |
| CyclomaticComplexity | The method 'updateDashboardIdentityInSession(DashboardIdentity, DashboardIdentity)' has a cyclomatic complexity of 15. | 3 | 855–893 |
| NPathComplexity | The method 'updateDashboardIdentityInSession(DashboardIdentity, DashboardIdentity)' has an NPath complexity of 2187 | 3 | 855–893 |
| ShortVariable | Avoid variables with short names like o | 3 | 942 |
| 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 |
|---|---|---|---|
| ImmutableField | Private field '_securityTokenService' could be made final; it is only initialized in the declaration or constructor. | 3 | 114 |
| ImmutableField | Private field '_dashboardPropertiesGroup' could be made final; it is only initialized in the declaration or constructor. | 3 | 115 |
| ImmutableField | Private field '_lstGenderList' could be made final; it is only initialized in the declaration or constructor. | 3 | 116 |
| ShortVariable | Avoid variables with short names like i | 3 | 127 |
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 3 | 206 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 287 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 298 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 345 |
| 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–964 |
| TooManyMethods | This class has too many methods, consider refactoring it. | 3 | 80–964 |
| UnusedPrivateField | Avoid unused private fields such as 'PROPERTY_FC_CERTIFIER_CODE'. | 3 | 148 |
| ImmutableField | Private field '_lstContactModeList' could be made final; it is only initialized in the declaration or constructor. | 3 | 153 |
| ImmutableField | Private field '_lstGenderList' could be made final; it is only initialized in the declaration or constructor. | 3 | 154 |
| SingularField | Perhaps '_identityService' could be replaced by a local variable. | 3 | 163 |
| ImmutableField | Private field '_identityService' could be made final; it is only initialized in the declaration or constructor. | 3 | 163 |
| UnusedPrivateField | Avoid unused private fields such as '_identityService'. | 3 | 163 |
| UnusedPrivateField | Avoid unused private fields such as '_bReInitAppCode'. | 3 | 164 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '_bReInitAppCode' | 3 | 164 |
| ImmutableField | Private field '_securityTokenService' could be made final; it is only initialized in the declaration or constructor. | 3 | 166 |
| ShortVariable | Avoid variables with short names like i | 3 | 178 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 243 |
| CyclomaticComplexity | The method 'getModifyIdentity(HttpServletRequest)' has a cyclomatic complexity of 10. | 3 | 265–312 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 302 |
| ExcessiveMethodLength | Avoid really long methods. | 3 | 341–442 |
| NPathComplexity | The method 'getCheckIdentity(HttpServletRequest)' has an NPath complexity of 2976 | 3 | 341–442 |
| CyclomaticComplexity | The method 'getCheckIdentity(HttpServletRequest)' has a cyclomatic complexity of 19. | 3 | 341–442 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 403–413 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 3 | 405–408 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 498 |
| 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 | 510 |
| CyclomaticComplexity | The method 'doCheckIdentity(HttpServletRequest)' has a cyclomatic complexity of 10. | 3 | 529–587 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 574 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 581 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 582 |
| CommentRequired | Public method and constructor comments are required | 3 | 898–924 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 900 |
| CommentRequired | Public method and constructor comments are required | 3 | 927–962 |
| 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 |
