PMD Results

The following document contains the results of PMD 4.2.5.

Files

com/trendmicro/grid/acl/ds/cache/aspects/AbstractCacheAspect.java

ViolationLine
The String literal "${not runClustered || readonly ? 15 : 6000}" appears 5 times in this file; the first occurrence is on line 4141
This abstract class does not have any abstract methods116 - 185
A method/constructor shouldn't explicitly throw java.lang.Exception121
A method/constructor shouldn't explicitly throw java.lang.Exception121
A method/constructor shouldn't explicitly throw java.lang.Exception133
A method/constructor shouldn't explicitly throw java.lang.Exception133
A method/constructor shouldn't explicitly throw java.lang.Exception143

com/trendmicro/grid/acl/ds/cache/aspects/FileDetailsCacheAspect.java

ViolationLine
A method/constructor shouldn't explicitly throw java.lang.Exception69
A method/constructor shouldn't explicitly throw java.lang.Exception91
A method/constructor shouldn't explicitly throw java.lang.Exception113
A method/constructor shouldn't explicitly throw java.lang.Exception131

com/trendmicro/grid/acl/ds/cache/aspects/PackageDetailsCacheAspect.java

ViolationLine
A method/constructor shouldn't explicitly throw java.lang.Exception87
A method/constructor shouldn't explicitly throw java.lang.Exception112
A method/constructor shouldn't explicitly throw java.lang.Exception136
A method/constructor shouldn't explicitly throw java.lang.Exception160
A method/constructor shouldn't explicitly throw java.lang.Exception184
A method/constructor shouldn't explicitly throw java.lang.Exception202
A method/constructor shouldn't explicitly throw java.lang.Exception305

com/trendmicro/grid/acl/ds/cache/CacheAdapter.java

ViolationLine
Avoid throwing raw exception types.97
Avoid throwing raw exception types.99

com/trendmicro/grid/acl/ds/cache/commands/AbstractCommand.java

ViolationLine
It is somewhat confusing to have a field name with the same name as a method42
It is a good practice to call super() in a constructor45 - 86
It is a good practice to call super() in a constructor140 - 142
It is a good practice to call super() in a constructor144 - 146

com/trendmicro/grid/acl/ds/cache/commands/BatchCommand.java

ViolationLine
A method/constructor shouldn't explicitly throw java.lang.Exception60
A method/constructor shouldn't explicitly throw java.lang.Exception98

com/trendmicro/grid/acl/ds/cache/commands/CacheResultsCommand.java

ViolationLine
A method/constructor shouldn't explicitly throw java.lang.Exception69

com/trendmicro/grid/acl/ds/cache/commands/GetFromCacheCommand.java

ViolationLine
A method/constructor shouldn't explicitly throw java.lang.Exception116
A method/constructor shouldn't explicitly throw java.lang.Exception136
A method/constructor shouldn't explicitly throw java.lang.Exception172
A method/constructor shouldn't explicitly throw java.lang.Exception195
Do not add empty strings224
A method/constructor shouldn't explicitly throw java.lang.Exception247

com/trendmicro/grid/acl/ds/cache/commands/GetFromPJPSourceCommand.java

ViolationLine
A method/constructor shouldn't explicitly throw java.lang.Exception31
A catch statement that catches an exception only to rethrow it should be avoided.36 - 38
A catch statement should never catch throwable since it includes errors.38
Avoid throwing raw exception types.39

com/trendmicro/grid/acl/ds/cache/commands/GetFromSourceCommand.java

ViolationLine
A method/constructor shouldn't explicitly throw java.lang.Exception30
A method/constructor shouldn't explicitly throw java.lang.Exception35

com/trendmicro/grid/acl/ds/cache/commands/RemoveCommand.java

ViolationLine
Avoid unused private fields such as 'log'.20
A method/constructor shouldn't explicitly throw java.lang.Exception55

com/trendmicro/grid/acl/ds/cache/commands/RetainMissingKeysCommand.java

ViolationLine
A method/constructor shouldn't explicitly throw java.lang.Exception57

com/trendmicro/grid/acl/ds/cache/ThirdLevelCacheStatisticsSummary.java

ViolationLine
Do not add empty strings33

com/trendmicro/grid/acl/ds/cache/translation/IdentifierToNameTranslator.java

ViolationLine
It is a good practice to call super() in a constructor41 - 43

com/trendmicro/grid/acl/ds/cache/translation/Translator.java

ViolationLine
This abstract class does not have any abstract methods27 - 108
The String literal "unchecked" appears 5 times in this file; the first occurrence is on line 3838

net/sf/tinyjee/cache/CacheContext.java

ViolationLine
It is a good practice to call super() in a constructor97 - 99
Avoid throwing raw exception types.124
New exception is thrown in catch block, original stack trace may be lost124 - 126
Avoid throwing raw exception types.133
A method/constructor shouldn't explicitly throw java.lang.Exception192
A method/constructor shouldn't explicitly throw java.lang.Exception202
A method/constructor shouldn't explicitly throw java.lang.Exception219
Avoid throwing raw exception types.514
New exception is thrown in catch block, original stack trace may be lost514
A catch statement should never catch throwable since it includes errors.600

net/sf/tinyjee/jgroups/config/JGroupsConfigurator.java

ViolationLine
A method/constructor shouldn't explicitly throw java.lang.Exception143
A method/constructor shouldn't explicitly throw java.lang.Exception166
A method/constructor shouldn't explicitly throw java.lang.Exception195
A method/constructor shouldn't explicitly throw java.lang.Exception224
A method/constructor shouldn't explicitly throw java.lang.Exception242
Ensure that resources like this Connection object are closed after use276
A method/constructor shouldn't explicitly throw java.lang.Exception307
Ensure that resources like this Connection object are closed after use311