Fork me on GitHub
Produit par Maven

Résultats PMD

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

Violations By Priority

Priority 1

fr/paris/lutece/plugins/asynchronousupload/service/AbstractAsynchronousUploadHandler.java

Rule Violation Ligne
EmptyMethodInAbstractClassShouldBeAbstract An empty method in an abstract class should be abstract instead 415–417

Priority 3

fr/paris/lutece/plugins/asynchronousupload/service/AbstractAsynchronousUploadHandler.java

Rule Violation Ligne
CyclomaticComplexity The method 'process(HttpServletRequest, HttpServletResponse, Map, List)' has a cyclomatic complexity of 11. 86–147
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 104
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 141
AvoidReassigningLoopVariables Avoid reassigning the loop control variable 'nFieldIndex' 264
UncommentedEmptyMethodBody Document empty method body 416–417
ShortVariable Avoid variables with short names like r 443
ShortVariable Avoid variables with short names like m 444
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 453
CommentRequired Protected method constructor comments are required 463–472

fr/paris/lutece/plugins/asynchronousupload/service/AsynchronousUploadHandler.java

Rule Violation Ligne
CommentRequired Header comments are required 54–180

fr/paris/lutece/plugins/asynchronousupload/service/AsynchronousUploadSessionListener.java

Rule Violation Ligne
ShortVariable Avoid variables with short names like se 50
ShortVariable Avoid variables with short names like se 59

fr/paris/lutece/plugins/asynchronousupload/service/IAsyncUploadHandler.java

Rule Violation Ligne
TooManyMethods This class has too many methods, consider refactoring it. 50–243
LinguisticNaming Linguistics Antipattern - The method 'canUploadFiles' indicates linguistically it returns a boolean, but it returns 'String' 85
UncommentedEmptyMethodBody Document empty method body 241–242

fr/paris/lutece/plugins/asynchronousupload/service/PartialFileItemGroup.java

Rule Violation Ligne
ImmutableField Private field '_items' could be made final; it is only initialized in the declaration or constructor. 61
ImmutableField Private field '_sequenceInputStream' could be made final; it is only initialized in the declaration or constructor. 62

fr/paris/lutece/plugins/asynchronousupload/service/UploadCacheService.java

Rule Violation Ligne
CallSuperInConstructor It is a good practice to call super() in a constructor 49–52

fr/paris/lutece/plugins/asynchronousupload/util/JSONUtils.java

Rule Violation Ligne
ConfusingTernary Avoid if (x != y) ..; else ..; 92–109

fr/paris/lutece/plugins/asynchronousupload/web/AsynchronousUploadApp.java

Rule Violation Ligne
NPathComplexity The method 'getMainUploadJs(HttpServletRequest, Boolean)' has an NPath complexity of 512 109–208
NcssCount The method 'getMainUploadJs(HttpServletRequest, Boolean)' has a NCSS line count of 60. 109–208
CyclomaticComplexity The method 'getMainUploadJs(HttpServletRequest, Boolean)' has a cyclomatic complexity of 10. 109–208
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 113
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 124
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 179

Fichiers

fr/paris/lutece/plugins/asynchronousupload/service/AbstractAsynchronousUploadHandler.java

Rule Violation Priority Ligne
CyclomaticComplexity The method 'process(HttpServletRequest, HttpServletResponse, Map, List)' has a cyclomatic complexity of 11. 3 86–147
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 3 104
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 141
AvoidReassigningLoopVariables Avoid reassigning the loop control variable 'nFieldIndex' 3 264
EmptyMethodInAbstractClassShouldBeAbstract An empty method in an abstract class should be abstract instead 1 415–417
UncommentedEmptyMethodBody Document empty method body 3 416–417
ShortVariable Avoid variables with short names like r 3 443
ShortVariable Avoid variables with short names like m 3 444
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 3 453
CommentRequired Protected method constructor comments are required 3 463–472

fr/paris/lutece/plugins/asynchronousupload/service/AsynchronousUploadHandler.java

Rule Violation Priority Ligne
CommentRequired Header comments are required 3 54–180

fr/paris/lutece/plugins/asynchronousupload/service/AsynchronousUploadSessionListener.java

Rule Violation Priority Ligne
ShortVariable Avoid variables with short names like se 3 50
ShortVariable Avoid variables with short names like se 3 59

fr/paris/lutece/plugins/asynchronousupload/service/IAsyncUploadHandler.java

Rule Violation Priority Ligne
TooManyMethods This class has too many methods, consider refactoring it. 3 50–243
LinguisticNaming Linguistics Antipattern - The method 'canUploadFiles' indicates linguistically it returns a boolean, but it returns 'String' 3 85
UncommentedEmptyMethodBody Document empty method body 3 241–242

fr/paris/lutece/plugins/asynchronousupload/service/PartialFileItemGroup.java

Rule Violation Priority Ligne
ImmutableField Private field '_items' could be made final; it is only initialized in the declaration or constructor. 3 61
ImmutableField Private field '_sequenceInputStream' could be made final; it is only initialized in the declaration or constructor. 3 62

fr/paris/lutece/plugins/asynchronousupload/service/UploadCacheService.java

Rule Violation Priority Ligne
CallSuperInConstructor It is a good practice to call super() in a constructor 3 49–52

fr/paris/lutece/plugins/asynchronousupload/util/JSONUtils.java

Rule Violation Priority Ligne
ConfusingTernary Avoid if (x != y) ..; else ..; 3 92–109

fr/paris/lutece/plugins/asynchronousupload/web/AsynchronousUploadApp.java

Rule Violation Priority Ligne
NPathComplexity The method 'getMainUploadJs(HttpServletRequest, Boolean)' has an NPath complexity of 512 3 109–208
NcssCount The method 'getMainUploadJs(HttpServletRequest, Boolean)' has a NCSS line count of 60. 3 109–208
CyclomaticComplexity The method 'getMainUploadJs(HttpServletRequest, Boolean)' has a cyclomatic complexity of 10. 3 109–208
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 113
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 124
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 179