Violation |
Line |
A high number of imports can indicate a high degree of coupling within an object. |
34–1249 |
Possible God class (WMC=101, ATFD=257, TCC=0.328042328042328) |
34–1249 |
The class 'SponsoredLinksJspBean' has a Cyclomatic Complexity of 4 (Highest = 12). |
82–1249 |
Avoid really long classes. |
82–1249 |
Private field '_nDefaultItemsPerPage' could be made final; it is only initialized in the declaration or constructor. |
189 |
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
207 |
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
240 |
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
241 |
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
293 |
The method 'doCreateSet' has a Cyclomatic Complexity of 10. |
311–366 |
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
395 |
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
397 |
The method 'doModifySet' has a Cyclomatic Complexity of 12. |
435–495 |
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
596 |
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
609 |
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
648 |
This statement may have some unnecessary parentheses |
654 |
The method doCreateGroup() has an NPath complexity of 234 |
675–714 |
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
739 |
This statement may have some unnecessary parentheses |
743 |
The method 'doModifyGroup' has a Cyclomatic Complexity of 12. |
776–820 |
The method doModifyGroup() has an NPath complexity of 2520 |
776–820 |
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
887 |
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
928 |
Avoid catching NullPointerException; consider removing the cause of the NPE. |
936 |
Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block |
936 |
Avoid catching NullPointerException; consider removing the cause of the NPE. |
974 |
Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block |
974 |
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
980 |
Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block |
1091 |
Avoid catching NullPointerException; consider removing the cause of the NPE. |
1091 |
Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block |
1104 |
Avoid catching NullPointerException; consider removing the cause of the NPE. |
1104 |
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
1110 |