PMD Results

The following document contains the results of PMD 4.2.5.

Files

com/trendmicro/grid/acl/ApiKeyHandlingFilter.java

ViolationLine
This final field could be made static105
Avoid unused private fields such as 'productionMode'.105
Avoid unused private fields such as 'enableAuthInHttp'.106

com/trendmicro/grid/acl/client/AbstractClient.java

ViolationLine
This abstract class does not have any abstract methods14 - 50

com/trendmicro/grid/acl/l0/AuthenticationException.java

ViolationLine
It is a good practice to call super() in a constructor22 - 23

com/trendmicro/grid/acl/l0/BatchCollection.java

ViolationLine
Avoid modifiers which are implied by the context35
Avoid using short method names44
Avoid using short method names55
Avoid using short method names66
It is a good practice to call super() in a constructor185 - 188
It is a good practice to call super() in a constructor194 - 200
It is a good practice to call super() in a constructor202 - 205

com/trendmicro/grid/acl/l0/datatypes/AbstractListPage.java

ViolationLine
Overridable method 'setElements' called during object construction51

com/trendmicro/grid/acl/l0/datatypes/Category.java

ViolationLine
It is a good practice to call super() in a constructor53 - 54
Overridable method 'localize' called during object construction58
clone() method should throw CloneNotSupportedException103 - 105

com/trendmicro/grid/acl/l0/datatypes/CategoryView.java

ViolationLine
Overridable method 'setChildCategories' called during object construction34
clone() method should throw CloneNotSupportedException53 - 66
Avoid throwing raw exception types.64

com/trendmicro/grid/acl/l0/datatypes/DaysRange.java

ViolationLine
It is a good practice to call super() in a constructor60 - 61
Overridable method 'setStartDate' called during object construction65
Overridable method 'setEndDate' called during object construction66

com/trendmicro/grid/acl/l0/datatypes/FileDetails.java

ViolationLine
It is a good practice to call super() in a constructor34 - 35

com/trendmicro/grid/acl/l0/datatypes/FileIdentiferListPage.java

ViolationLine
It is a good practice to call super() in a constructor27 - 28
Overriding method merely calls super65 - 67

com/trendmicro/grid/acl/l0/datatypes/FileIdentifier.java

ViolationLine
The user-supplied array 'sha1' is stored directly.106
The user-supplied array 'md5' is stored directly.106

com/trendmicro/grid/acl/l0/datatypes/FileReport.java

ViolationLine
It is a good practice to call super() in a constructor45 - 46

com/trendmicro/grid/acl/l0/datatypes/Job.java

ViolationLine
Avoid unused constructor parameters such as 'metadata'.80 - 93

com/trendmicro/grid/acl/l0/datatypes/NamedFileIdentifier.java

ViolationLine
It is a good practice to call super() in a constructor32 - 33

com/trendmicro/grid/acl/l0/datatypes/NamedFileIdentifierListPage.java

ViolationLine
It is a good practice to call super() in a constructor25 - 26
Overriding method merely calls super64 - 66

com/trendmicro/grid/acl/l0/datatypes/NameListPage.java

ViolationLine
It is a good practice to call super() in a constructor25 - 26
Overriding method merely calls super63 - 65

com/trendmicro/grid/acl/l0/datatypes/Source.java

ViolationLine
Overridable method 'convertURIsToPlainASCII' called during object construction60
Overridable method 'ensureIdentifierIsSetAndValid' called during object construction61

com/trendmicro/grid/acl/l0/datatypes/SourceDomainListPage.java

ViolationLine
It is a good practice to call super() in a constructor28 - 29
Overriding method merely calls super66 - 68

com/trendmicro/grid/acl/l0/datatypes/SourceIdentiferListPage.java

ViolationLine
It is a good practice to call super() in a constructor27 - 28
Overriding method merely calls super65 - 67

com/trendmicro/grid/acl/l0/datatypes/SourceIdentifier.java

ViolationLine
Avoid throwing raw exception types.44
Overridable method 'setSHA1Hash' called during object construction83
Overridable method 'setSHA1Hash' called during object construction83
Overridable method 'setSHA1Hash' called during object construction83
Overridable method 'setSHA1Hash' called during object construction100
Overridable method 'setSHA1Hash' called during object construction100
Overridable method 'setSHA1Hash' called during object construction100
Overridable method 'setSHA1Hash' called during object construction104
Overridable method 'setSHA1Hash' called during object construction104
Overridable method 'setSHA1Hash' called during object construction104
The user-supplied array 'sha1' is stored directly.111
Avoid throwing raw exception types.133

com/trendmicro/grid/acl/l0/datatypes/SourceInformation.java

ViolationLine
Overridable method 'assertIsValid' called during object construction47

com/trendmicro/grid/acl/l0/datatypes/UsageStatisticsCollection.java

ViolationLine
It is somewhat confusing to have a field name matching the declaring class name38
Overridable method 'setUsageStatisticsCollection' called during object construction58

com/trendmicro/grid/acl/l0/datatypes/UUIDListPage.java

