Violation |
Line |
Possible God class (WMC=97, ATFD=167, TCC=0.03508771929824561) |
34–825 |
A high number of imports can indicate a high degree of coupling within an object. |
34–825 |
The class 'CRMDraftBackupService' has a Modified Cyclomatic Complexity of 4 (Highest = 12). |
86–825 |
The class 'CRMDraftBackupService' has a Standard Cyclomatic Complexity of 4 (Highest = 12). |
86–825 |
The class 'CRMDraftBackupService' has a Cyclomatic Complexity of 5 (Highest = 13). |
86–825 |
The Logger variable declaration does not contain the static and final modifiers |
88 |
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
230 |
Avoid unused method parameters such as 'form'. |
306 |
Avoid unnecessary if..then..else statements when returning booleans |
309–312 |
The method 'storeFiles' has a Modified Cyclomatic Complexity of 10. |
324–402 |
The method 'storeFiles' has a Cyclomatic Complexity of 10. |
324–402 |
The method 'storeFiles' has a Standard Cyclomatic Complexity of 10. |
324–402 |
Avoid variables with short names like is |
346 |
Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block |
384 |
The method 'create' has a Cyclomatic Complexity of 11. |
409–482 |
Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block |
474 |
The method 'restore' has a Modified Cyclomatic Complexity of 12. |
488–563 |
The method 'restore' has a Standard Cyclomatic Complexity of 12. |
488–563 |
The method 'restore' has a Cyclomatic Complexity of 13. |
488–563 |
Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block |
547 |
Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block |
651 |
Rather than using a lot of String arguments, consider using a container object for those values. |
783–784 |