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 3

fr/paris/lutece/plugins/workflow/modules/formsautomaticassignment/business/AutomaticAssignment.java

Rule Violation Ligne
CommentRequired Public method and constructor comments are required 155–158
CommentRequired Public method and constructor comments are required 160–163

fr/paris/lutece/plugins/workflow/modules/formsautomaticassignment/business/AutomaticAssignmentDAO.java

Rule Violation Ligne
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 122–135

fr/paris/lutece/plugins/workflow/modules/formsautomaticassignment/business/TaskAutomaticAssignmentConfigDAO.java

Rule Violation Ligne
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 67–87

fr/paris/lutece/plugins/workflow/modules/formsautomaticassignment/service/AutomaticAssignmentService.java

Rule Violation Ligne
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 34–602
TooManyMethods This class has too many methods, consider refactoring it. 94–602
ConfusingTernary Avoid if (x != y) ..; else ..; 448–465
ShortVariable Avoid variables with short names like t 481
CyclomaticComplexity The method 'buildModel(TaskAutomaticAssignmentConfig, ResourceHistory, List, HttpServletRequest, Locale)' has a cyclomatic complexity of 10. 516–600
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 519
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 585
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 591

fr/paris/lutece/plugins/workflow/modules/formsautomaticassignment/service/IAutomaticAssignmentService.java

Rule Violation Ligne
CommentRequired Public method and constructor comments are required 160

fr/paris/lutece/plugins/workflow/modules/formsautomaticassignment/service/TaskAutomaticAssignment.java

Rule Violation Ligne
CyclomaticComplexity The method 'processTask(int, HttpServletRequest, Locale)' has a cyclomatic complexity of 10. 91–138
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 93

fr/paris/lutece/plugins/workflow/modules/formsautomaticassignment/web/AutomaticAssignmentJspBean.java

Rule Violation Ligne
ImmutableField Private field '_automaticAssignmentService' could be made final; it is only initialized in the declaration or constructor. 90
ImmutableField Private field '_taskAutomaticAssignmentConfigService' could be made final; it is only initialized in the declaration or constructor. 91
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 178
ConfusingTernary Avoid if (x != y) ..; else ..; 210–217

fr/paris/lutece/plugins/workflow/modules/formsautomaticassignment/web/AutomaticAssignmentTaskComponent.java

Rule Violation Ligne
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 34–426
ExcessiveMethodLength Avoid really long methods. 147–295
NcssCount The method 'doSaveConfig(HttpServletRequest, Locale, ITask)' has a NCSS line count of 78. 147–295
NPathComplexity The method 'doSaveConfig(HttpServletRequest, Locale, ITask)' has an NPath complexity of 2211840 147–295
CyclomaticComplexity The method 'doSaveConfig(HttpServletRequest, Locale, ITask)' has a cyclomatic complexity of 29. 147–295
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 156
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 157
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 160
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 161
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 162
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 163
ForLoopCanBeForeach This for loop can be replaced by a foreach loop 220–240
ConfusingTernary Avoid if (x != y) ..; else ..; 228–236
ForLoopCanBeForeach This for loop can be replaced by a foreach loop 273–276
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 304
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 319
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 380

Fichiers

fr/paris/lutece/plugins/workflow/modules/formsautomaticassignment/business/AutomaticAssignment.java

Rule Violation Priority Ligne
CommentRequired Public method and constructor comments are required 3 155–158
CommentRequired Public method and constructor comments are required 3 160–163

fr/paris/lutece/plugins/workflow/modules/formsautomaticassignment/business/AutomaticAssignmentDAO.java

Rule Violation Priority Ligne
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 3 122–135

fr/paris/lutece/plugins/workflow/modules/formsautomaticassignment/business/TaskAutomaticAssignmentConfigDAO.java

Rule Violation Priority Ligne
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 3 67–87

fr/paris/lutece/plugins/workflow/modules/formsautomaticassignment/service/AutomaticAssignmentService.java

Rule Violation Priority Ligne
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 3 34–602
TooManyMethods This class has too many methods, consider refactoring it. 3 94–602
ConfusingTernary Avoid if (x != y) ..; else ..; 3 448–465
ShortVariable Avoid variables with short names like t 3 481
CyclomaticComplexity The method 'buildModel(TaskAutomaticAssignmentConfig, ResourceHistory, List, HttpServletRequest, Locale)' has a cyclomatic complexity of 10. 3 516–600
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 519
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 585
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 591

fr/paris/lutece/plugins/workflow/modules/formsautomaticassignment/service/IAutomaticAssignmentService.java

Rule Violation Priority Ligne
CommentRequired Public method and constructor comments are required 3 160

fr/paris/lutece/plugins/workflow/modules/formsautomaticassignment/service/TaskAutomaticAssignment.java

Rule Violation Priority Ligne
CyclomaticComplexity The method 'processTask(int, HttpServletRequest, Locale)' has a cyclomatic complexity of 10. 3 91–138
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 93

fr/paris/lutece/plugins/workflow/modules/formsautomaticassignment/web/AutomaticAssignmentJspBean.java

Rule Violation Priority Ligne
ImmutableField Private field '_automaticAssignmentService' could be made final; it is only initialized in the declaration or constructor. 3 90
ImmutableField Private field '_taskAutomaticAssignmentConfigService' could be made final; it is only initialized in the declaration or constructor. 3 91
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 178
ConfusingTernary Avoid if (x != y) ..; else ..; 3 210–217

fr/paris/lutece/plugins/workflow/modules/formsautomaticassignment/web/AutomaticAssignmentTaskComponent.java

Rule Violation Priority Ligne
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 3 34–426
ExcessiveMethodLength Avoid really long methods. 3 147–295
NcssCount The method 'doSaveConfig(HttpServletRequest, Locale, ITask)' has a NCSS line count of 78. 3 147–295
NPathComplexity The method 'doSaveConfig(HttpServletRequest, Locale, ITask)' has an NPath complexity of 2211840 3 147–295
CyclomaticComplexity The method 'doSaveConfig(HttpServletRequest, Locale, ITask)' has a cyclomatic complexity of 29. 3 147–295
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 156
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 157
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 160
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 161
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 162
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 163
ForLoopCanBeForeach This for loop can be replaced by a foreach loop 3 220–240
ConfusingTernary Avoid if (x != y) ..; else ..; 3 228–236
ForLoopCanBeForeach This for loop can be replaced by a foreach loop 3 273–276
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 304
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 319
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 380