Fork me on GitHub

PMD Results

The following document contains the results of PMD 5.3.5.

Files

fr/paris/lutece/plugins/selfregistration/business/LdapUser.java

Violation Line
Too many fields 44382
Avoid unnecessary constructors - the compiler will generate these for you 7577
Document empty constructor 7577

fr/paris/lutece/plugins/selfregistration/business/LdapUserDAO.java

Violation Line
Possible God class (WMC=52, ATFD=39, TCC=0.0735632183908046) 34579
The class 'LdapUserDAO' has a Modified Cyclomatic Complexity of 2 (Highest = 14). 60579
The class 'LdapUserDAO' has a Standard Cyclomatic Complexity of 2 (Highest = 14). 60579
The class 'LdapUserDAO' has a Cyclomatic Complexity of 2 (Highest = 14). 60579
Consider replacing this Hashtable with the newer java.util.Map 116
The method createLdapUser() has an NPath complexity of 8192 253343
The method 'createLdapUser' has a Cyclomatic Complexity of 14. 253343
The method 'createLdapUser' has a Modified Cyclomatic Complexity of 14. 253343
The method 'createLdapUser' has a Standard Cyclomatic Complexity of 14. 253343

fr/paris/lutece/plugins/selfregistration/web/SelfRegistrationApp.java

Violation Line
The class 'SelfRegistrationApp' has a Modified Cyclomatic Complexity of 7 (Highest = 14). 73680
The class 'SelfRegistrationApp' has a Standard Cyclomatic Complexity of 7 (Highest = 14). 73680
The class 'SelfRegistrationApp' has a Cyclomatic Complexity of 17 (Highest = 48). 73680
Private field '_captchaService' could be made final; it is only initialized in the declaration or constructor. 166
The method 'getPage' has a Cyclomatic Complexity of 10. 188260
Avoid variables with short names like t 304
Avoid variables with short names like t 351
The method 'doCreateRegistration' has a Cyclomatic Complexity of 36. 364511
The method 'doCreateRegistration' has a Modified Cyclomatic Complexity of 13. 364511
Avoid really long methods. 364511
The method 'doCreateRegistration' has a Standard Cyclomatic Complexity of 13. 364511
The method doCreateRegistration() has an NPath complexity of 526500 364511
Avoid declaring a variable if it is unreferenced before a possible exit point. 384
Avoid declaring a variable if it is unreferenced before a possible exit point. 385
Avoid declaring a variable if it is unreferenced before a possible exit point. 386
Avoid declaring a variable if it is unreferenced before a possible exit point. 387
Avoid declaring a variable if it is unreferenced before a possible exit point. 389
Avoid declaring a variable if it is unreferenced before a possible exit point. 390
Avoid declaring a variable if it is unreferenced before a possible exit point. 391
These nested if statements could be combined 458461
The method 'doModifyRegistration' has a Cyclomatic Complexity of 48. 518679
The method 'doModifyRegistration' has a Modified Cyclomatic Complexity of 14. 518679
Avoid really long methods. 518679
The method 'doModifyRegistration' has a Standard Cyclomatic Complexity of 14. 518679
The method doModifyRegistration() has an NPath complexity of 6439500 518679
Avoid declaring a variable if it is unreferenced before a possible exit point. 539
Avoid declaring a variable if it is unreferenced before a possible exit point. 540
Avoid declaring a variable if it is unreferenced before a possible exit point. 541
Avoid declaring a variable if it is unreferenced before a possible exit point. 542
Avoid declaring a variable if it is unreferenced before a possible exit point. 543
Avoid declaring a variable if it is unreferenced before a possible exit point. 544
Avoid declaring a variable if it is unreferenced before a possible exit point. 545
Avoid declaring a variable if it is unreferenced before a possible exit point. 546
Avoid declaring a variable if it is unreferenced before a possible exit point. 547
Avoid declaring a variable if it is unreferenced before a possible exit point. 548