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 317331
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 336341
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 349355
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 360362
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 369374
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 380382
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 393418
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 171203
ExcessiveClassLength Avoid really long classes. 301203
CyclomaticComplexity The class 'SimpleTextParser' has a total cyclomatic complexity of 135 (highest 9). 301203
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 985987
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) 29147

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 317331
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 3 336341
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 3 349355
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 3 360362
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 3 369374
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 3 380382
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 3 393418
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 171203
ExcessiveClassLength Avoid really long classes. 3 301203
CyclomaticComplexity The class 'SimpleTextParser' has a total cyclomatic complexity of 135 (highest 9). 3 301203
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 985987
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 29147