Fork me on GitHub

Résultats PMD

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

Fichiers

fr/paris/lutece/plugins/document/modules/rest/rs/DocumentRest.java

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

fr/paris/lutece/plugins/document/modules/rest/service/formatters/AttributeFormatterXml.java

Violation Ligne
Avoid unused imports such as 'org.apache.commons.lang.StringUtils' 42
Avoid unused imports such as 'org.apache.commons.lang.StringUtils' 42

fr/paris/lutece/plugins/document/modules/rest/service/formatters/DocumentPublicationFormatterXml.java

Violation Ligne
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 70

fr/paris/lutece/plugins/document/modules/rest/service/formatters/ReferenceItemFormatterXml.java

Violation Ligne
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 100

fr/paris/lutece/plugins/document/modules/rest/service/writers/AttributeWriter.java

Violation Ligne
This statement may have some unnecessary parentheses 76

fr/paris/lutece/plugins/document/modules/rest/service/writers/DocumentPublicationWriter.java

Violation Ligne
This statement may have some unnecessary parentheses 45

fr/paris/lutece/plugins/document/modules/rest/service/writers/DocumentWriter.java

Violation Ligne
This statement may have some unnecessary parentheses 76

fr/paris/lutece/plugins/document/modules/rest/service/writers/ReferenceItemWriter.java

Violation Ligne
This statement may have some unnecessary parentheses 75

fr/paris/lutece/plugins/document/modules/rest/service/writers/SpaceWriter.java

Violation Ligne
This statement may have some unnecessary parentheses 76

fr/paris/lutece/plugins/document/modules/rest/service/writers/TypeWriter.java

Violation Ligne
This statement may have some unnecessary parentheses 76

fr/paris/lutece/plugins/document/modules/rest/util/builderxml/AddHeaderXml.java

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

fr/paris/lutece/plugins/document/modules/rest/util/builderxml/FieldsToCreateDocumentBuilderXml.java

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

fr/paris/lutece/plugins/document/modules/rest/util/builderxml/ResponseActionBuilderXml.java

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

fr/paris/lutece/plugins/document/modules/rest/util/constants/DocumentRestConstants.java

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