Fork me on GitHub

Résultats PMD

Le document suivant contient les résultats de PMD 5.3.5.

Fichiers

fr/paris/lutece/plugins/crm/modules/form/service/CRMParametersService.java

Violation Ligne
Avoid using implementation types like 'HashMap'; use the interface instead 24
Avoid using implementation types like 'HashMap'; use the interface instead 24

fr/paris/lutece/plugins/crm/modules/form/service/draft/CRMDraftBackupService.java

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

fr/paris/lutece/plugins/crm/modules/form/util/JSONUtils.java

Violation Ligne
Possible God class (WMC=50, ATFD=48, TCC=0.06666666666666667) 1–402
The class 'JSONUtils' has a Cyclomatic Complexity of 5 (Highest = 12). 37–402
The method 'buildJson' has a Cyclomatic Complexity of 12. 93–145
Deeply nested if..then statements are hard to read 125–132
The method 'getBlobIds' has a Cyclomatic Complexity of 12. 173–216
Avoid empty catch blocks 210–213
Avoid unused method parameters such as 'session'. 275
Assigning an Object to null is a code smell. Consider refactoring. 388
Assigning an Object to null is a code smell. Consider refactoring. 394