Violation |
Ligne |
A high number of imports can indicate a high degree of coupling within an object. |
34–884 |
Possible God class (WMC=73, ATFD=159, TCC=0.018461538461538463) |
34–884 |
The class 'IdentityXPage' has a Standard Cyclomatic Complexity of 3 (Highest = 20). |
77–884 |
The class 'IdentityXPage' has a Modified Cyclomatic Complexity of 3 (Highest = 20). |
77–884 |
The class 'IdentityXPage' has a Cyclomatic Complexity of 3 (Highest = 21). |
77–884 |
Private field '_lstContactModeList' could be made final; it is only initialized in the declaration or constructor. |
116 |
Private field '_lstGenderList' could be made final; it is only initialized in the declaration or constructor. |
117 |
Private field '_identityService' could be made final; it is only initialized in the declaration or constructor. |
121 |
Avoid variables with short names like i |
133 |
Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block |
256 |
Assigning an Object to null is a code smell. Consider refactoring. |
279 |
The method 'checkDashboardIdentityFields' has a Modified Cyclomatic Complexity of 20. |
371–515 |
Avoid really long methods. |
371–515 |
The method 'checkDashboardIdentityFields' has a Cyclomatic Complexity of 21. |
371–515 |
The method 'checkDashboardIdentityFields' has a Standard Cyclomatic Complexity of 20. |
371–515 |
The method checkDashboardIdentityFields() has an NPath complexity of 294912 |
371–515 |
These nested if statements could be combined |
484–488 |
These nested if statements could be combined |
494–498 |
The String literal "bAccept" appears 6 times in this file; the first occurrence is on line 826 |
826 |
Avoid unnecessary comparisons in boolean expressions |
829 |
Avoid unnecessary comparisons in boolean expressions |
851 |
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
875 |