| Violation |
Line |
| 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 |
