PMD Results

The following document contains the results of PMD.

Summary

FilesErrors
3076

Files

FilesViolations
org/apache/commons/jelly/Jelly.java 1
org/apache/commons/jelly/JellyContext.java 7
org/apache/commons/jelly/TagSupport.java 3
org/apache/commons/jelly/XMLOutput.java 1
org/apache/commons/jelly/expression/CompositeExpression.java 7
org/apache/commons/jelly/expression/xpath/XPathExpression.java 1
org/apache/commons/jelly/impl/CompositeTextScriptBlock.java 1
org/apache/commons/jelly/impl/DynamicTagLibrary.java 1
org/apache/commons/jelly/impl/StaticTag.java 1
org/apache/commons/jelly/impl/TagScript.java 3
org/apache/commons/jelly/parser/XMLParser.java 15
org/apache/commons/jelly/tags/Resources.java 1
org/apache/commons/jelly/tags/core/ArgTag.java 4
org/apache/commons/jelly/tags/core/CoreTagLibrary.java 1
org/apache/commons/jelly/tags/core/FileTag.java 4
org/apache/commons/jelly/tags/core/ForEachTag.java 5
org/apache/commons/jelly/tags/core/IfTag.java 1
org/apache/commons/jelly/tags/core/ImportTag.java 1
org/apache/commons/jelly/tags/core/IncludeTag.java 1
org/apache/commons/jelly/tags/core/InvokeStaticTag.java 1
org/apache/commons/jelly/tags/core/InvokeTag.java 1
org/apache/commons/jelly/tags/core/NewTag.java 1
org/apache/commons/jelly/tags/core/ParseTag.java 2
org/apache/commons/jelly/tags/core/RemoveTag.java 1
org/apache/commons/jelly/tags/core/SetTag.java 4
org/apache/commons/jelly/tags/core/UseBeanTag.java 1
org/apache/commons/jelly/tags/core/WhileTag.java 1
org/apache/commons/jelly/util/ClassLoaderUtils.java 1
org/apache/commons/jelly/util/CommandLineParser.java 2
org/apache/commons/jelly/xpath/XPathComparator.java 2

org/apache/commons/jelly/Jelly.java

ViolationLine
Avoid empty catch blocks 126

org/apache/commons/jelly/JellyContext.java

ViolationLine
Avoid unused imports such as 'Collections' 23
Avoid unused imports such as 'HashMap' 24
Avoid unused imports such as 'WeakHashMap' 28
Avoid if (x != y) ..; else ..; 430
Avoid if (x != y) ..; else ..; 935
A high number of public methods and attributes in an object can indicate the class may need to be broken up for exhaustive testing may prove difficult. 979
A value of 21 may denote a high amount of coupling within the class 979

org/apache/commons/jelly/TagSupport.java

ViolationLine
Avoid reassigning parameters such as 'from' 57
Avoid reassigning parameters such as 'from' 78
Avoid calls to overridable methods during construction 108

org/apache/commons/jelly/XMLOutput.java

ViolationLine
Avoid if (x != y) ..; else ..; 793

org/apache/commons/jelly/expression/CompositeExpression.java

ViolationLine
The class 'CompositeExpression' has a Cyclomatic Complexity of 4 (Highest = 27). 35
Avoid really long methods. 66
The method 'parse' has a Cyclomatic Complexity of 27. 66
A high ratio of statements to labels in a switch statement. Consider refactoring. 97
A high ratio of statements to labels in a switch statement. Consider refactoring. 103
A non-case label was present in a switch statement 123
A non-case label was present in a switch statement 146

org/apache/commons/jelly/expression/xpath/XPathExpression.java

ViolationLine
Avoid calls to overridable methods during construction 60

org/apache/commons/jelly/impl/CompositeTextScriptBlock.java

ViolationLine
Deeply nested if..then statements are hard to read 56

org/apache/commons/jelly/impl/DynamicTagLibrary.java

ViolationLine
Avoid if (x != y) ..; else ..; 119

org/apache/commons/jelly/impl/StaticTag.java

ViolationLine
Avoid reassigning parameters such as 'localName' 117

org/apache/commons/jelly/impl/TagScript.java

ViolationLine
Avoid reassigning parameters such as 'tagNamespacesMap' 148
Avoid if (x != y) ..; else ..; 460
A value of 29 may denote a high amount of coupling within the class 692

