Fork me on GitHub

Résultats PMD

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

Violations By Priority

Priority 2

fr/paris/lutece/plugins/subscribe/business/SubscriptionDAO.java

Rule Violation Ligne
AvoidBranchingStatementAsLastInLoop Avoid using a branching statement as the last in a loop. 315

fr/paris/lutece/plugins/subscribe/service/SubscribePlugin.java

Rule Violation Ligne
AvoidUsingVolatile Use of modifier volatile is not recommended. 47

Priority 3

fr/paris/lutece/plugins/subscribe/business/Subscription.java

Rule Violation Ligne
AvoidDuplicateLiterals The String literal "#i18n{portal.validation.message.notEmpty}" appears 4 times in this file; the first occurrence is on line 53 53

fr/paris/lutece/plugins/subscribe/business/SubscriptionDAO.java

Rule Violation Ligne
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 97114
NPathComplexity The method 'findByFilter(SubscriptionFilter)' has an NPath complexity of 1782 215304
CyclomaticComplexity The method 'findByFilter(SubscriptionFilter)' has a cyclomatic complexity of 13. 215304

fr/paris/lutece/plugins/subscribe/service/ISubscriptionProviderService.java

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

fr/paris/lutece/plugins/subscribe/service/SubscribePlugin.java

Rule Violation Ligne
NonThreadSafeSingleton Singleton is not thread safe 5662

fr/paris/lutece/plugins/subscribe/service/SubscriptionService.java

Rule Violation Ligne
ImmutableField Private field '_dao' could be made final; it is only initialized in the declaration or constructor. 57

fr/paris/lutece/plugins/subscribe/web/SubscribeApp.java

Rule Violation Ligne
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 34253
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 149
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 175

Priority 4

fr/paris/lutece/plugins/subscribe/business/SubscriptionDTO.java

Rule Violation Ligne
BooleanGetMethodName A getX() method which returns a boolean should be named isX() 9396

Fichiers

fr/paris/lutece/plugins/subscribe/business/Subscription.java

Rule Violation Priority Ligne
AvoidDuplicateLiterals The String literal "#i18n{portal.validation.message.notEmpty}" appears 4 times in this file; the first occurrence is on line 53 3 53

fr/paris/lutece/plugins/subscribe/business/SubscriptionDAO.java

Rule Violation Priority Ligne
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 3 97114
NPathComplexity The method 'findByFilter(SubscriptionFilter)' has an NPath complexity of 1782 3 215304
CyclomaticComplexity The method 'findByFilter(SubscriptionFilter)' has a cyclomatic complexity of 13. 3 215304
AvoidBranchingStatementAsLastInLoop Avoid using a branching statement as the last in a loop. 2 315

fr/paris/lutece/plugins/subscribe/business/SubscriptionDTO.java

Rule Violation Priority Ligne
BooleanGetMethodName A getX() method which returns a boolean should be named isX() 4 9396

fr/paris/lutece/plugins/subscribe/service/ISubscriptionProviderService.java

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

fr/paris/lutece/plugins/subscribe/service/SubscribePlugin.java

Rule Violation Priority Ligne
AvoidUsingVolatile Use of modifier volatile is not recommended. 2 47
NonThreadSafeSingleton Singleton is not thread safe 3 5662

fr/paris/lutece/plugins/subscribe/service/SubscriptionService.java

Rule Violation Priority Ligne
ImmutableField Private field '_dao' could be made final; it is only initialized in the declaration or constructor. 3 57

fr/paris/lutece/plugins/subscribe/web/SubscribeApp.java

Rule Violation Priority Ligne
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 3 34253
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 149
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 175