PMD Resultaten

Het volgende document bevat de resultaten van PMD 6.21.0.

Violations By Priority

Priority 2

nl/geozet/openls/databinding/gml/Pos.java

Rule Overtreding Regel
GuardLogStatement Logger calls should be surrounded by log level guards. 54–55

nl/geozet/openls/parser/OpenLSRequestParser.java

Rule Overtreding Regel
GuardLogStatement Logger calls should be surrounded by log level guards. 70–71

nl/geozet/openls/parser/OpenLSResponseParser.java

Rule Overtreding Regel
GuardLogStatement Logger calls should be surrounded by log level guards. 75–76

Priority 3

nl/geozet/openls/databinding/common/XmlNamespaceConstants.java

Rule Overtreding Regel
CommentRequired Class comments are required 3–17
ConstantsInInterface Avoid constants in interfaces. Interfaces define types, constants are implementation details better placed in classes or enums. See Effective Java, item 19. 4
CommentRequired Field comments are required 4
CommentRequired Field comments are required 5
ConstantsInInterface Avoid constants in interfaces. Interfaces define types, constants are implementation details better placed in classes or enums. See Effective Java, item 19. 5
ConstantsInInterface Avoid constants in interfaces. Interfaces define types, constants are implementation details better placed in classes or enums. See Effective Java, item 19. 7
CommentRequired Field comments are required 7
CommentRequired Field comments are required 8
ConstantsInInterface Avoid constants in interfaces. Interfaces define types, constants are implementation details better placed in classes or enums. See Effective Java, item 19. 8

nl/geozet/openls/databinding/gml/CoordinatesType.java

Rule Overtreding Regel
CommentSize Comment is too large: Too many lines 3–36

nl/geozet/openls/databinding/gml/Point.java

Rule Overtreding Regel
CommentRequired Class comments are required 7–78
ReplaceVectorWithList Consider replacing this Vector with the newer java.util.List 16
LooseCoupling Avoid using implementation types like 'Vector'; use the interface instead 16
CommentRequired Field comments are required 16
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 16
CommentRequired Field comments are required 17
AvoidFieldNameMatchingMethodName Field hasSrsName has the same name as a method 19
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 19
CommentRequired Field comments are required 19
CommentRequired Public method and constructor comments are required 21–23
CommentRequired Public method and constructor comments are required 25–27
CommentRequired Public method and constructor comments are required 29–31
CommentRequired Public method and constructor comments are required 33–35
CommentRequired Public method and constructor comments are required 50–77

nl/geozet/openls/databinding/gml/Pos.java

Rule Overtreding Regel
CommentRequired Class comments are required 7–113
CommentSize Comment is too large: Too many lines 8–26
CommentRequired Field comments are required 27
CommentRequired Field comments are required 29
CommentRequired Field comments are required 30
CommentRequired Field comments are required 31
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 33
CommentRequired Field comments are required 33
CommentRequired Field comments are required 34
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 34
AvoidFieldNameMatchingMethodName Field hasXY has the same name as a method 35
CommentRequired Field comments are required 35
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 35
CommentRequired Field comments are required 36
AvoidFieldNameMatchingMethodName Field hasDimension has the same name as a method 36
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 36
CommentRequired Public method and constructor comments are required 38–43
CommentRequired Public method and constructor comments are required 45–57
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 48
CommentRequired Public method and constructor comments are required 81–83
CommentRequired Public method and constructor comments are required 101–112

nl/geozet/openls/databinding/openls/Address.java

