PMD Results

The following document contains the results of PMD 5.0.2.

Files

org/andromda/scriptwrappers/BshScriptWrapper.java

Violation Line
Found non-transient, non-static member. Please mark as transient or provide accessors. 21
Found non-transient, non-static member. Please mark as transient or provide accessors. 22
Found non-transient, non-static member. Please mark as transient or provide accessors. 23
Parameter 'stub' is not assigned and could be declared final 33
Parameter 'scriptPath' is not assigned and could be declared final 34
Assigning an Object to null is a code smell. Consider refactoring. 47
Parameter 'stub' is not assigned and could be declared final 58
Avoid unused method parameters such as 'scriptPath'. 59
Parameter 'scriptPath' is not assigned and could be declared final 59
Parameter 'methodName' is not assigned and could be declared final 74
Parameter 'args' is not assigned and could be declared final 75
Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 93
Avoid printStackTrace(); use a logger call instead. 95
Unnecessary use of fully qualified name 'bsh.Primitive' due to existing import 'bsh.Primitive' 117
Avoid throwing raw exception types. 126
Avoid variables with short names like to 140
A method/constructor shouldnt explicitly throw java.lang.Exception 141
A method/constructor shouldnt explicitly throw java.lang.Exception 141
Local variable 'propertyName' could be declared final 155 - 157
Avoid instantiating new objects inside loops 165
Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 167
Avoid empty catch blocks 167 - 170

org/andromda/scriptwrappers/GroovyScriptWrapper.java

Violation Line
Found non-transient, non-static member. Please mark as transient or provide accessors. 26
Private field 'scriptPath' could be made final; it is only initialized in the declaration or constructor. 26
Found non-transient, non-static member. Please mark as transient or provide accessors. 27
Private field 'stub' could be made final; it is only initialized in the declaration or constructor. 27
Parameter 'stub' is not assigned and could be declared final 37
Parameter 'scriptPath' is not assigned and could be declared final 38
Parameter 'methodName' is not assigned and could be declared final 53
Parameter 'args' is not assigned and could be declared final 54
Local variable 'rtn' could be declared final 68 - 70
A catch statement should never catch throwable since it includes errors. 74
Avoid throwing raw exception types. 76
Local variable 'resource' could be declared final 111
Assigning an Object to null is a code smell. Consider refactoring. 121
A catch statement should never catch throwable since it includes errors. 123
Avoid throwing raw exception types. 125
Avoid empty catch blocks 133 - 136
Avoid variables with short names like to 156
A method/constructor shouldnt explicitly throw java.lang.Exception 157
A method/constructor shouldnt explicitly throw java.lang.Exception 157
Local variable 'propertyName' could be declared final 171 - 173
Avoid instantiating new objects inside loops 181
Avoid empty catch blocks 183 - 186
Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 183
Avoid instantiating new objects inside loops 194