Résultats PMD
Le document suivant contient les résultats de PMD 6.13.0.
Violations By Priority
Priority 1
fr/paris/lutece/plugins/blog/modules/extendcomment/utils/BlogCommentConstants.java
Rule |
Violation |
Ligne |
ClassNamingConventions |
The utility class name 'BlogCommentConstants' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' |
41–61 |
Priority 2
fr/paris/lutece/plugins/blog/modules/extendcomment/web/BlogExtendCommentJspBean.java
Rule |
Violation |
Ligne |
BooleanInstantiation |
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. |
199 |
BooleanInstantiation |
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. |
203 |
Priority 3
fr/paris/lutece/plugins/blog/modules/extendcomment/web/AbstractManageBlogExtendcommentJspBean.java
fr/paris/lutece/plugins/blog/modules/extendcomment/web/BlogExtendCommentJspBean.java
Fichiers
fr/paris/lutece/plugins/blog/modules/extendcomment/utils/BlogCommentConstants.java
Rule |
Violation |
Priority |
Ligne |
ClassNamingConventions |
The utility class name 'BlogCommentConstants' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' |
1 |
41–61 |
fr/paris/lutece/plugins/blog/modules/extendcomment/web/AbstractManageBlogExtendcommentJspBean.java
fr/paris/lutece/plugins/blog/modules/extendcomment/web/BlogExtendCommentJspBean.java
Rule |
Violation |
Priority |
Ligne |
ExcessiveImports |
A high number of imports can indicate a high degree of coupling within an object. |
3 |
34–281 |
MissingSerialVersionUID |
Classes implementing Serializable should set a serialVersionUID |
3 |
86–281 |
ImmutableField |
Private field '_extenderService' could be made final; it is only initialized in the declaration or constructor. |
3 |
100 |
ImmutableField |
Private field '_commentService' could be made final; it is only initialized in the declaration or constructor. |
3 |
102 |
ImmutableField |
Private field '_configService' could be made final; it is only initialized in the declaration or constructor. |
3 |
103 |
ImmutableField |
Private field '_resourceExtenderService' could be made final; it is only initialized in the declaration or constructor. |
3 |
104 |
NPathComplexity |
The method 'getCommentBlog(HttpServletRequest)' has an NPath complexity of 46080 |
3 |
114–279 |
ExcessiveMethodLength |
Avoid really long methods. |
3 |
114–279 |
CyclomaticComplexity |
The method 'getCommentBlog(HttpServletRequest)' has a cyclomatic complexity of 20. |
3 |
114–279 |
NcssCount |
The method 'getCommentBlog(HttpServletRequest)' has a NCSS line count of 93. |
3 |
114–279 |
PrematureDeclaration |
Avoid declaring a variable if it is unreferenced before a possible exit point. |
3 |
120 |
PrematureDeclaration |
Avoid declaring a variable if it is unreferenced before a possible exit point. |
3 |
121 |
ControlStatementBraces |
This statement should have braces |
3 |
136 |
BooleanInstantiation |
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. |
2 |
199 |
BooleanInstantiation |
Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. |
2 |
203 |
UseDiamondOperator |
Explicit type arguments can be replaced by Diamond Operator |
3 |
220–221 |
UseDiamondOperator |
Explicit type arguments can be replaced by Diamond Operator |
3 |
223 |
UseConcurrentHashMap |
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
3 |
223 |
UnnecessaryLocalBeforeReturn |
Consider simply returning the value vs storing it in local variable 'strContent' |
3 |
277 |