Rule Overtreding Regel
CommentRequired Class comments are required 9–130
CommentSize Comment is too large: Too many lines 10–21
CommentRequired Field comments are required 22
CommentRequired Field comments are required 23
CommentRequired Field comments are required 24
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 25
ReplaceVectorWithList Consider replacing this Vector with the newer java.util.List 25
CommentRequired Field comments are required 25
LooseCoupling Avoid using implementation types like 'Vector'; use the interface instead 25
CommentRequired Field comments are required 26
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 26
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 28
AvoidFieldNameMatchingMethodName Field hasCountryCode has the same name as a method 28
CommentRequired Field comments are required 28
AvoidFieldNameMatchingMethodName Field hasStreetAddress has the same name as a method 29
CommentRequired Field comments are required 29
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 29
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 30
CommentRequired Field comments are required 30
AvoidFieldNameMatchingMethodName Field hasPostalCode has the same name as a method 30
CommentRequired Public method and constructor comments are required 32–36
CommentRequired Public method and constructor comments are required 38–46
CommentRequired Public method and constructor comments are required 87–92
CommentRequired Public method and constructor comments are required 94–96
CommentRequired Public method and constructor comments are required 98–100
CommentRequired Public method and constructor comments are required 102–104
CommentRequired Public method and constructor comments are required 106–108
CommentRequired Public method and constructor comments are required 110–129

nl/geozet/openls/databinding/openls/Building.java

Rule Overtreding Regel
CommentRequired Class comments are required 5–84
CommentSize Comment is too large: Too many lines 6–20
CommentRequired Field comments are required 21
CommentRequired Field comments are required 22
CommentRequired Field comments are required 23
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 25
AvoidFieldNameMatchingMethodName Field hasNumber has the same name as a method 25
CommentRequired Field comments are required 25
CommentRequired Field comments are required 26
AvoidFieldNameMatchingMethodName Field hasSubdivision has the same name as a method 26
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 26
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 27
AvoidFieldNameMatchingMethodName Field hasBuildingName has the same name as a method 27
CommentRequired Field comments are required 27
CommentRequired Public method and constructor comments are required 29–33
CommentRequired Public method and constructor comments are required 74–83

nl/geozet/openls/databinding/openls/GeocodeRequest.java

Rule Overtreding Regel
CommentSize Comment is too large: Too many lines 7–41
CommentSize Comment is too large: Line too long 12
CommentSize Comment is too large: Line too long 27
CommentSize Comment is too large: Line too long 29
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 44
CommentRequired Field comments are required 44
LooseCoupling Avoid using implementation types like 'Vector'; use the interface instead 44
ReplaceVectorWithList Consider replacing this Vector with the newer java.util.List 44
CommentRequired Public method and constructor comments are required 46–48
CommentRequired Public method and constructor comments are required 50–52
CommentRequired Public method and constructor comments are required 54–56
CommentRequired Public method and constructor comments are required 58–71

nl/geozet/openls/databinding/openls/GeocodeResponse.java

Rule Overtreding Regel
CommentRequired Class comments are required 7–47
CommentSize Comment is too large: Too many lines 8–18
ReplaceVectorWithList Consider replacing this Vector with the newer java.util.List 19
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 19
CommentRequired Field comments are required 19
LooseCoupling Avoid using implementation types like 'Vector'; use the interface instead 19
CommentRequired Public method and constructor comments are required 21–23
CommentRequired Public method and constructor comments are required 25–27
CommentRequired Public method and constructor comments are required 29–31
CommentRequired Public method and constructor comments are required 33–46

nl/geozet/openls/databinding/openls/GeocodeResponseList.java

Rule Overtreding Regel
CommentRequired Class comments are required 7–57
CommentSize Comment is too large: Too many lines 8–21
LooseCoupling Avoid using implementation types like 'Vector'; use the interface instead 23
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 23
ReplaceVectorWithList Consider replacing this Vector with the newer java.util.List 23
CommentRequired Field comments are required 23
CommentRequired Field comments are required 24
CommentRequired Public method and constructor comments are required 26–28
CommentRequired Public method and constructor comments are required 30–32
CommentRequired Public method and constructor comments are required 34–36
CommentRequired Public method and constructor comments are required 46–56

nl/geozet/openls/databinding/openls/GeocodedAddress.java

