Fork me on GitHub

Résultats PMD

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

Fichiers

fr/paris/lutece/plugins/atelieraba/business/CoursDAO.java

Violation Ligne
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 328
Avoid variables with short names like cc 335
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 357

fr/paris/lutece/plugins/atelieraba/business/EleveArpege.java

Violation Ligne
Avoid unused imports such as 'org.hibernate.validator.constraints.NotEmpty' 37
Avoid unused imports such as 'org.hibernate.validator.constraints.NotEmpty' 37

fr/paris/lutece/plugins/atelieraba/business/EleveArpegeDAO.java

Violation Ligne
Rather than using a lot of String arguments, consider using a container object for those values. 215

fr/paris/lutece/plugins/atelieraba/business/EleveArpegeHome.java

Violation Ligne
Rather than using a lot of String arguments, consider using a container object for those values. 138

fr/paris/lutece/plugins/atelieraba/business/IEleveArpegeDAO.java

Violation Ligne
Rather than using a lot of String arguments, consider using a container object for those values. 131

fr/paris/lutece/plugins/atelieraba/business/Inscrit.java

Violation Ligne
This class has a bunch of public methods and attributes 34572
Too many fields 48572
A getX() method which returns a boolean should be named isX() 415418
A getX() method which returns a boolean should be named isX() 452455
A getX() method which returns a boolean should be named isX() 462465
A getX() method which returns a boolean should be named isX() 472475
A getX() method which returns a boolean should be named isX() 482485
A getX() method which returns a boolean should be named isX() 492495
A getX() method which returns a boolean should be named isX() 502505
A getX() method which returns a boolean should be named isX() 512515
A getX() method which returns a boolean should be named isX() 522525
A getX() method which returns a boolean should be named isX() 532535
A getX() method which returns a boolean should be named isX() 542545
A getX() method which returns a boolean should be named isX() 552555
A getX() method which returns a boolean should be named isX() 562565

fr/paris/lutece/plugins/atelieraba/business/InscritDAO.java

Violation Ligne
StringBuffer constructor is initialized with size 16, but has at least 44 characters appended. 346
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 348
StringBuffer (or StringBuilder).append is called 2 consecutive times with literal Strings. Use a single append with a single combined String. 348
When doing a String.toLowerCase()/toUpperCase() call, use a Locale 371
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 433

fr/paris/lutece/plugins/atelieraba/business/InscritHome.java

Violation Ligne
Avoid unused imports such as 'java.util.HashMap' 40
Avoid unused imports such as 'java.util.HashMap' 40
The class 'InscritHome' has a Standard Cyclomatic Complexity of 2 (Highest = 10). 53324
The class 'InscritHome' has a Cyclomatic Complexity of 2 (Highest = 17). 53324
The class 'InscritHome' has a Modified Cyclomatic Complexity of 2 (Highest = 10). 53324
The method 'getInscritsListOrderedPartialDuplicatesExport' has a Modified Cyclomatic Complexity of 10. 198279
The method 'getInscritsListOrderedPartialDuplicatesExport' has a Cyclomatic Complexity of 17. 198279
The method 'getInscritsListOrderedPartialDuplicatesExport' has a Standard Cyclomatic Complexity of 10. 198279
Avoid declaring a variable if it is unreferenced before a possible exit point. 201

fr/paris/lutece/plugins/atelieraba/utils/constants/AtelierabaConstants.java

Violation Ligne
A class which only has private constructors should be final 830

fr/paris/lutece/plugins/atelieraba/web/CandidaturesJspBean.java

Violation Ligne
A high number of imports can indicate a high degree of coupling within an object. 1410
The class 'CandidaturesJspBean' has a Cyclomatic Complexity of 10 (Highest = 29). 44410
The class 'CandidaturesJspBean' has a Standard Cyclomatic Complexity of 8 (Highest = 21). 44410
The class 'CandidaturesJspBean' has a Modified Cyclomatic Complexity of 8 (Highest = 21). 44410
The String literal "STR" appears 7 times in this file; the first occurrence is on line 50 50
Avoid using implementation types like 'HashMap'; use the interface instead 101
Avoid using implementation types like 'HashMap'; use the interface instead 101
Perhaps '_nDefaultItemsPerPage' could be replaced by a local variable. 102
The method getManageCandidatures() has an NPath complexity of 46176 125278
The method 'getManageCandidatures' has a Standard Cyclomatic Complexity of 21. 125278
The method 'getManageCandidatures' has a Modified Cyclomatic Complexity of 21. 125278
Avoid really long methods. 125278
The method 'getManageCandidatures' has a Cyclomatic Complexity of 29. 125278
Avoid variables with short names like c 153
Avoid variables with short names like c 153
Avoid using Literals in Conditional Statements 172
Avoid using Literals in Conditional Statements 172
Avoid using Literals in Conditional Statements 186
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 195
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 198
The String literal "places_vivantes" appears 4 times in this file; the first occurrence is on line 199 199
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 209
Avoid using Literals in Conditional Statements 219
Do not add empty strings 273

