Fork me on GitHub

Résultats PMD

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

Fichiers

fr/paris/lutece/plugins/module/announce/subscribe/business/AnnounceSubscribtionDAO.java

Violation Ligne
The class 'AnnounceSubscribtionDAO' has a Cyclomatic Complexity of 4 (Highest = 13). 12318
The class 'AnnounceSubscribtionDAO' has a Modified Cyclomatic Complexity of 4 (Highest = 13). 12318
The class 'AnnounceSubscribtionDAO' has a Standard Cyclomatic Complexity of 4 (Highest = 13). 12318
Avoid unused private fields such as 'SQL_QUERY_SELECT_DISTINCT'. 17
Avoid unused private fields such as 'CONSTANT_COMMA'. 34
The method 'findByFilter' has a Cyclomatic Complexity of 13. 163249
The method 'findByFilter' has a Standard Cyclomatic Complexity of 13. 163249
The method 'findByFilter' has a Modified Cyclomatic Complexity of 13. 163249
The method findByFilter() has an NPath complexity of 1728 163249
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 285
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 286

fr/paris/lutece/plugins/module/announce/subscribe/business/AnnounceSubscribtionDTO.java

Violation Ligne
Use explicit scoping instead of the default package private level 7
Document empty constructor 910
It is a good practice to call super() in a constructor 910

fr/paris/lutece/plugins/module/announce/subscribe/service/AnnounceSubscribePlugin.java

Violation Ligne
Avoid unused imports such as 'fr.paris.lutece.plugins.subscribe.service.SubscribePlugin' 36
Avoid unused imports such as 'fr.paris.lutece.plugins.subscribe.service.SubscribePlugin' 36
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. 4666
Use of modifier volatile is not recommended. 49
Singleton is not thread safe 5763

fr/paris/lutece/plugins/module/announce/subscribe/service/AnnounceSubscriptionService.java

Violation Ligne
A class which only has private constructors should be final 18121
Private field '_dao' could be made final; it is only initialized in the declaration or constructor. 21