Fork me on GitHub

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 415417

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. 86147
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 416417
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 463472

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

Rule Violation Ligne
CommentRequired Header comments are required 54180

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. 50243
LinguisticNaming Linguistics Antipattern - The method 'canUploadFiles' indicates linguistically it returns a boolean, but it returns 'String' 85
UncommentedEmptyMethodBody Document empty method body 241242

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 4952

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

Rule Violation Ligne
ConfusingTernary Avoid if (x != y) ..; else ..; 92109

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

Rule Violation Ligne
CyclomaticComplexity The method 'getMainUploadJs(HttpServletRequest, Boolean)' has a cyclomatic complexity of 10. 109208
NcssCount The method 'getMainUploadJs(HttpServletRequest, Boolean)' has a NCSS line count of 60. 109208
NPathComplexity The method 'getMainUploadJs(HttpServletRequest, Boolean)' has an NPath complexity of 512 109208
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 86147
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 415417
UncommentedEmptyMethodBody Document empty method body 3 416417
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 463472

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

Rule Violation Priority Ligne
CommentRequired Header comments are required 3 54180

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 50243
LinguisticNaming Linguistics Antipattern - The method 'canUploadFiles' indicates linguistically it returns a boolean, but it returns 'String' 3 85
UncommentedEmptyMethodBody Document empty method body 3 241242

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 4952

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

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

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

Rule Violation Priority Ligne
CyclomaticComplexity The method 'getMainUploadJs(HttpServletRequest, Boolean)' has a cyclomatic complexity of 10. 3 109208
NcssCount The method 'getMainUploadJs(HttpServletRequest, Boolean)' has a NCSS line count of 60. 3 109208
NPathComplexity The method 'getMainUploadJs(HttpServletRequest, Boolean)' has an NPath complexity of 512 3 109208
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