Rule Overtreding Regel
CommentRequired Class comments are required 6–63
CommentRequired Field comments are required 15
CommentRequired Field comments are required 16
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 18
AvoidFieldNameMatchingMethodName Field hasPoint has the same name as a method 18
CommentRequired Field comments are required 18
AvoidFieldNameMatchingMethodName Field hasAddress has the same name as a method 19
CommentRequired Field comments are required 19
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 19
CommentRequired Public method and constructor comments are required 21–24
CommentRequired Public method and constructor comments are required 52–62

nl/geozet/openls/databinding/openls/OpenLSConstants.java

Rule Overtreding Regel
CommentRequired Class comments are required 3–7
CommentRequired Field comments are required 4
ConstantsInInterface Avoid constants in interfaces. Interfaces define types, constants are implementation details better placed in classes or enums. See Effective Java, item 19. 4
ConstantsInInterface Avoid constants in interfaces. Interfaces define types, constants are implementation details better placed in classes or enums. See Effective Java, item 19. 5
CommentRequired Field comments are required 5
ConstantsInInterface Avoid constants in interfaces. Interfaces define types, constants are implementation details better placed in classes or enums. See Effective Java, item 19. 6
CommentRequired Field comments are required 6

nl/geozet/openls/databinding/openls/Place.java

Rule Overtreding Regel
CommentRequired Class comments are required 5–69
CommentSize Comment is too large: Too many lines 6–20
CommentRequired Field comments are required 21
CommentRequired Field comments are required 22
AvoidFieldNameMatchingTypeName It is somewhat confusing to have a field name matching the declaring class name 22
CommentRequired Field comments are required 24
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 24
AvoidFieldNameMatchingMethodName Field hasType has the same name as a method 24
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 25
CommentRequired Field comments are required 25
AvoidFieldNameMatchingMethodName Field hasPlace has the same name as a method 25
CommentRequired Public method and constructor comments are required 27–30
CommentRequired Public method and constructor comments are required 58–68

nl/geozet/openls/databinding/openls/PostalCode.java

Rule Overtreding Regel
CommentRequired Class comments are required 5–50
CommentSize Comment is too large: Too many lines 6–19
CommentRequired Field comments are required 20
AvoidFieldNameMatchingTypeName It is somewhat confusing to have a field name matching the declaring class name 20
AvoidFieldNameMatchingMethodName Field hasPostalCode has the same name as a method 22
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 22
CommentRequired Field comments are required 22
CommentRequired Public method and constructor comments are required 24–26
CommentRequired Public method and constructor comments are required 41–49

nl/geozet/openls/databinding/openls/Street.java

Rule Overtreding Regel
CommentRequired Class comments are required 5–66
CommentSize Comment is too large: Too many lines 6–35
CommentRequired Field comments are required 36
AvoidFieldNameMatchingTypeName It is somewhat confusing to have a field name matching the declaring class name 36
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 38
CommentRequired Field comments are required 38
AvoidFieldNameMatchingMethodName Field hasStreet has the same name as a method 38
CommentRequired Public method and constructor comments are required 40–42
CommentRequired Public method and constructor comments are required 57–65

nl/geozet/openls/databinding/openls/StreetAddress.java

Rule Overtreding Regel
CommentRequired Class comments are required 5–74
CommentSize Comment is too large: Too many lines 6–25
CommentRequired Field comments are required 26
CommentRequired Field comments are required 27
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 29
AvoidFieldNameMatchingMethodName Field hasBuilding has the same name as a method 29
CommentRequired Field comments are required 29
CommentRequired Field comments are required 30
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 30
AvoidFieldNameMatchingMethodName Field hasStreet has the same name as a method 30
CommentRequired Public method and constructor comments are required 32–35
CommentRequired Public method and constructor comments are required 63–73

nl/geozet/openls/parser/OpenLSRequestParser.java

Rule Overtreding Regel
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 35
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 38
AvoidStringBufferField StringBuffers can grow quite a lot, and so may become a source of memory leak (if the owning class has a long life time). 41
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 41
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 90
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 95
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 98
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 104
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 108
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 111
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 117
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 121
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 124
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 130
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 134
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 150
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 155
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 160
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 165
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 170
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 176
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 182
MissingOverride The method 'characters(char, int, int)' is missing an @Override annotation. 196–198
CommentRequired Public method and constructor comments are required 196–198

