Fork me on GitHub

Résultats PMD

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

Fichiers

fr/paris/lutece/plugins/mydashboard/modules/identity/business/DashboardAttribute.java

Violation Ligne
Document empty constructor 5154
Assigning an Object to null is a code smell. Consider refactoring. 70
Assigning an Object to null is a code smell. Consider refactoring. 71
Assigning an Object to null is a code smell. Consider refactoring. 72

fr/paris/lutece/plugins/mydashboard/modules/identity/business/DashboardIdentity.java

Violation Ligne
This class has a bunch of public methods and attributes 34857
Possible God class (WMC=94, ATFD=137, TCC=0.0) 34857
Private field '_mapAttributes' could be made final; it is only initialized in the declaration or constructor. 47

fr/paris/lutece/plugins/mydashboard/modules/identity/util/DashboardIdentityUtils.java

Violation Ligne
A class which only has private constructors should be final 54258
Possible unsafe assignment to a non-final static field in a constructor. 59
Singleton is not thread safe 112115
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 167

fr/paris/lutece/plugins/mydashboard/modules/identity/web/IdentityXPage.java

Violation Ligne
A high number of imports can indicate a high degree of coupling within an object. 34884
Possible God class (WMC=73, ATFD=159, TCC=0.018461538461538463) 34884
The class 'IdentityXPage' has a Standard Cyclomatic Complexity of 3 (Highest = 20). 77884
The class 'IdentityXPage' has a Modified Cyclomatic Complexity of 3 (Highest = 20). 77884
The class 'IdentityXPage' has a Cyclomatic Complexity of 3 (Highest = 21). 77884
Private field '_lstContactModeList' could be made final; it is only initialized in the declaration or constructor. 116
Private field '_lstGenderList' could be made final; it is only initialized in the declaration or constructor. 117
Private field '_identityService' could be made final; it is only initialized in the declaration or constructor. 121
Avoid variables with short names like i 133
Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 256
Assigning an Object to null is a code smell. Consider refactoring. 279
The method 'checkDashboardIdentityFields' has a Modified Cyclomatic Complexity of 20. 371515
Avoid really long methods. 371515
The method 'checkDashboardIdentityFields' has a Cyclomatic Complexity of 21. 371515
The method 'checkDashboardIdentityFields' has a Standard Cyclomatic Complexity of 20. 371515
The method checkDashboardIdentityFields() has an NPath complexity of 294912 371515
These nested if statements could be combined 484488
These nested if statements could be combined 494498
The String literal "bAccept" appears 6 times in this file; the first occurrence is on line 826 826
Avoid unnecessary comparisons in boolean expressions 829
Avoid unnecessary comparisons in boolean expressions 851
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 875

fr/paris/lutece/plugins/mydashboard/modules/identity/web/MyDashboardIdentityComponent.java

Violation Ligne
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 88
Assigning an Object to null is a code smell. Consider refactoring. 90