PMD Results
The following document contains the results of PMD 6.36.0.
Violations By Priority
Priority 3
org/apache/commons/geometry/euclidean/internal/AbstractPathConnector.java
Rule |
Violation |
Line |
CompareObjectsWithEquals |
Use equals() to compare object references. |
194 |
org/apache/commons/geometry/euclidean/oned/OrientedPoint.java
Rule |
Violation |
Line |
InsufficientStringBufferDeclaration |
StringBuilder has been initialized with size 16, but has at least 22 characters appended. |
239 |
org/apache/commons/geometry/euclidean/threed/AbstractEmbeddedRegionPlaneSubset.java
Rule |
Violation |
Line |
InsufficientStringBufferDeclaration |
StringBuilder has been initialized with size 16, but has at least 27 characters appended. |
107 |
org/apache/commons/geometry/euclidean/threed/Bounds3D.java
Rule |
Violation |
Line |
AccessorClassGeneration |
Avoid instantiation through private constructors from outside of the constructors class. |
184 |
AccessorClassGeneration |
Avoid instantiation through private constructors from outside of the constructors class. |
287 |
org/apache/commons/geometry/euclidean/threed/EmbeddingPlane.java
Rule |
Violation |
Line |
InsufficientStringBufferDeclaration |
StringBuilder has been initialized with size 16, but has at least 25 characters appended. |
239 |
org/apache/commons/geometry/euclidean/threed/RegionBSPTree3D.java
Rule |
Violation |
Line |
LinguisticNaming |
Linguistics Antipattern - The transform method 'toConvexRecursive' should not return void linguistically |
108–123 |
AccessorClassGeneration |
Avoid instantiation through private constructors from outside of the constructors class. |
136 |
AccessorClassGeneration |
Avoid instantiation through private constructors from outside of the constructors class. |
232 |
AssignmentInOperand |
Avoid assignments in operands |
256 |
org/apache/commons/geometry/euclidean/threed/line/EmbeddedTreeLineSubset3D.java
Rule |
Violation |
Line |
InsufficientStringBufferDeclaration |
StringBuilder has been initialized with size 16, but has at least 41 characters appended. |
144 |
ConsecutiveLiteralAppends |
StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. |
146 |
org/apache/commons/geometry/euclidean/threed/line/Ray3D.java
Rule |
Violation |
Line |
InsufficientStringBufferDeclaration |
StringBuilder has been initialized with size 16, but has at least 27 characters appended. |
148 |
org/apache/commons/geometry/euclidean/threed/line/ReverseRay3D.java
Rule |
Violation |
Line |
InsufficientStringBufferDeclaration |
StringBuilder has been initialized with size 16, but has at least 25 characters appended. |
142 |
org/apache/commons/geometry/euclidean/threed/line/Segment3D.java
Rule |
Violation |
Line |
InsufficientStringBufferDeclaration |
StringBuilder has been initialized with size 16, but has at least 26 characters appended. |
139 |
org/apache/commons/geometry/euclidean/threed/rotation/AxisAngleSequence.java
Rule |
Violation |
Line |
InsufficientStringBufferDeclaration |
StringBuilder has been initialized with size 16, but has at least 41 characters appended. |
167 |
org/apache/commons/geometry/euclidean/threed/rotation/QuaternionRotation.java
Rule |
Violation |
Line |
PrematureDeclaration |
Avoid declaring a variable if it is unreferenced before a possible exit point. |
403 |
PrematureDeclaration |
Avoid declaring a variable if it is unreferenced before a possible exit point. |
472 |
PrematureDeclaration |
Avoid declaring a variable if it is unreferenced before a possible exit point. |
589 |
org/apache/commons/geometry/euclidean/threed/shape/Parallelepiped.java
Rule |
Violation |
Line |
AccessorClassGeneration |
Avoid instantiation through private constructors from outside of the constructors class. |
178 |
LinguisticNaming |
Linguistics Antipattern - The setter 'setPosition' should not return any type except void linguistically |
254–257 |
LinguisticNaming |
Linguistics Antipattern - The setter 'setScale' should not return any type except void linguistically |
264–267 |
LinguisticNaming |
Linguistics Antipattern - The setter 'setScale' should not return any type except void linguistically |
276–278 |
LinguisticNaming |
Linguistics Antipattern - The setter 'setScale' should not return any type except void linguistically |
285–287 |
LinguisticNaming |
Linguistics Antipattern - The setter 'setRotation' should not return any type except void linguistically |
293–296 |
org/apache/commons/geometry/euclidean/twod/Bounds2D.java
Rule |
Violation |
Line |
AccessorClassGeneration |
Avoid instantiation through private constructors from outside of the constructors class. |
176 |
AccessorClassGeneration |
Avoid instantiation through private constructors from outside of the constructors class. |
268 |
org/apache/commons/geometry/euclidean/twod/EmbeddedTreeLineSubset.java
Rule |
Violation |
Line |
PrematureDeclaration |
Avoid declaring a variable if it is unreferenced before a possible exit point. |
173 |
InsufficientStringBufferDeclaration |
StringBuilder has been initialized with size 16, but has at least 41 characters appended. |
233 |
ConsecutiveLiteralAppends |
StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. |
235 |
org/apache/commons/geometry/euclidean/twod/Ray.java
Rule |
Violation |
Line |
InsufficientStringBufferDeclaration |
StringBuilder has been initialized with size 16, but has at least 27 characters appended. |
155 |
org/apache/commons/geometry/euclidean/twod/RegionBSPTree2D.java
Rule |
Violation |
Line |
LinguisticNaming |
Linguistics Antipattern - The transform method 'toConvexRecursive' should not return void linguistically |
131–145 |
NullAssignment |
Assigning an Object to null is a code smell. Consider refactoring. |
265 |
AccessorClassGeneration |
Avoid instantiation through private constructors from outside of the constructors class. |
271 |
AccessorClassGeneration |
Avoid instantiation through private constructors from outside of the constructors class. |
317 |
AssignmentInOperand |
Avoid assignments in operands |
341 |
org/apache/commons/geometry/euclidean/twod/ReverseRay.java
Rule |
Violation |
Line |
InsufficientStringBufferDeclaration |
StringBuilder has been initialized with size 16, but has at least 25 characters appended. |
150 |
org/apache/commons/geometry/euclidean/twod/Segment.java
Rule |
Violation |
Line |
InsufficientStringBufferDeclaration |
StringBuilder has been initialized with size 16, but has at least 26 characters appended. |
145 |
PrematureDeclaration |
Avoid declaring a variable if it is unreferenced before a possible exit point. |
184 |
org/apache/commons/geometry/euclidean/twod/shape/Parallelogram.java
Rule |
Violation |
Line |
AccessorClassGeneration |
Avoid instantiation through private constructors from outside of the constructors class. |
161 |
LinguisticNaming |
Linguistics Antipattern - The setter 'setPosition' should not return any type except void linguistically |
196–199 |
LinguisticNaming |
Linguistics Antipattern - The setter 'setScale' should not return any type except void linguistically |
207–210 |
LinguisticNaming |
Linguistics Antipattern - The setter 'setScale' should not return any type except void linguistically |
219–221 |
LinguisticNaming |
Linguistics Antipattern - The setter 'setScale' should not return any type except void linguistically |
228–230 |
LinguisticNaming |
Linguistics Antipattern - The setter 'setRotation' should not return any type except void linguistically |
236–239 |
LinguisticNaming |
Linguistics Antipattern - The setter 'setXDirection' should not return any type except void linguistically |
248–251 |
LinguisticNaming |
Linguistics Antipattern - The setter 'setYDirection' should not return any type except void linguistically |
260–263 |
Priority 4
org/apache/commons/geometry/euclidean/threed/mesh/Mesh.java
Rule |
Violation |
Line |
ShortClassName |
Avoid short class names like Mesh |
30–114 |
ShortClassName |
Avoid short class names like Face |
78–113 |
org/apache/commons/geometry/euclidean/threed/mesh/TriangleMesh.java
Rule |
Violation |
Line |
ShortClassName |
Avoid short class names like Face |
33–53 |
org/apache/commons/geometry/euclidean/twod/Line.java
Rule |
Violation |
Line |
ShortClassName |
Avoid short class names like Line |
59–545 |
org/apache/commons/geometry/euclidean/twod/Ray.java
Rule |
Violation |
Line |
ShortClassName |
Avoid short class names like Ray |
31–211 |
Files
org/apache/commons/geometry/euclidean/internal/AbstractPathConnector.java
Rule |
Violation |
Priority |
Line |
CompareObjectsWithEquals |
Use equals() to compare object references. |
3 |
194 |
org/apache/commons/geometry/euclidean/oned/OrientedPoint.java
Rule |
Violation |
Priority |
Line |
InsufficientStringBufferDeclaration |
StringBuilder has been initialized with size 16, but has at least 22 characters appended. |
3 |
239 |
org/apache/commons/geometry/euclidean/threed/AbstractConvexPolygon3D.java
Rule |
Violation |
Priority |
Line |
CyclomaticComplexity |
The method 'split(Hyperplane)' has a cyclomatic complexity of 14. |
3 |
296–382 |
CognitiveComplexity |
The method 'split(Hyperplane)' has a cognitive complexity of 20, current threshold is 15 |
3 |
296–382 |
InsufficientStringBufferDeclaration |
StringBuilder has been initialized with size 16, but has at least 22 characters appended. |
3 |
441 |
org/apache/commons/geometry/euclidean/threed/AbstractEmbeddedRegionPlaneSubset.java
Rule |
Violation |
Priority |
Line |
InsufficientStringBufferDeclaration |
StringBuilder has been initialized with size 16, but has at least 27 characters appended. |
3 |
107 |
org/apache/commons/geometry/euclidean/threed/Bounds3D.java
Rule |
Violation |
Priority |
Line |
AccessorClassGeneration |
Avoid instantiation through private constructors from outside of the constructors class. |
3 |
184 |
AccessorClassGeneration |
Avoid instantiation through private constructors from outside of the constructors class. |
3 |
287 |
org/apache/commons/geometry/euclidean/threed/ConvexVolume.java
Rule |
Violation |
Priority |
Line |
AvoidFieldNameMatchingMethodName |
Field FULL has the same name as a method |
3 |
37 |
org/apache/commons/geometry/euclidean/threed/EmbeddingPlane.java
Rule |
Violation |
Priority |
Line |
InsufficientStringBufferDeclaration |
StringBuilder has been initialized with size 16, but has at least 25 characters appended. |
3 |
239 |
org/apache/commons/geometry/euclidean/threed/RegionBSPTree3D.java
Rule |
Violation |
Priority |
Line |
LinguisticNaming |
Linguistics Antipattern - The transform method 'toConvexRecursive' should not return void linguistically |
3 |
108–123 |
AccessorClassGeneration |
Avoid instantiation through private constructors from outside of the constructors class. |
3 |
136 |
AccessorClassGeneration |
Avoid instantiation through private constructors from outside of the constructors class. |
3 |
232 |
AssignmentInOperand |
Avoid assignments in operands |
3 |
256 |
org/apache/commons/geometry/euclidean/threed/line/EmbeddedTreeLineSubset3D.java
Rule |
Violation |
Priority |
Line |
InsufficientStringBufferDeclaration |
StringBuilder has been initialized with size 16, but has at least 41 characters appended. |
3 |
144 |
ConsecutiveLiteralAppends |
StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. |
3 |
146 |
org/apache/commons/geometry/euclidean/threed/line/Ray3D.java
Rule |
Violation |
Priority |
Line |
InsufficientStringBufferDeclaration |
StringBuilder has been initialized with size 16, but has at least 27 characters appended. |
3 |
148 |
org/apache/commons/geometry/euclidean/threed/line/ReverseRay3D.java
Rule |
Violation |
Priority |
Line |
InsufficientStringBufferDeclaration |
StringBuilder has been initialized with size 16, but has at least 25 characters appended. |
3 |
142 |
org/apache/commons/geometry/euclidean/threed/line/Segment3D.java
Rule |
Violation |
Priority |
Line |
InsufficientStringBufferDeclaration |
StringBuilder has been initialized with size 16, but has at least 26 characters appended. |
3 |
139 |
org/apache/commons/geometry/euclidean/threed/mesh/Mesh.java
Rule |
Violation |
Priority |
Line |
ShortClassName |
Avoid short class names like Mesh |
4 |
30–114 |
ShortClassName |
Avoid short class names like Face |
4 |
78–113 |
org/apache/commons/geometry/euclidean/threed/mesh/TriangleMesh.java
Rule |
Violation |
Priority |
Line |
ShortClassName |
Avoid short class names like Face |
4 |
33–53 |
org/apache/commons/geometry/euclidean/threed/rotation/AxisAngleSequence.java
Rule |
Violation |
Priority |
Line |
InsufficientStringBufferDeclaration |
StringBuilder has been initialized with size 16, but has at least 41 characters appended. |
3 |
167 |
org/apache/commons/geometry/euclidean/threed/rotation/QuaternionRotation.java
Rule |
Violation |
Priority |
Line |
PrematureDeclaration |
Avoid declaring a variable if it is unreferenced before a possible exit point. |
3 |
403 |
PrematureDeclaration |
Avoid declaring a variable if it is unreferenced before a possible exit point. |
3 |
472 |
PrematureDeclaration |
Avoid declaring a variable if it is unreferenced before a possible exit point. |
3 |
589 |
org/apache/commons/geometry/euclidean/threed/shape/Parallelepiped.java
Rule |
Violation |
Priority |
Line |
AccessorClassGeneration |
Avoid instantiation through private constructors from outside of the constructors class. |
3 |
178 |
LinguisticNaming |
Linguistics Antipattern - The setter 'setPosition' should not return any type except void linguistically |
3 |
254–257 |
LinguisticNaming |
Linguistics Antipattern - The setter 'setScale' should not return any type except void linguistically |
3 |
264–267 |
LinguisticNaming |
Linguistics Antipattern - The setter 'setScale' should not return any type except void linguistically |
3 |
276–278 |
LinguisticNaming |
Linguistics Antipattern - The setter 'setScale' should not return any type except void linguistically |
3 |
285–287 |
LinguisticNaming |
Linguistics Antipattern - The setter 'setRotation' should not return any type except void linguistically |
3 |
293–296 |
org/apache/commons/geometry/euclidean/twod/Bounds2D.java
Rule |
Violation |
Priority |
Line |
AccessorClassGeneration |
Avoid instantiation through private constructors from outside of the constructors class. |
3 |
176 |
AccessorClassGeneration |
Avoid instantiation through private constructors from outside of the constructors class. |
3 |
268 |
org/apache/commons/geometry/euclidean/twod/EmbeddedTreeLineSubset.java
Rule |
Violation |
Priority |
Line |
PrematureDeclaration |
Avoid declaring a variable if it is unreferenced before a possible exit point. |
3 |
173 |
InsufficientStringBufferDeclaration |
StringBuilder has been initialized with size 16, but has at least 41 characters appended. |
3 |
233 |
ConsecutiveLiteralAppends |
StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. |
3 |
235 |
org/apache/commons/geometry/euclidean/twod/Line.java
Rule |
Violation |
Priority |
Line |
ShortClassName |
Avoid short class names like Line |
4 |
59–545 |
org/apache/commons/geometry/euclidean/twod/RegionBSPTree2D.java
Rule |
Violation |
Priority |
Line |
LinguisticNaming |
Linguistics Antipattern - The transform method 'toConvexRecursive' should not return void linguistically |
3 |
131–145 |
NullAssignment |
Assigning an Object to null is a code smell. Consider refactoring. |
3 |
265 |
AccessorClassGeneration |
Avoid instantiation through private constructors from outside of the constructors class. |
3 |
271 |
AccessorClassGeneration |
Avoid instantiation through private constructors from outside of the constructors class. |
3 |
317 |
AssignmentInOperand |
Avoid assignments in operands |
3 |
341 |
org/apache/commons/geometry/euclidean/twod/ReverseRay.java
Rule |
Violation |
Priority |
Line |
InsufficientStringBufferDeclaration |
StringBuilder has been initialized with size 16, but has at least 25 characters appended. |
3 |
150 |
org/apache/commons/geometry/euclidean/twod/Segment.java
Rule |
Violation |
Priority |
Line |
InsufficientStringBufferDeclaration |
StringBuilder has been initialized with size 16, but has at least 26 characters appended. |
3 |
145 |
PrematureDeclaration |
Avoid declaring a variable if it is unreferenced before a possible exit point. |
3 |
184 |
org/apache/commons/geometry/euclidean/twod/rotation/Rotation2D.java
Rule |
Violation |
Priority |
Line |
AvoidFieldNameMatchingMethodName |
Field IDENTITY has the same name as a method |
3 |
30 |
org/apache/commons/geometry/euclidean/twod/shape/Parallelogram.java
Rule |
Violation |
Priority |
Line |
AccessorClassGeneration |
Avoid instantiation through private constructors from outside of the constructors class. |
3 |
161 |
LinguisticNaming |
Linguistics Antipattern - The setter 'setPosition' should not return any type except void linguistically |
3 |
196–199 |
LinguisticNaming |
Linguistics Antipattern - The setter 'setScale' should not return any type except void linguistically |
3 |
207–210 |
LinguisticNaming |
Linguistics Antipattern - The setter 'setScale' should not return any type except void linguistically |
3 |
219–221 |
LinguisticNaming |
Linguistics Antipattern - The setter 'setScale' should not return any type except void linguistically |
3 |
228–230 |
LinguisticNaming |
Linguistics Antipattern - The setter 'setRotation' should not return any type except void linguistically |
3 |
236–239 |
LinguisticNaming |
Linguistics Antipattern - The setter 'setXDirection' should not return any type except void linguistically |
3 |
248–251 |
LinguisticNaming |
Linguistics Antipattern - The setter 'setYDirection' should not return any type except void linguistically |
3 |
260–263 |
|