nl/geozet/openls/parser/OpenLSResponseParser.java

Rule Overtreding Regel
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 39
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 42
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 45
AvoidStringBufferField StringBuffers can grow quite a lot, and so may become a source of memory leak (if the owning class has a long life time). 45
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 95
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 100
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 103
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 109
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 114
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 117
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 123
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 127
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 133
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 137
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 143
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 147
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 150
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 156
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 160
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 163
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 169
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 173
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 189
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 194
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 200
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 206
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 211
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 217
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 222
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 227
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 232
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 238
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 244
MissingOverride The method 'characters(char, int, int)' is missing an @Override annotation. 258–260
CommentRequired Public method and constructor comments are required 258–260

Priority 5

nl/geozet/openls/parser/OpenLSRequestParser.java

Rule Overtreding Regel
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'value' (lines '103'-'138'). 103–138
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'value' (lines '103'-'103'). 103
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'obj' (lines '156'-'189'). 156–189
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'geocodeRequest' (lines '206'-'209'). 206–209

nl/geozet/openls/parser/OpenLSResponseParser.java

Rule Overtreding Regel
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'value' (lines '108'-'177'). 108–177
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'value' (lines '108'-'108'). 108
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'obj' (lines '195'-'251'). 195–251
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'geocodeResponse' (lines '268'-'271'). 268–271

Bestanden

nl/geozet/openls/databinding/common/XmlNamespaceConstants.java

Rule Overtreding Priority Regel
CommentRequired Class comments are required 3 3–17
ConstantsInInterface Avoid constants in interfaces. Interfaces define types, constants are implementation details better placed in classes or enums. See Effective Java, item 19. 3 4
CommentRequired Field comments are required 3 4
CommentRequired Field comments are required 3 5
ConstantsInInterface Avoid constants in interfaces. Interfaces define types, constants are implementation details better placed in classes or enums. See Effective Java, item 19. 3 5
ConstantsInInterface Avoid constants in interfaces. Interfaces define types, constants are implementation details better placed in classes or enums. See Effective Java, item 19. 3 7
CommentRequired Field comments are required 3 7
CommentRequired Field comments are required 3 8
ConstantsInInterface Avoid constants in interfaces. Interfaces define types, constants are implementation details better placed in classes or enums. See Effective Java, item 19. 3 8

nl/geozet/openls/databinding/gml/CoordinatesType.java

Rule Overtreding Priority Regel
CommentSize Comment is too large: Too many lines 3 3–36

nl/geozet/openls/databinding/gml/Point.java

Rule Overtreding Priority Regel
CommentRequired Class comments are required 3 7–78
ReplaceVectorWithList Consider replacing this Vector with the newer java.util.List 3 16
LooseCoupling Avoid using implementation types like 'Vector'; use the interface instead 3 16
CommentRequired Field comments are required 3 16
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 16
CommentRequired Field comments are required 3 17
AvoidFieldNameMatchingMethodName Field hasSrsName has the same name as a method 3 19
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 19
CommentRequired Field comments are required 3 19
CommentRequired Public method and constructor comments are required 3 21–23
CommentRequired Public method and constructor comments are required 3 25–27
CommentRequired Public method and constructor comments are required 3 29–31
CommentRequired Public method and constructor comments are required 3 33–35
CommentRequired Public method and constructor comments are required 3 50–77

nl/geozet/openls/databinding/gml/Pos.java

