PMD Results

The following document contains the results of PMD.

Summary

FilesErrors
515

Files

FilesViolations
org/apache/commons/jelly/tags/betwixt/IntrospectorTag.java 1
org/apache/commons/jelly/tags/betwixt/ParseTag.java 4
org/apache/commons/digester/rss/Channel.java 6
org/apache/commons/digester/rss/RSSDigester.java 3
org/apache/commons/jelly/betwixt/TestJelly.java 1

org/apache/commons/jelly/tags/betwixt/IntrospectorTag.java

ViolationLine
Avoid unused private fields such as 'log' 44

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

ViolationLine
Avoid unused private fields such as 'log' 47
Private field could be made final. It is only initialized in the declaration or constructor. 50
Avoid if (x != y) ..; else ..; 89
Avoid if (x != y) ..; else ..; 101

org/apache/commons/digester/rss/Channel.java

ViolationLine
The class 'Channel' has a Cyclomatic Complexity of 1 (Highest = 16). 35
Too many fields 35
An empty statement (semicolon) not part of a loop 402
Avoid really long methods. 485
The method 'render' has a Cyclomatic Complexity of 16. 485
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. 615

org/apache/commons/digester/rss/RSSDigester.java

ViolationLine
The String literal "setDescription" appears 4 times in this file; the first occurrence is on line 241 241
The String literal "setLink" appears 4 times in this file; the first occurrence is on line 245 245
The String literal "setTitle" appears 4 times in this file; the first occurrence is on line 251 251

org/apache/commons/jelly/betwixt/TestJelly.java

ViolationLine
All methods are static. Consider using Singleton instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. 38