fr/paris/lutece/plugins/atelieraba/web/ConsultationCoursJspBean.java

Violation Ligne
Do not add empty strings 84

fr/paris/lutece/plugins/atelieraba/web/CoursJspBean.java

Violation Ligne
The class 'CoursJspBean' has a Cyclomatic Complexity of 6 (Highest = 11). 29176
The method 'doImport' has a Cyclomatic Complexity of 11. 82174
Avoid variables with short names like cd 91
Avoid variables with short names like cm 93
Avoid declaring a variable if it is unreferenced before a possible exit point. 94
Do not add empty strings 110
Do not add empty strings 126

fr/paris/lutece/plugins/atelieraba/web/DedoublonnageManuelJspBean.java

Violation Ligne
Avoid unused imports such as 'java.util.Collection' 5
Avoid unused imports such as 'java.util.Collection' 5
Avoid unused private fields such as 'PARAMETER_ID_ACTION'. 60
Avoid unused private fields such as 'PARAMETER_ID_INSCRIT'. 61
Avoid unused private fields such as 'MARK_ACTIONS_1'. 70
Avoid unused private fields such as 'MARK_ACTIONS_2'. 74
Perhaps '_nDefaultItemsPerPage' could be replaced by a local variable. 89
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 118
Do not add empty strings 157
Avoid declaring a variable if it is unreferenced before a possible exit point. 186

fr/paris/lutece/plugins/atelieraba/web/ElevesArpegeJspBean.java

Violation Ligne
The class 'ElevesArpegeJspBean' has a Cyclomatic Complexity of 6 (Highest = 10). 29172
The class 'ElevesArpegeJspBean' has a Modified Cyclomatic Complexity of 6 (Highest = 10). 29172
The class 'ElevesArpegeJspBean' has a Standard Cyclomatic Complexity of 6 (Highest = 10). 29172
The method 'doImport' has a Modified Cyclomatic Complexity of 10. 83170
The method 'doImport' has a Cyclomatic Complexity of 10. 83170
The method 'doImport' has a Standard Cyclomatic Complexity of 10. 83170
Avoid variables with short names like cd 92
Avoid variables with short names like cm 94
Avoid declaring a variable if it is unreferenced before a possible exit point. 95
Do not add empty strings 111
Do not add empty strings 127

fr/paris/lutece/plugins/atelieraba/web/FormulaireXPage.java

Violation Ligne
Possible God class (WMC=151, ATFD=189, TCC=0.27076923076923076) 341236
A high number of imports can indicate a high degree of coupling within an object. 341236
The class 'FormulaireXPage' has a Standard Cyclomatic Complexity of 4 (Highest = 10). 861236
The class 'FormulaireXPage' has a Cyclomatic Complexity of 6 (Highest = 15). 861236
Avoid really long classes. 861236
The class 'FormulaireXPage' has a Modified Cyclomatic Complexity of 4 (Highest = 10). 861236
The String literal "atelier" appears 4 times in this file; the first occurrence is on line 105 105
Private field '_mapErrorFields' could be made final; it is only initialized in the declaration or constructor. 210
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 210
The method 'getPage' has a Cyclomatic Complexity of 10. 267302
Deeply nested if..then statements are hard to read 279283
Assigning an Object to null is a code smell. Consider refactoring. 299
The method 'getTOS' has a Cyclomatic Complexity of 10. 312351
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 339
Avoid using Literals in Conditional Statements 369
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 374
The method 'getAtelier' has a Cyclomatic Complexity of 10. 399471
The method getAtelier() has an NPath complexity of 288 399471
Avoid variables with short names like c 431
Avoid variables with short names like c 431
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 433
Avoid variables with short names like id 437
The method doAtelier() has an NPath complexity of 1088 481542
The method 'doAtelier' has a Cyclomatic Complexity of 15. 481542
The method doInformations() has an NPath complexity of 864 587650
The method 'doInformations' has a Cyclomatic Complexity of 14. 587650
When instantiating a SimpleDateFormat object, specify a Locale 625
The method doContact() has an NPath complexity of 288 697756
The method 'doContact' has a Cyclomatic Complexity of 12. 697756
Avoid declaring a variable if it is unreferenced before a possible exit point. 862
The method 'doRecap' has a Standard Cyclomatic Complexity of 10. 892957
The method doRecap() has an NPath complexity of 300 892957
The method 'doRecap' has a Cyclomatic Complexity of 10. 892957
The method 'doRecap' has a Modified Cyclomatic Complexity of 10. 892957
Avoid empty catch blocks 898900
Avoid using Literals in Conditional Statements 938
Avoid using Literals in Conditional Statements 942
Avoid declaring a variable if it is unreferenced before a possible exit point. 970
When doing a String.toLowerCase()/toUpperCase() call, use a Locale 1065
When instantiating a SimpleDateFormat object, specify a Locale 1115
Avoid empty catch blocks 11171119
Avoid variables with short names like m 1147
Avoid variables with short names like l 1148
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 1215