Rule Overtreding Priority Regel
CommentRequired Class comments are required 3 7–113
CommentSize Comment is too large: Too many lines 3 8–26
CommentRequired Field comments are required 3 27
CommentRequired Field comments are required 3 29
CommentRequired Field comments are required 3 30
CommentRequired Field comments are required 3 31
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 33
CommentRequired Field comments are required 3 33
CommentRequired Field comments are required 3 34
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 34
AvoidFieldNameMatchingMethodName Field hasXY has the same name as a method 3 35
CommentRequired Field comments are required 3 35
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 35
CommentRequired Field comments are required 3 36
AvoidFieldNameMatchingMethodName Field hasDimension has the same name as a method 3 36
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 36
CommentRequired Public method and constructor comments are required 3 38–43
CommentRequired Public method and constructor comments are required 3 45–57
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 3 48
GuardLogStatement Logger calls should be surrounded by log level guards. 2 54–55
CommentRequired Public method and constructor comments are required 3 81–83
CommentRequired Public method and constructor comments are required 3 101–112

nl/geozet/openls/databinding/openls/Address.java

Rule Overtreding Priority Regel
CommentRequired Class comments are required 3 9–130
CommentSize Comment is too large: Too many lines 3 10–21
CommentRequired Field comments are required 3 22
CommentRequired Field comments are required 3 23
CommentRequired Field comments are required 3 24
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 25
ReplaceVectorWithList Consider replacing this Vector with the newer java.util.List 3 25
CommentRequired Field comments are required 3 25
LooseCoupling Avoid using implementation types like 'Vector'; use the interface instead 3 25
CommentRequired Field comments are required 3 26
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 26
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 28
AvoidFieldNameMatchingMethodName Field hasCountryCode has the same name as a method 3 28
CommentRequired Field comments are required 3 28
AvoidFieldNameMatchingMethodName Field hasStreetAddress has the same name as a method 3 29
CommentRequired Field comments are required 3 29
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 29
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 30
CommentRequired Field comments are required 3 30
AvoidFieldNameMatchingMethodName Field hasPostalCode has the same name as a method 3 30
CommentRequired Public method and constructor comments are required 3 32–36
CommentRequired Public method and constructor comments are required 3 38–46
CommentRequired Public method and constructor comments are required 3 87–92
CommentRequired Public method and constructor comments are required 3 94–96
CommentRequired Public method and constructor comments are required 3 98–100
CommentRequired Public method and constructor comments are required 3 102–104
CommentRequired Public method and constructor comments are required 3 106–108
CommentRequired Public method and constructor comments are required 3 110–129

nl/geozet/openls/databinding/openls/Building.java

Rule Overtreding Priority Regel
CommentRequired Class comments are required 3 5–84
CommentSize Comment is too large: Too many lines 3 6–20
CommentRequired Field comments are required 3 21
CommentRequired Field comments are required 3 22
CommentRequired Field comments are required 3 23
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 25
AvoidFieldNameMatchingMethodName Field hasNumber has the same name as a method 3 25
CommentRequired Field comments are required 3 25
CommentRequired Field comments are required 3 26
AvoidFieldNameMatchingMethodName Field hasSubdivision has the same name as a method 3 26
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 26
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 27
AvoidFieldNameMatchingMethodName Field hasBuildingName has the same name as a method 3 27
CommentRequired Field comments are required 3 27
CommentRequired Public method and constructor comments are required 3 29–33
CommentRequired Public method and constructor comments are required 3 74–83

nl/geozet/openls/databinding/openls/GeocodeRequest.java

Rule Overtreding Priority Regel
CommentSize Comment is too large: Too many lines 3 7–41
CommentSize Comment is too large: Line too long 3 12
CommentSize Comment is too large: Line too long 3 27
CommentSize Comment is too large: Line too long 3 29
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 44
CommentRequired Field comments are required 3 44
LooseCoupling Avoid using implementation types like 'Vector'; use the interface instead 3 44
ReplaceVectorWithList Consider replacing this Vector with the newer java.util.List 3 44
CommentRequired Public method and constructor comments are required 3 46–48
CommentRequired Public method and constructor comments are required 3 50–52
CommentRequired Public method and constructor comments are required 3 54–56
CommentRequired Public method and constructor comments are required 3 58–71

nl/geozet/openls/databinding/openls/GeocodeResponse.java

