PMD Results

The following document contains the results of PMD.

Summary

FilesErrors
1582

Files

FilesViolations
org/devyant/decorutils/Utils.java 4
org/devyant/decorutils/Wrapper.java 2
org/devyant/decorutils/sdo/SDO.java 2
org/devyant/decorutils/tags/decorator/DecorateAndStoreTag.java 1
org/devyant/decorutils/tags/decorator/DecorateTag.java 9
org/devyant/decorutils/tags/decorator/IterateAndDecorateTag.java 8
org/devyant/decorutils/tags/xmldecorator/BaseXTag.java 4
org/devyant/decorutils/tags/xmldecorator/XAttributeTag.java 2
org/devyant/decorutils/tags/xmldecorator/XDecorateTag.java 9
org/devyant/decorutils/tags/xmldecorator/XDecorateTagExtraInfo.java 1
org/devyant/decorutils/tags/xmldecorator/XNestedTag.java 12
org/devyant/decorutils/tags/xmldecorator/XNestedTagExtraInfo.java 2
org/devyant/decorutils/tags/xmldecorator/XRuleTag.java 6
org/devyant/decorutils/xml/DynaNode.java 12
org/devyant/decorutils/xml/SimpleXmlWrapper.java 8

org/devyant/decorutils/Utils.java

ViolationLine
Avoid assigments in operands 95
Avoid assigments in operands 114
A method argument that is not assigned could be declared final 134
All methods are static. Consider using Singleton instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. 186

org/devyant/decorutils/Wrapper.java

ViolationLine
Abstract classes should be named 'AbstractXXX' 23
Found non-transient, non-static member. Please mark as transient or provide accessors. 27

org/devyant/decorutils/sdo/SDO.java

ViolationLine
Avoid variables with short names like o 131
Avoid variables with short names like s 145

org/devyant/decorutils/tags/decorator/DecorateAndStoreTag.java

ViolationLine
A signature (constructor or method) shouldn't have Exception in throws declaration 45

org/devyant/decorutils/tags/decorator/DecorateTag.java

ViolationLine
A method argument that is not assigned could be declared final 144
A method argument that is not assigned could be declared final 144
Avoid variables with short names like i 227
A signature (constructor or method) shouldn't have Exception in throws declaration 249
Avoid variables with short names like r 249
A method argument that is not assigned could be declared final 262
A method argument that is not assigned could be declared final 262
A method argument that is not assigned could be declared final 262
A method argument that is not assigned could be declared final 262

org/devyant/decorutils/tags/decorator/IterateAndDecorateTag.java

ViolationLine
Found non-transient, non-static member. Please mark as transient or provide accessors. 45
Found non-transient, non-static member. Please mark as transient or provide accessors. 49
Found non-transient, non-static member. Please mark as transient or provide accessors. 53
Found non-transient, non-static member. Please mark as transient or provide accessors. 57
Avoid variables with short names like r 62
A method argument that is not assigned could be declared final 85
A method argument that is not assigned could be declared final 97
A method argument that is not assigned could be declared final 121

org/devyant/decorutils/tags/xmldecorator/BaseXTag.java

ViolationLine
A method argument that is not assigned could be declared final 49
A method argument that is not assigned could be declared final 62
A method argument that is not assigned could be declared final 72
A method argument that is not assigned could be declared final 82

org/devyant/decorutils/tags/xmldecorator/XAttributeTag.java

ViolationLine
Found non-transient, non-static member. Please mark as transient or provide accessors. 36
A method argument that is not assigned could be declared final 41

org/devyant/decorutils/tags/xmldecorator/XDecorateTag.java

ViolationLine
Found non-transient, non-static member. Please mark as transient or provide accessors. 42
Found non-transient, non-static member. Please mark as transient or provide accessors. 46
Found non-transient, non-static member. Please mark as transient or provide accessors. 50
A method argument that is not assigned could be declared final 70
Local variable could be declared final 103
Local variable could be declared final 108
A method argument that is not assigned could be declared final 123
Local variable could be declared final 124
Local variable could be declared final 143

org/devyant/decorutils/tags/xmldecorator/XDecorateTagExtraInfo.java

ViolationLine
Avoid instantiating new objects inside loops 43

org/devyant/decorutils/tags/xmldecorator/XNestedTag.java

ViolationLine
Found non-transient, non-static member. Please mark as transient or provide accessors. 45
A method argument that is not assigned could be declared final 67
A method argument that is not assigned could be declared final 74
Local variable could be declared final 75
Avoid catching NullPointerException; consider removing the cause of the NPE. 89
A method argument that is not assigned could be declared final 103
Local variable could be declared final 106
Local variable could be declared final 111
Local variable could be declared final 112
A catch statement should never catch throwable since it includes errors 121
A method argument that is not assigned could be declared final 130
Local variable could be declared final 150

org/devyant/decorutils/tags/xmldecorator/XNestedTagExtraInfo.java

ViolationLine
Avoid if (x != y) ..; else ..; 35
Avoid instantiating new objects inside loops 46

org/devyant/decorutils/tags/xmldecorator/XRuleTag.java

ViolationLine
Found non-transient, non-static member. Please mark as transient or provide accessors. 46
A method argument that is not assigned could be declared final 58
Local variable could be declared final 59
Local variable could be declared final 72
A method argument that is not assigned could be declared final 107
Local variable could be declared final 108

org/devyant/decorutils/xml/DynaNode.java

ViolationLine
A method argument that is not assigned could be declared final 71
A method argument that is not assigned could be declared final 85
A method argument that is not assigned could be declared final 99
Local variable could be declared final 120
Local variable could be declared final 121
Avoid instantiating new objects inside loops 136
Local variable could be declared final 140
Local variable could be declared final 142
Local variable could be declared final 144
A method argument that is not assigned could be declared final 165
A method argument that is not assigned could be declared final 165
A method argument that is not assigned could be declared final 165

org/devyant/decorutils/xml/SimpleXmlWrapper.java

ViolationLine
Found non-transient, non-static member. Please mark as transient or provide accessors. 46
Private field could be made final. It is only initialized in the declaration or constructor. 46
Local variable could be declared final 76
Local variable could be declared final 90
Local variable could be declared final 104
Local variable could be declared final 109
Local variable could be declared final 135
Local variable could be declared final 140