About Us Home Contributors License Components Sandbox Jelly Overview FAQ Getting Started Tutorial Detail JellyDoc Download Tag Reference Community IRC Powered By To Do List Features JellyUnit JellySwing JellySWT XML Pipeline Core Tags Tag Libraries Change Log Changes Checkstyle Developer Activity File Activity JavaDocs JavaDoc Report JavaDoc Warnings Report JCoverage Metrics Unit Tests Source Xref Test Xref Project License PMD Report Task List General Information Charter Volunteering Contributing Patches Releasing Components Jakarta Community Get Involved Mailing Lists Commons Resources (Unofficial) Wiki Japanese Translation Korean Translation Related DB Commons XML Commons | ### PMD Results The following document contains the results of PMD. ### Summary
Files | Errors
--- | ---
30 | 76
### Files
Files | Violations
--- | ---
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
Violation | Line
--- | ---
Avoid empty catch blocks | 126
#### org/apache/commons/jelly/JellyContext.java
Violation | Line
--- | ---
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
Violation | Line
--- | ---
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
Violation | Line
--- | ---
Avoid if (x != y) ..; else ..; | 793
#### org/apache/commons/jelly/expression/CompositeExpression.java
Violation | Line
--- | ---
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
Violation | Line
--- | ---
Avoid calls to overridable methods during construction | 60
#### org/apache/commons/jelly/impl/CompositeTextScriptBlock.java
Violation | Line
--- | ---
Deeply nested if..then statements are hard to read | 56
#### org/apache/commons/jelly/impl/DynamicTagLibrary.java
Violation | Line
--- | ---
Avoid if (x != y) ..; else ..; | 119
#### org/apache/commons/jelly/impl/StaticTag.java
Violation | Line
--- | ---
Avoid reassigning parameters such as 'localName' | 117
#### org/apache/commons/jelly/impl/TagScript.java
Violation | Line
--- | ---
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
Violation | Line
--- | ---
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
Violation | Line
--- | ---
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
Violation | Line
--- | ---
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
Violation | Line
--- | ---
A value of 31 may denote a high amount of coupling within the class | 73
#### org/apache/commons/jelly/tags/core/FileTag.java
Violation | Line
--- | ---
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
Violation | Line
--- | ---
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
Violation | Line
--- | ---
Avoid if (x != y) ..; else ..; | 40
#### org/apache/commons/jelly/tags/core/ImportTag.java
Violation | Line
--- | ---
Avoid if (x != y) ..; else ..; | 77
#### org/apache/commons/jelly/tags/core/IncludeTag.java
Violation | Line
--- | ---
Avoid if (x != y) ..; else ..; | 99
#### org/apache/commons/jelly/tags/core/InvokeStaticTag.java
Violation | Line
--- | ---
Avoid if (x != y) ..; else ..; | 145
#### org/apache/commons/jelly/tags/core/InvokeTag.java
Violation | Line
--- | ---
Avoid if (x != y) ..; else ..; | 106
#### org/apache/commons/jelly/tags/core/NewTag.java
Violation | Line
--- | ---
Avoid if (x != y) ..; else ..; | 88
#### org/apache/commons/jelly/tags/core/ParseTag.java
Violation | Line
--- | ---
Avoid if (x != y) ..; else ..; | 74
Avoid if (x != y) ..; else ..; | 82
#### org/apache/commons/jelly/tags/core/RemoveTag.java
Violation | Line
--- | ---
Avoid if (x != y) ..; else ..; | 39
#### org/apache/commons/jelly/tags/core/SetTag.java
Violation | Line
--- | ---
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
Violation | Line
--- | ---
Avoid if (x != y) ..; else ..; | 223
#### org/apache/commons/jelly/tags/core/WhileTag.java
Violation | Line
--- | ---
Avoid if (x != y) ..; else ..; | 56
#### org/apache/commons/jelly/util/ClassLoaderUtils.java
Violation | Line
--- | ---
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
Violation | Line
--- | ---
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
Violation | Line
--- | ---
Avoid if (x != y) ..; else ..; | 84
Avoid reassigning parameters such as 'value' | 100