org/apache/commons/jelly/parser/XMLParser.java

ViolationLine
Avoid really long Classes. 74
Avoid reassigning parameters such as 'namespaceURI' 579
Avoid if (x != y) ..; else ..; 599
Avoid if (x != y) ..; else ..; 675
An empty statement (semicolon) not part of a loop 761
An empty statement (semicolon) not part of a loop 774
An empty statement (semicolon) not part of a loop 794
Avoid if (x != y) ..; else ..; 847
Avoid if (x != y) ..; else ..; 871
Avoid unused local variables such as 'tag' 1051
Avoid if (x != y) ..; else ..; 1159
Avoid if (x != y) ..; else ..; 1179
Avoid if (x != y) ..; else ..; 1187
A high number of imports can indicate a high degree of coupling within an object. 1213
A value of 44 may denote a high amount of coupling within the class 1213

org/apache/commons/jelly/tags/Resources.java

ViolationLine
All methods are static. Consider using Singleton instead. Alternatively, you could add a private constructor to silence this warning. 125

org/apache/commons/jelly/tags/core/ArgTag.java

ViolationLine
The class 'ArgTag' has a Cyclomatic Complexity of 3 (Highest = 14). 42
The method 'doTag' has a Cyclomatic Complexity of 14. 71
Avoid if (x != y) ..; else ..; 99
Avoid if (x != y) ..; else ..; 162

org/apache/commons/jelly/tags/core/CoreTagLibrary.java

ViolationLine
A value of 31 may denote a high amount of coupling within the class 73

org/apache/commons/jelly/tags/core/FileTag.java

ViolationLine
Avoid if (x != y) ..; else ..; 57
Avoid if (x != y) ..; else ..; 58
Avoid if (x != y) ..; else ..; 62
Avoid empty catch blocks 166

org/apache/commons/jelly/tags/core/ForEachTag.java

ViolationLine
The class 'ForEachTag' has a Cyclomatic Complexity of 3 (Highest = 21). 40
Avoid really long methods. 81
The method 'doTag' has a Cyclomatic Complexity of 21. 81
Avoid if (x != y) ..; else ..; 88
Avoid if (x != y) ..; else ..; 139

org/apache/commons/jelly/tags/core/IfTag.java

ViolationLine
Avoid if (x != y) ..; else ..; 40

org/apache/commons/jelly/tags/core/ImportTag.java

ViolationLine
Avoid if (x != y) ..; else ..; 77

org/apache/commons/jelly/tags/core/IncludeTag.java

ViolationLine
Avoid if (x != y) ..; else ..; 99

org/apache/commons/jelly/tags/core/InvokeStaticTag.java

ViolationLine
Avoid if (x != y) ..; else ..; 145

org/apache/commons/jelly/tags/core/InvokeTag.java

ViolationLine
Avoid if (x != y) ..; else ..; 106

org/apache/commons/jelly/tags/core/NewTag.java

ViolationLine
Avoid if (x != y) ..; else ..; 88

org/apache/commons/jelly/tags/core/ParseTag.java

ViolationLine
Avoid if (x != y) ..; else ..; 74
Avoid if (x != y) ..; else ..; 82

org/apache/commons/jelly/tags/core/RemoveTag.java

ViolationLine
Avoid if (x != y) ..; else ..; 39

org/apache/commons/jelly/tags/core/SetTag.java

ViolationLine
Avoid if (x != y) ..; else ..; 68
Avoid if (x != y) ..; else ..; 83
Avoid if (x != y) ..; else ..; 93
Avoid if (x != y) ..; else ..; 94

org/apache/commons/jelly/tags/core/UseBeanTag.java

ViolationLine
Avoid if (x != y) ..; else ..; 223

org/apache/commons/jelly/tags/core/WhileTag.java

ViolationLine
Avoid if (x != y) ..; else ..; 56

org/apache/commons/jelly/util/ClassLoaderUtils.java

ViolationLine
All methods are static. Consider using Singleton instead. Alternatively, you could add a private constructor to silence this warning. 128

org/apache/commons/jelly/util/CommandLineParser.java

ViolationLine
The same String literal appears 4 times in this file; the first occurrence is on line 89 89
Avoid empty catch blocks 141

org/apache/commons/jelly/xpath/XPathComparator.java

ViolationLine
Avoid if (x != y) ..; else ..; 84
Avoid reassigning parameters such as 'value' 100