Fork me on GitHub

Résultats PMD

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

Violations By Priority

Priority 1

fr/paris/lutece/plugins/poll/business/PollData.java

Rule Violation Ligne
FormalParameterNamingConventions The method parameter name '_nSize' doesn't match '[a-z][a-zA-Z0-9]*' 48
FormalParameterNamingConventions The method parameter name '_labelName' doesn't match '[a-z][a-zA-Z0-9]*' 48
FormalParameterNamingConventions The method parameter name '_labelName' doesn't match '[a-z][a-zA-Z0-9]*' 59
FormalParameterNamingConventions The method parameter name '_nSize' doesn't match '[a-z][a-zA-Z0-9]*' 69

fr/paris/lutece/plugins/poll/business/PollVisualization.java

Rule Violation Ligne
FormalParameterNamingConventions The method parameter name '_nId' doesn't match '[a-z][a-zA-Z0-9]*' 49
FormalParameterNamingConventions The method parameter name '_strTitle' doesn't match '[a-z][a-zA-Z0-9]*' 49
FormalParameterNamingConventions The method parameter name '_nId' doesn't match '[a-z][a-zA-Z0-9]*' 60
FormalParameterNamingConventions The method parameter name '_strTitle' doesn't match '[a-z][a-zA-Z0-9]*' 70
FormalParameterNamingConventions The method parameter name '_type' doesn't match '[a-z][a-zA-Z0-9]*' 80

Priority 3

fr/paris/lutece/plugins/poll/business/PollData.java

Rule Violation Ligne
CommentRequired Header comments are required 36–74
UncommentedEmptyConstructor Document empty constructor 44–46
CommentRequired Public method and constructor comments are required 44–46
CommentRequired Public method and constructor comments are required 48–52
CommentRequired Public method and constructor comments are required 64–67
CommentRequired Public method and constructor comments are required 69–72

fr/paris/lutece/plugins/poll/business/PollFormQuestion.java

Rule Violation Ligne
CommentRequired Public method and constructor comments are required 60–64

fr/paris/lutece/plugins/poll/business/PollFormQuestionDAO.java

Rule Violation Ligne
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 235
MissingOverride The method 'loadByQuestionId(int, int, Plugin)' is missing an @Override annotation. 261–283

fr/paris/lutece/plugins/poll/business/PollVisualization.java

Rule Violation Ligne
CommentRequired Header comments are required 36–95
UncommentedEmptyConstructor Document empty constructor 44–47
CommentRequired Public method and constructor comments are required 44–47
CommentRequired Public method and constructor comments are required 49–53
CommentRequired Public method and constructor comments are required 55–58
CommentRequired Public method and constructor comments are required 60–63
CommentRequired Public method and constructor comments are required 65–68
CommentRequired Public method and constructor comments are required 70–73
CommentRequired Public method and constructor comments are required 75–78
CommentRequired Public method and constructor comments are required 80–83
CommentRequired Public method and constructor comments are required 85–88
CommentRequired Public method and constructor comments are required 90–93

fr/paris/lutece/plugins/poll/service/PollFormService.java

Rule Violation Ligne
CommentRequired Header comments are required 56–116
UseUtilityClass 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. 57–116
CommentRequired Public method and constructor comments are required 59–114
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 62
SimplifyBooleanExpressions Avoid unnecessary comparisons in boolean expressions 71
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 87

fr/paris/lutece/plugins/poll/web/PollFormJspBean.java

Rule Violation Ligne
UnusedPrivateField Avoid unused private fields such as 'INFO_POLLFORM_QUESTION_REMOVED'. 134
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 149
ConfusingTernary Avoid if (x != y) ..; else ..; 165
ConfusingTernary Avoid if (x != y) ..; else ..; 330–337
LinguisticNaming Linguistics Antipattern - The variable 'isToolBox' indicates linguistically it is a boolean, but it is 'String' 387
ConfusingTernary Avoid if (x != y) ..; else ..; 391–398

Priority 4

fr/paris/lutece/plugins/poll/business/PollForm.java

Rule Violation Ligne
BooleanGetMethodName A getX() method which returns a boolean should be named isX() 65–68
BooleanGetMethodName A getX() method which returns a boolean should be named isX() 191–194

fr/paris/lutece/plugins/poll/business/PollFormQuestion.java

Rule Violation Ligne
BooleanGetMethodName A getX() method which returns a boolean should be named isX() 134–137
BooleanGetMethodName A getX() method which returns a boolean should be named isX() 155–158

