Fork me on GitHub

PMD Results

The following document contains the results of PMD 5.3.5.

Files

fr/paris/lutece/plugins/workflow/modules/evaluation/business/TaskEvaluationConfigDAO.java

Violation Line
Use block level rather than method level synchronization 6280

fr/paris/lutece/plugins/workflow/modules/evaluation/business/TaskEvaluationCriteriaDAO.java

Violation Line
Use block level rather than method level synchronization 121138

fr/paris/lutece/plugins/workflow/modules/evaluation/business/dossier/DossierInformationComplementaireDAO.java

Violation Line
Use block level rather than method level synchronization 8195

fr/paris/lutece/plugins/workflow/modules/evaluation/business/evaluation/Evaluation.java

Violation Line
Avoid variables with short names like id 86

fr/paris/lutece/plugins/workflow/modules/evaluation/business/evaluation/EvaluationCriteriaValueDAO.java

Violation Line
Use block level rather than method level synchronization 5770

fr/paris/lutece/plugins/workflow/modules/evaluation/business/evaluation/EvaluationDAO.java

Violation Line
Use block level rather than method level synchronization 5770

fr/paris/lutece/plugins/workflow/modules/evaluation/business/note/Note.java

Violation Line
Avoid short class names like Note 44111

fr/paris/lutece/plugins/workflow/modules/evaluation/business/synthesis/SynthesisCriteria.java

Violation Line
Avoid short class names like Pair 190230

fr/paris/lutece/plugins/workflow/modules/evaluation/business/synthesis/SynthesisCriteriaDAO.java

Violation Line
Use block level rather than method level synchronization 87110

fr/paris/lutece/plugins/workflow/modules/evaluation/business/synthesis/SynthesisCriteriaValueDAO.java

Violation Line
Use block level rather than method level synchronization 92117

fr/paris/lutece/plugins/workflow/modules/evaluation/service/EvaluationPlugin.java

Violation Line
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. 4759

fr/paris/lutece/plugins/workflow/modules/evaluation/service/TaskEvaluation.java

Violation Line
Document empty method 9899

fr/paris/lutece/plugins/workflow/modules/evaluation/service/TaskEvaluationConfigService.java

Violation Line
The class 'TaskEvaluationConfigService' has a Standard Cyclomatic Complexity of 6 (Highest = 14). 58225
The class 'TaskEvaluationConfigService' has a Cyclomatic Complexity of 6 (Highest = 16). 58225
The class 'TaskEvaluationConfigService' has a Modified Cyclomatic Complexity of 6 (Highest = 14). 58225
The method 'update' has a Modified Cyclomatic Complexity of 14. 103184
The method 'update' has a Standard Cyclomatic Complexity of 14. 103184
The method update() has an NPath complexity of 1053 103184
The method 'update' has a Cyclomatic Complexity of 16. 103184

fr/paris/lutece/plugins/workflow/modules/evaluation/web/EvaluationTaskComponent.java

Violation Line
A high number of imports can indicate a high degree of coupling within an object. 34847
Possible God class (WMC=98, ATFD=172, TCC=0.2222222222222222) 34847
The class 'EvaluationTaskComponent' has a Cyclomatic Complexity of 11 (Highest = 30). 84847
The class 'EvaluationTaskComponent' has a Modified Cyclomatic Complexity of 8 (Highest = 14). 84847
The class 'EvaluationTaskComponent' has a Standard Cyclomatic Complexity of 8 (Highest = 14). 84847
Avoid excessively long variable names like MESSAGE_NO_CONFIGURATION_FOR_TASK_EVALUATION_EXPERT 142
Private field '_nNbCritere' could be made final; it is only initialized in the declaration or constructor. 155
Private field '_nCreateValues' could be made final; it is only initialized in the declaration or constructor. 156
The method 'doSaveConfig' has a Modified Cyclomatic Complexity of 14. 173297
Avoid really long methods. 173297
The method doSaveConfig() has an NPath complexity of 9216 173297
The method 'doSaveConfig' has a Standard Cyclomatic Complexity of 14. 173297
The method 'doSaveConfig' has a Cyclomatic Complexity of 23. 173297
Avoid declaring a variable if it is unreferenced before a possible exit point. 178
Avoid declaring a variable if it is unreferenced before a possible exit point. 179
Avoid declaring a variable if it is unreferenced before a possible exit point. 181
Avoid declaring a variable if it is unreferenced before a possible exit point. 190
The method doValidateTask() has an NPath complexity of 354 303404
The method 'doValidateTask' has a Standard Cyclomatic Complexity of 13. 303404
Avoid really long methods. 303404
The method 'doValidateTask' has a Cyclomatic Complexity of 30. 303404
The method 'doValidateTask' has a Modified Cyclomatic Complexity of 13. 303404
Avoid declaring a variable if it is unreferenced before a possible exit point. 306
Avoid declaring a variable if it is unreferenced before a possible exit point. 307
Avoid declaring a variable if it is unreferenced before a possible exit point. 308
Avoid declaring a variable if it is unreferenced before a possible exit point. 309
Avoid declaring a variable if it is unreferenced before a possible exit point. 311
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 412
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 465
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 493
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 502
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 517
The method 'getTaskInformationXml' has a Modified Cyclomatic Complexity of 11. 545655
Avoid really long methods. 545655
The method 'getTaskInformationXml' has a Standard Cyclomatic Complexity of 11. 545655
The method 'getTaskInformationXml' has a Cyclomatic Complexity of 11. 545655
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 583
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 619
Avoid declaring a variable if it is unreferenced before a possible exit point. 674
Avoid declaring a variable if it is unreferenced before a possible exit point. 675676
The method 'getSynthesisCriteriaData' has a Standard Cyclomatic Complexity of 10. 723824
The method 'getSynthesisCriteriaData' has a Cyclomatic Complexity of 11. 723824
Avoid really long methods. 723824
The method 'getSynthesisCriteriaData' has a Modified Cyclomatic Complexity of 10. 723824
The method getSynthesisCriteriaData() has an NPath complexity of 289 723824
Avoid declaring a variable if it is unreferenced before a possible exit point. 756757
Avoid declaring a variable if it is unreferenced before a possible exit point. 759
Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 781