Le document suivant contient les résultats de PMD 5.5.1.
Violation | Ligne |
---|---|
A high number of imports can indicate a high degree of coupling within an object. | 34–1129 |
Possible God class (WMC=128, ATFD=333, TCC=0.0) | 34–1129 |
The class 'DocumentRest' has a Modified Cyclomatic Complexity of 4 (Highest = 18). | 115–1129 |
The class 'DocumentRest' has a Standard Cyclomatic Complexity of 4 (Highest = 18). | 115–1129 |
The class 'DocumentRest' has a Cyclomatic Complexity of 6 (Highest = 23). | 115–1129 |
Avoid really long classes. | 115–1129 |
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 138 |
Avoid variables with short names like t | 141 |
The method 'doSubmitOrUnsubmitDocumentToValidate' has a Cyclomatic Complexity of 13. | 459–502 |
Deeply nested if..then statements are hard to read | 479–486 |
The method 'doValidateDocument' has a Cyclomatic Complexity of 13. | 521–572 |
The method 'getDocumentData' has a Cyclomatic Complexity of 23. | 813–930 |
The method 'getDocumentData' has a Standard Cyclomatic Complexity of 18. | 813–930 |
The method getDocumentData() has an NPath complexity of 583200 | 813–930 |
The method 'getDocumentData' has a Modified Cyclomatic Complexity of 18. | 813–930 |
Avoid really long methods. | 813–930 |
Avoid declaring a variable if it is unreferenced before a possible exit point. | 817 |
Avoid declaring a variable if it is unreferenced before a possible exit point. | 818 |
Avoid declaring a variable if it is unreferenced before a possible exit point. | 819 |
Avoid declaring a variable if it is unreferenced before a possible exit point. | 822 |
Avoid declaring a variable if it is unreferenced before a possible exit point. | 825 |
Avoid declaring a variable if it is unreferenced before a possible exit point. | 826 |
Avoid declaring a variable if it is unreferenced before a possible exit point. | 854 |
These nested if statements could be combined | 893–896 |
The method 'setAttribute' has a Cyclomatic Complexity of 15. | 943–1013 |
The method 'setAttribute' has a Modified Cyclomatic Complexity of 10. | 943–1013 |
The method 'setAttribute' has a Standard Cyclomatic Complexity of 10. | 943–1013 |
The method setAttribute() has an NPath complexity of 660 | 943–1013 |
Avoid declaring a variable if it is unreferenced before a possible exit point. | 948 |
The method 'modifyDocument' has a Cyclomatic Complexity of 10. | 1063–1128 |
Violation | Ligne |
---|---|
Avoid unused imports such as 'org.apache.commons.lang.StringUtils' | 42 |
Avoid unused imports such as 'org.apache.commons.lang.StringUtils' | 42 |
Violation | Ligne |
---|---|
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 70 |
Violation | Ligne |
---|---|
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 100 |
Violation | Ligne |
---|---|
This statement may have some unnecessary parentheses | 76 |
Violation | Ligne |
---|---|
This statement may have some unnecessary parentheses | 45 |
Violation | Ligne |
---|---|
This statement may have some unnecessary parentheses | 76 |
Violation | Ligne |
---|---|
This statement may have some unnecessary parentheses | 75 |
Violation | Ligne |
---|---|
This statement may have some unnecessary parentheses | 76 |
Violation | Ligne |
---|---|
This statement may have some unnecessary parentheses | 76 |
Violation | Ligne |
---|---|
All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. | 47–70 |
Document empty constructor | 51–53 |
Violation | Ligne |
---|---|
All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. | 57–285 |
Document empty constructor | 62–64 |
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 112 |
The String literal "text" appears 6 times in this file; the first occurrence is on line 114 | 114 |
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 225 |
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 245 |
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 266 |
Violation | Ligne |
---|---|
All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. | 46–79 |
Document empty constructor | 50–52 |
Violation | Ligne |
---|---|
A class which only has private constructors should be final | 41–201 |
Avoid excessively long variable names like PATH_GET_SPACES_ALLOWING_DOCUMENT_CREATION_BY_CODE_DOCUMENT_TYPE | 52 |
Avoid excessively long variable names like TAG_CREATE_DOCUMENT_FIELDS_FORM_PARAMETER_ATTRIBUT_VALUE | 120 |