Apache Commons logo Apache Commons Geometry

PMD Results

The following document contains the results of PMD 6.36.0.

Violations By Priority

Priority 3

org/apache/commons/geometry/io/core/BoundaryIOManager.java

Rule Violation Line
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 307
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 310
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 317–331
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 336–341
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 349–355
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 360–362
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 369–374
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 380–382
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 393–418
UnusedAssignment The initializer for variable 'handler' is never used (overwritten on lines 396 and 405) 394

org/apache/commons/geometry/io/core/internal/GeometryIOUtils.java

Rule Violation Line
CloseResource Ensure that resources like this Closeable object are closed after use 241
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 246

org/apache/commons/geometry/io/core/internal/SimpleTextParser.java

Rule Violation Line
ExcessivePublicCount This class has a bunch of public methods and attributes 17–1203
ExcessiveClassLength Avoid really long classes. 30–1203
CyclomaticComplexity The class 'SimpleTextParser' has a total cyclomatic complexity of 135 (highest 9). 30–1203
AssignmentInOperand Avoid assignments in operands 541
AssignmentInOperand Avoid assignments in operands 561
AssignmentInOperand Avoid assignments in operands 607
InsufficientStringBufferDeclaration StringBuilder has been initialized with size 16, but has at least 20 characters appended. 882
LocalVariableCouldBeFinal Local variable 'msg' could be declared final 882
EmptyCatchBlock Avoid empty catch blocks 985–987
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). 1150
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 1190
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 1200

org/apache/commons/geometry/io/core/utils/AbstractTextFormatWriter.java

Rule Violation Line
DataClass The class 'AbstractTextFormatWriter' is suspected to be a Data Class (WOC=20.000%, NOPA=0, NOAM=4, WMC=17) 29–147

Files

org/apache/commons/geometry/io/core/BoundaryIOManager.java

Rule Violation Priority Line
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 307
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 310
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 3 317–331
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 3 336–341
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 3 349–355
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 3 360–362
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 3 369–374
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 3 380–382
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 3 393–418
UnusedAssignment The initializer for variable 'handler' is never used (overwritten on lines 396 and 405) 3 394

org/apache/commons/geometry/io/core/internal/GeometryIOUtils.java

Rule Violation Priority Line
CloseResource Ensure that resources like this Closeable object are closed after use 3 241
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 246

org/apache/commons/geometry/io/core/internal/SimpleTextParser.java

Rule Violation Priority Line
ExcessivePublicCount This class has a bunch of public methods and attributes 3 17–1203
ExcessiveClassLength Avoid really long classes. 3 30–1203
CyclomaticComplexity The class 'SimpleTextParser' has a total cyclomatic complexity of 135 (highest 9). 3 30–1203
AssignmentInOperand Avoid assignments in operands 3 541
AssignmentInOperand Avoid assignments in operands 3 561
AssignmentInOperand Avoid assignments in operands 3 607
InsufficientStringBufferDeclaration StringBuilder has been initialized with size 16, but has at least 20 characters appended. 3 882
LocalVariableCouldBeFinal Local variable 'msg' could be declared final 3 882
EmptyCatchBlock Avoid empty catch blocks 3 985–987
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 1150
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 1190
AccessorMethodGeneration Avoid autogenerated methods to access private fields and methods of inner / outer classes 3 1200

org/apache/commons/geometry/io/core/utils/AbstractTextFormatWriter.java

Rule Violation Priority Line
DataClass The class 'AbstractTextFormatWriter' is suspected to be a Data Class (WOC=20.000%, NOPA=0, NOAM=4, WMC=17) 3 29–147