Fichiers

fr/paris/lutece/plugins/poll/business/PollData.java

Rule Violation Priority Ligne
CommentRequired Header comments are required 3 36–74
UncommentedEmptyConstructor Document empty constructor 3 44–46
CommentRequired Public method and constructor comments are required 3 44–46
FormalParameterNamingConventions The method parameter name '_nSize' doesn't match '[a-z][a-zA-Z0-9]*' 1 48
CommentRequired Public method and constructor comments are required 3 48–52
FormalParameterNamingConventions The method parameter name '_labelName' doesn't match '[a-z][a-zA-Z0-9]*' 1 48
FormalParameterNamingConventions The method parameter name '_labelName' doesn't match '[a-z][a-zA-Z0-9]*' 1 59
CommentRequired Public method and constructor comments are required 3 64–67
CommentRequired Public method and constructor comments are required 3 69–72
FormalParameterNamingConventions The method parameter name '_nSize' doesn't match '[a-z][a-zA-Z0-9]*' 1 69

fr/paris/lutece/plugins/poll/business/PollForm.java

Rule Violation Priority Ligne
BooleanGetMethodName A getX() method which returns a boolean should be named isX() 4 65–68
BooleanGetMethodName A getX() method which returns a boolean should be named isX() 4 191–194

fr/paris/lutece/plugins/poll/business/PollFormQuestion.java

Rule Violation Priority Ligne
CommentRequired Public method and constructor comments are required 3 60–64
BooleanGetMethodName A getX() method which returns a boolean should be named isX() 4 134–137
BooleanGetMethodName A getX() method which returns a boolean should be named isX() 4 155–158

fr/paris/lutece/plugins/poll/business/PollFormQuestionDAO.java

Rule Violation Priority Ligne
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 235
MissingOverride The method 'loadByQuestionId(int, int, Plugin)' is missing an @Override annotation. 3 261–283

fr/paris/lutece/plugins/poll/business/PollVisualization.java

Rule Violation Priority Ligne
CommentRequired Header comments are required 3 36–95
UncommentedEmptyConstructor Document empty constructor 3 44–47
CommentRequired Public method and constructor comments are required 3 44–47
CommentRequired Public method and constructor comments are required 3 49–53
FormalParameterNamingConventions The method parameter name '_nId' doesn't match '[a-z][a-zA-Z0-9]*' 1 49
FormalParameterNamingConventions The method parameter name '_strTitle' doesn't match '[a-z][a-zA-Z0-9]*' 1 49
CommentRequired Public method and constructor comments are required 3 55–58
FormalParameterNamingConventions The method parameter name '_nId' doesn't match '[a-z][a-zA-Z0-9]*' 1 60
CommentRequired Public method and constructor comments are required 3 60–63
CommentRequired Public method and constructor comments are required 3 65–68
CommentRequired Public method and constructor comments are required 3 70–73
FormalParameterNamingConventions The method parameter name '_strTitle' doesn't match '[a-z][a-zA-Z0-9]*' 1 70
CommentRequired Public method and constructor comments are required 3 75–78
CommentRequired Public method and constructor comments are required 3 80–83
FormalParameterNamingConventions The method parameter name '_type' doesn't match '[a-z][a-zA-Z0-9]*' 1 80
CommentRequired Public method and constructor comments are required 3 85–88
CommentRequired Public method and constructor comments are required 3 90–93

fr/paris/lutece/plugins/poll/service/PollFormService.java

Rule Violation Priority Ligne
CommentRequired Header comments are required 3 56–116
UseUtilityClass 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. 3 57–116
CommentRequired Public method and constructor comments are required 3 59–114
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 62
SimplifyBooleanExpressions Avoid unnecessary comparisons in boolean expressions 3 71
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 87

fr/paris/lutece/plugins/poll/web/PollFormJspBean.java

Rule Violation Priority Ligne
UnusedPrivateField Avoid unused private fields such as 'INFO_POLLFORM_QUESTION_REMOVED'. 3 134
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 149
ConfusingTernary Avoid if (x != y) ..; else ..; 3 165
ConfusingTernary Avoid if (x != y) ..; else ..; 3 330–337
LinguisticNaming Linguistics Antipattern - The variable 'isToolBox' indicates linguistically it is a boolean, but it is 'String' 3 387
ConfusingTernary Avoid if (x != y) ..; else ..; 3 391–398