Rule Overtreding Priority Regel
CommentRequired Class comments are required 3 7–47
CommentSize Comment is too large: Too many lines 3 8–18
ReplaceVectorWithList Consider replacing this Vector with the newer java.util.List 3 19
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 19
CommentRequired Field comments are required 3 19
LooseCoupling Avoid using implementation types like 'Vector'; use the interface instead 3 19
CommentRequired Public method and constructor comments are required 3 21–23
CommentRequired Public method and constructor comments are required 3 25–27
CommentRequired Public method and constructor comments are required 3 29–31
CommentRequired Public method and constructor comments are required 3 33–46

nl/geozet/openls/databinding/openls/GeocodeResponseList.java

Rule Overtreding Priority Regel
CommentRequired Class comments are required 3 7–57
CommentSize Comment is too large: Too many lines 3 8–21
LooseCoupling Avoid using implementation types like 'Vector'; use the interface instead 3 23
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 23
ReplaceVectorWithList Consider replacing this Vector with the newer java.util.List 3 23
CommentRequired Field comments are required 3 23
CommentRequired Field comments are required 3 24
CommentRequired Public method and constructor comments are required 3 26–28
CommentRequired Public method and constructor comments are required 3 30–32
CommentRequired Public method and constructor comments are required 3 34–36
CommentRequired Public method and constructor comments are required 3 46–56

nl/geozet/openls/databinding/openls/GeocodedAddress.java

Rule Overtreding Priority Regel
CommentRequired Class comments are required 3 6–63
CommentRequired Field comments are required 3 15
CommentRequired Field comments are required 3 16
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 18
AvoidFieldNameMatchingMethodName Field hasPoint has the same name as a method 3 18
CommentRequired Field comments are required 3 18
AvoidFieldNameMatchingMethodName Field hasAddress has the same name as a method 3 19
CommentRequired Field comments are required 3 19
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 19
CommentRequired Public method and constructor comments are required 3 21–24
CommentRequired Public method and constructor comments are required 3 52–62

nl/geozet/openls/databinding/openls/OpenLSConstants.java

Rule Overtreding Priority Regel
CommentRequired Class comments are required 3 3–7
CommentRequired Field comments are required 3 4
ConstantsInInterface Avoid constants in interfaces. Interfaces define types, constants are implementation details better placed in classes or enums. See Effective Java, item 19. 3 4
ConstantsInInterface Avoid constants in interfaces. Interfaces define types, constants are implementation details better placed in classes or enums. See Effective Java, item 19. 3 5
CommentRequired Field comments are required 3 5
ConstantsInInterface Avoid constants in interfaces. Interfaces define types, constants are implementation details better placed in classes or enums. See Effective Java, item 19. 3 6
CommentRequired Field comments are required 3 6

nl/geozet/openls/databinding/openls/Place.java

Rule Overtreding Priority Regel
CommentRequired Class comments are required 3 5–69
CommentSize Comment is too large: Too many lines 3 6–20
CommentRequired Field comments are required 3 21
CommentRequired Field comments are required 3 22
AvoidFieldNameMatchingTypeName It is somewhat confusing to have a field name matching the declaring class name 3 22
CommentRequired Field comments are required 3 24
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 24
AvoidFieldNameMatchingMethodName Field hasType has the same name as a method 3 24
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 25
CommentRequired Field comments are required 3 25
AvoidFieldNameMatchingMethodName Field hasPlace has the same name as a method 3 25
CommentRequired Public method and constructor comments are required 3 27–30
CommentRequired Public method and constructor comments are required 3 58–68

nl/geozet/openls/databinding/openls/PostalCode.java

Rule Overtreding Priority Regel
CommentRequired Class comments are required 3 5–50
CommentSize Comment is too large: Too many lines 3 6–19
CommentRequired Field comments are required 3 20
AvoidFieldNameMatchingTypeName It is somewhat confusing to have a field name matching the declaring class name 3 20
AvoidFieldNameMatchingMethodName Field hasPostalCode has the same name as a method 3 22
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 22
CommentRequired Field comments are required 3 22
CommentRequired Public method and constructor comments are required 3 24–26
CommentRequired Public method and constructor comments are required 3 41–49

