PMD Results

Last Published: Mon Jul 10 14:57:32 CEST 2006

Servletspy

Download

Project Documentation

About Servletspy
Project Info
Project Reports
- CPD Report
- Checkstyle
- JavaDocs
- Maven Surefire Report
- PMD Report
- Source Xref

Built by Maven

PMD Results

The following document contains the results of PMD

Files

net/sourceforge/servletspy/ctx/Context.java

Violation	Line
Found non-transient, non-static member. Please mark as transient or provide accessors.	63
Found non-transient, non-static member. Please mark as transient or provide accessors.	69
Found non-transient, non-static member. Please mark as transient or provide accessors.	75
Found non-transient, non-static member. Please mark as transient or provide accessors.	78

net/sourceforge/servletspy/util/ClassUtil.java

Violation	Line
Document empty constructor	32 - 33

net/sourceforge/servletspy/Spy.java

Violation	Line
Document empty constructor	46 - 47
Consider simply returning the value vs storing it in local variable 'node'	83

net/sourceforge/servletspy/About.java

Violation	Line
Private field could be made final. It is only initialized in the declaration or constructor.	43
Found non-transient, non-static member. Please mark as transient or provide accessors.	43

net/sourceforge/servletspy/IContextHandler.java

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

net/sourceforge/servletspy/config/ContextHandlerFactory.java

Violation	Line
Possible unsafe assignment to a non-final static field in a constructor.	38
Found non-transient, non-static member. Please mark as transient or provide accessors.	48
Avoid throwing null pointer exceptions	52
Avoid empty catch blocks	80 - 82

net/sourceforge/servletspy/config/Handler.java

Violation	Line
Found non-transient, non-static member. Please mark as transient or provide accessors.	29
Found non-transient, non-static member. Please mark as transient or provide accessors.	31
Document empty constructor	33 - 34

net/sourceforge/servletspy/config/Param.java

Violation	Line
Avoid protected fields in a final class. Change to private or package access.	29
Found non-transient, non-static member. Please mark as transient or provide accessors.	29
Avoid protected fields in a final class. Change to private or package access.	31
Found non-transient, non-static member. Please mark as transient or provide accessors.	31
Document empty constructor	33 - 34

net/sourceforge/servletspy/config/Loader.java

Violation	Line
Found non-transient, non-static member. Please mark as transient or provide accessors.	48
Use block level rather than method level synchronization	50 - 73
Avoid instantiation through private constructors from outside of the constructor's class.	59 - 60
Avoid instantiation through private constructors from outside of the constructor's class.	71
Use block level rather than method level synchronization	75 - 81

net/sourceforge/servletspy/config/ServletSpy.java

Violation	Line
Avoid protected fields in a final class. Change to private or package access.	38
Found non-transient, non-static member. Please mark as transient or provide accessors.	38
Avoid protected fields in a final class. Change to private or package access.	40
Found non-transient, non-static member. Please mark as transient or provide accessors.	40
Avoid throwing null pointer exceptions	45
Document empty constructor	67 - 68

net/sourceforge/servletspy/Node.java

Violation	Line
Found non-transient, non-static member. Please mark as transient or provide accessors.	41
Found non-transient, non-static member. Please mark as transient or provide accessors.	47
Found non-transient, non-static member. Please mark as transient or provide accessors.	50
Private field could be made final. It is only initialized in the declaration or constructor.	56
Found non-transient, non-static member. Please mark as transient or provide accessors.	56
Found non-transient, non-static member. Please mark as transient or provide accessors.	59
No need to check for null before an instanceof	105

net/sourceforge/servletspy/handler/std/DebugHandler.java

Violation	Line
Document empty method	43 - 44
Document empty method	49 - 50
A signature (constructor or method) shouldn't have Exception in throws declaration	55

net/sourceforge/servletspy/handler/std/CycleDetectorHandler.java

Violation	Line
Document empty method	42 - 43
Document empty method	50 - 51
Unnecessary final modifier in final class	58 - 60
A signature (constructor or method) shouldn't have Exception in throws declaration	58
A catch statement should never catch throwable since it includes errors	73

net/sourceforge/servletspy/handler/std/NullHandler.java

Violation	Line
Document empty method	35 - 36
Document empty method	41 - 42
A signature (constructor or method) shouldn't have Exception in throws declaration	47

net/sourceforge/servletspy/handler/std/PrimitivesHandler.java

Violation	Line
Document empty method	50 - 51
Document empty method	56 - 57
A signature (constructor or method) shouldn't have Exception in throws declaration	62

net/sourceforge/servletspy/handler/base/AbstractHandler.java

Violation	Line
Document empty method	37 - 38
A signature (constructor or method) shouldn't have Exception in throws declaration	45
Document empty method	54 - 55