ViolationLine
It is a good practice to call super() in a constructor28 - 29
Overriding method merely calls super66 - 68

com/trendmicro/grid/acl/l0/IllegalRequestException.java

ViolationLine
It is a good practice to call super() in a constructor19 - 20

com/trendmicro/grid/acl/l0/InsufficientRightsException.java

ViolationLine
It is a good practice to call super() in a constructor30 - 31
The user-supplied array 'requiredRoles' is stored directly.37
The user-supplied array 'actualRoles' is stored directly.37

com/trendmicro/grid/acl/l0/Level0RestService.java

ViolationLine
Avoid modifiers which are implied by the context15 - 43

com/trendmicro/grid/acl/l0/PackageService.java

ViolationLine
The String literal "packageDetails" appears 4 times in this file; the first occurrence is on line 6868

com/trendmicro/grid/acl/l0/ProcessingService.java

ViolationLine
The String literal "jobGUID" appears 16 times in this file; the first occurrence is on line 4343
The String literal "contentIdentifier" appears 4 times in this file; the first occurrence is on line 155155
The String literal "sourceIdentifier" appears 4 times in this file; the first occurrence is on line 185185
The String literal "job" appears 4 times in this file; the first occurrence is on line 456456

com/trendmicro/grid/acl/l0/PublicCategoryRestService.java

ViolationLine
The String literal "locale" appears 4 times in this file; the first occurrence is on line 3939

com/trendmicro/grid/acl/l0/PublicCategoryService.java

ViolationLine
The String literal "locale" appears 5 times in this file; the first occurrence is on line 4343
The String literal "categoryName" appears 5 times in this file; the first occurrence is on line 9292
The String literal "fileId" appears 4 times in this file; the first occurrence is on line 137137
The String literal "pageNumber" appears 4 times in this file; the first occurrence is on line 141141
The String literal "names" appears 8 times in this file; the first occurrence is on line 207207

com/trendmicro/grid/acl/l0/PublicFileRestService.java

ViolationLine
The String literal "text/plain" appears 6 times in this file; the first occurrence is on line 8080
The String literal "sha1OrMd5" appears 5 times in this file; the first occurrence is on line 8383
The String literal "sha1" appears 5 times in this file; the first occurrence is on line 102102
The String literal "md5" appears 5 times in this file; the first occurrence is on line 102102

com/trendmicro/grid/acl/l0/PublicFileService.java

ViolationLine
The String literal "file" appears 14 times in this file; the first occurrence is on line 4545
The String literal "result" appears 4 times in this file; the first occurrence is on line 145145
The String literal "tag" appears 5 times in this file; the first occurrence is on line 149149

com/trendmicro/grid/acl/l0/PublicPackageRestService.java

ViolationLine
The String literal "pageNumber" appears 12 times in this file; the first occurrence is on line 7171
The String literal "text/plain" appears 5 times in this file; the first occurrence is on line 8585
The String literal "sha1OrMd5" appears 6 times in this file; the first occurrence is on line 196196
The String literal "packageName" appears 6 times in this file; the first occurrence is on line 218218

com/trendmicro/grid/acl/l0/PublicPackageService.java

ViolationLine
The String literal "pageNumber" appears 13 times in this file; the first occurrence is on line 4242
The String literal "name" appears 4 times in this file; the first occurrence is on line 5656
The String literal "packageNames" appears 6 times in this file; the first occurrence is on line 112112
The String literal "result" appears 8 times in this file; the first occurrence is on line 145145
The String literal "file" appears 4 times in this file; the first occurrence is on line 148148
The String literal "tag" appears 9 times in this file; the first occurrence is on line 149149
The String literal "packageName" appears 12 times in this file; the first occurrence is on line 238238
The String literal "packageFileId" appears 7 times in this file; the first occurrence is on line 429429
The String literal "packageInfo" appears 4 times in this file; the first occurrence is on line 490490
The String literal "packageDetails" appears 4 times in this file; the first occurrence is on line 596596

com/trendmicro/grid/acl/l0/ServerStatisticsRestService.java

ViolationLine
The String literal "localhost" appears 4 times in this file; the first occurrence is on line 7373

com/trendmicro/grid/acl/l0/SourceService.java

ViolationLine
The String literal "sourceIdentifier" appears 9 times in this file; the first occurrence is on line 4545
The String literal "pageNumber" appears 6 times in this file; the first occurrence is on line 4646
The String literal "remoteSourceURL" appears 4 times in this file; the first occurrence is on line 100100
The String literal "sourceInformation" appears 4 times in this file; the first occurrence is on line 111111
The String literal "sourceDomain" appears 4 times in this file; the first occurrence is on line 300300

com/trendmicro/grid/acl/l0/StorageFailedException.java

ViolationLine
It is a good practice to call super() in a constructor18 - 19

com/trendmicro/grid/acl/l0/WebException.java

ViolationLine
Avoid unused private fields such as 'message'.27
It is a good practice to call super() in a constructor29 - 30