nl/geozet/openls/databinding/openls/Street.java

Rule Overtreding Priority Regel
CommentRequired Class comments are required 3 5–66
CommentSize Comment is too large: Too many lines 3 6–35
CommentRequired Field comments are required 3 36
AvoidFieldNameMatchingTypeName It is somewhat confusing to have a field name matching the declaring class name 3 36
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 38
CommentRequired Field comments are required 3 38
AvoidFieldNameMatchingMethodName Field hasStreet has the same name as a method 3 38
CommentRequired Public method and constructor comments are required 3 40–42
CommentRequired Public method and constructor comments are required 3 57–65

nl/geozet/openls/databinding/openls/StreetAddress.java

Rule Overtreding Priority Regel
CommentRequired Class comments are required 3 5–74
CommentSize Comment is too large: Too many lines 3 6–25
CommentRequired Field comments are required 3 26
CommentRequired Field comments are required 3 27
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 29
AvoidFieldNameMatchingMethodName Field hasBuilding has the same name as a method 3 29
CommentRequired Field comments are required 3 29
CommentRequired Field comments are required 3 30
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 30
AvoidFieldNameMatchingMethodName Field hasStreet has the same name as a method 3 30
CommentRequired Public method and constructor comments are required 3 32–35
CommentRequired Public method and constructor comments are required 3 63–73

nl/geozet/openls/parser/OpenLSRequestParser.java

Rule Overtreding Priority Regel
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 35
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 38
AvoidStringBufferField StringBuffers can grow quite a lot, and so may become a source of memory leak (if the owning class has a long life time). 3 41
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 41
GuardLogStatement Logger calls should be surrounded by log level guards. 2 70–71
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 3 90
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 3 95
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 3 98
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'value' (lines '103'-'138'). 5 103–138
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'value' (lines '103'-'103'). 5 103
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 3 104
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 3 108
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 3 111
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 3 117
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 3 121
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 3 124
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 3 130
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 3 134
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 3 150
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 3 155
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'obj' (lines '156'-'189'). 5 156–189
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 3 160
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 3 165
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 3 170
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 3 176
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 3 182
MissingOverride The method 'characters(char, int, int)' is missing an @Override annotation. 3 196–198
CommentRequired Public method and constructor comments are required 3 196–198
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'geocodeRequest' (lines '206'-'209'). 5 206–209

nl/geozet/openls/parser/OpenLSResponseParser.java

Rule Overtreding Priority Regel
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 39
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 42
BeanMembersShouldSerialize Found non-transient, non-static member. Please mark as transient or provide accessors. 3 45
AvoidStringBufferField StringBuffers can grow quite a lot, and so may become a source of memory leak (if the owning class has a long life time). 3 45
GuardLogStatement Logger calls should be surrounded by log level guards. 2 75–76
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 3 95
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 3 100
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 3 103
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'value' (lines '108'-'177'). 5 108–177
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'value' (lines '108'-'108'). 5 108
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 3 109
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 3 114
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 3 117
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 3 123
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 3 127
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 3 133
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 3 137
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 3 143
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 3 147
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 3 150
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 3 156
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 3 160
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 3 163
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 3 169
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 3 173
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 3 189
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 3 194
DataflowAnomalyAnalysis Found 'DU'-anomaly for variable 'obj' (lines '195'-'251'). 5 195–251
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 3 200
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 3 206
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 3 211
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 3 217
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 3 222
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 3 227
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 3 232
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 3 238
PositionLiteralsFirstInCaseInsensitiveComparisons Position literals first in String comparisons for EqualsIgnoreCase 3 244
MissingOverride The method 'characters(char, int, int)' is missing an @Override annotation. 3 258–260
CommentRequired Public method and constructor comments are required 3 258–260
DataflowAnomalyAnalysis Found 'DD'-anomaly for variable 'geocodeResponse' (lines '268'-'271'). 5 268–271