net/sourceforge/servletspy/handler/base/AbstractInterfaceTransformer.java

Violation	Line
This abstract class does not have any abstract methods	41 - 170
This call to Collection.toArray() may be optimizable	63
Found non-transient, non-static member. Please mark as transient or provide accessors.	86
Document empty constructor	91 - 92
A signature (constructor or method) shouldn't have Exception in throws declaration	119
A signature (constructor or method) shouldn't have Exception in throws declaration	133

net/sourceforge/servletspy/handler/base/AbstractClassFilter.java

Violation	Line
Found non-transient, non-static member. Please mark as transient or provide accessors.	92
Deeply nested if..then statements are hard to read	117 - 119
Document empty method	128 - 129
A signature (constructor or method) shouldn't have Exception in throws declaration	134

net/sourceforge/servletspy/handler/servlet/SessionContextHandler.java

Violation	Line
Document empty method	59 - 60
A signature (constructor or method) shouldn't have Exception in throws declaration	83
Document empty method	111 - 112

net/sourceforge/servletspy/handler/servlet/PageContextHandler.java

Violation	Line
Document empty method	59 - 60
A signature (constructor or method) shouldn't have Exception in throws declaration	65
Document empty method	85 - 86

net/sourceforge/servletspy/handler/servlet/RequestHandler.java

Violation	Line
A catch statement should never catch throwable since it includes errors	113
Document empty method	188 - 189
A signature (constructor or method) shouldn't have Exception in throws declaration	195
Document empty method	231 - 232

net/sourceforge/servletspy/handler/servlet/ServletContextHandler.java

Violation	Line
A signature (constructor or method) shouldn't have Exception in throws declaration	54
Document empty method	109 - 110
Document empty method	115 - 116

net/sourceforge/servletspy/handler/container/ArrayTransformer.java

Violation	Line
Found non-transient, non-static member. Please mark as transient or provide accessors.	38
A signature (constructor or method) shouldn't have Exception in throws declaration	44
Document empty method	80 - 81
Avoid empty catch blocks	93 - 94

net/sourceforge/servletspy/handler/container/MapTransformer.java

Violation	Line
Found non-transient, non-static member. Please mark as transient or provide accessors.	41
A signature (constructor or method) shouldn't have Exception in throws declaration	55
Document empty method	97 - 98
Avoid empty catch blocks	111 - 112

net/sourceforge/servletspy/handler/container/DynamicProxyTransformer.java

Violation	Line
The class 'DynamicProxyTransformer' has a Cyclomatic Complexity of 5 (Highest = 10).	41 - 129
Found non-transient, non-static member. Please mark as transient or provide accessors.	44
Document empty constructor	49 - 50
Unnecessary final modifier in final class	55 - 107
The method 'handle' has a Cyclomatic Complexity of 10.	55 - 107
A signature (constructor or method) shouldn't have Exception in throws declaration	55
The user-supplied array 'params' is stored directly	114
The user-supplied array 'params' is stored directly	114
The user-supplied array 'params' is stored directly	114

net/sourceforge/servletspy/handler/container/ListTransformer.java

Violation	Line
Found non-transient, non-static member. Please mark as transient or provide accessors.	40
A signature (constructor or method) shouldn't have Exception in throws declaration	46
Avoid empty catch blocks	94 - 95
Document empty method	108 - 109

net/sourceforge/servletspy/handler/container/BeanTransformer.java

Violation	Line
A signature (constructor or method) shouldn't have Exception in throws declaration	39

net/sourceforge/servletspy/handler/filter/PackageFilter.java

Violation	Line
Avoid protected fields in a final class. Change to private or package access.	55
Found non-transient, non-static member. Please mark as transient or provide accessors.	55
Document empty constructor	60 - 61
Document empty method	76 - 77
Unnecessary final modifier in final class	84 - 107
A signature (constructor or method) shouldn't have Exception in throws declaration	84

net/sourceforge/servletspy/handler/struts/StrutsHandler.java

Violation	Line
A signature (constructor or method) shouldn't have Exception in throws declaration	46
Document empty method	81 - 82
Document empty method	89 - 90

net/sourceforge/servletspy/writer/HtmlNodeWriter.java

Violation	Line
Found non-transient, non-static member. Please mark as transient or provide accessors.	45
Avoid throwing null pointer exceptions	80
Found non-transient, non-static member. Please mark as transient or provide accessors.	100
Found non-transient, non-static member. Please mark as transient or provide accessors.	102
Avoid reassigning parameters such as 'counter'	181
Avoid reassigning parameters such as 'counter'	181
Avoid reassigning parameters such as 'counter'	181
Avoid reassigning parameters such as 'counter'	181
Avoid reassigning parameters such as 'counter'	181
Avoid reassigning parameters such as 'counter'	181