Fork me on GitHub

Résultats PMD

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

Fichiers

fr/paris/lutece/plugins/quiz/modules/games/business/QuizParticipant.java

Violation Ligne
Avoid unnecessary constructors - the compiler will generate these for you 5456
Document empty constructor 5456
A getX() method which returns a boolean should be named isX() 153156

fr/paris/lutece/plugins/quiz/modules/games/business/QuizParticipantDAO.java

Violation Ligne
Avoid excessively long variable names like SQL_QUERY_SELECT_PARTICIPANTS_BY_IDENTIFIANT_AND_QUIZ_ID 51
Use explicit scoping instead of the default package private level 6077

fr/paris/lutece/plugins/quiz/modules/games/business/portlet/QuizPortlet.java

Violation Ligne
It is a good practice to call super() in a constructor 6770

fr/paris/lutece/plugins/quiz/modules/games/business/portlet/QuizPortletDAO.java

Violation Ligne
Avoid variables with short names like p 61
Avoid variables with short names like p 91

fr/paris/lutece/plugins/quiz/modules/games/business/portlet/QuizPortletHome.java

Violation Ligne
Avoid using redundant field initializer for '_singleton' 52
Possible unsafe assignment to a non-final static field in a constructor. 52
It is a good practice to call super() in a constructor 5763
Singleton is not thread safe 8588

fr/paris/lutece/plugins/quiz/modules/games/web/RandApp.java

Violation Ligne
Avoid unused private fields such as 'TEMPLATE_QUESTIONS_LIST'. 68
Avoid unused private fields such as 'PROPERTY_QUIZ_PAGE_PATH'. 74
Avoid unused private fields such as 'PROPERTY_QUIZ_PAGE_TITLE'. 75
Avoid unused private fields such as 'PARAMETER_RESULTS'. 78
Avoid unused private fields such as 'PARAMETER_ACTION'. 79
Private field '_randService' could be made final; it is only initialized in the declaration or constructor. 88
Avoid declaring a variable if it is unreferenced before a possible exit point. 126
Avoid declaring a variable if it is unreferenced before a possible exit point. 140
New exception is thrown in catch block, original stack trace may be lost 200
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 213

fr/paris/lutece/plugins/quiz/modules/games/web/RandJspBean.java

Violation Ligne
Avoid declaring a variable if it is unreferenced before a possible exit point. 217
Avoid declaring a variable if it is unreferenced before a possible exit point. 218
Avoid declaring a variable if it is unreferenced before a possible exit point. 248
Use opposite operator instead of the logic complement operator. 258
Avoid declaring a variable if it is unreferenced before a possible exit point. 365
Avoid declaring a variable if it is unreferenced before a possible exit point. 367
Avoid declaring a variable if it is unreferenced before a possible exit point. 425
Avoid declaring a variable if it is unreferenced before a possible exit point. 426
Avoid declaring a variable if it is unreferenced before a possible exit point. 428

fr/paris/lutece/plugins/quiz/modules/games/web/portlet/QuizPortletJspBean.java

Violation Ligne
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 90
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 113