PMD Results
The following document contains the results of PMD 7.19.0.
Violations By Priority
Priority 3
dev/metaschema/oscal/lib/OscalBindingContext.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryWarningSuppression | Unnecessary @SuppressWarnings annotation | 96 |
dev/metaschema/oscal/lib/OscalModelConstants.java
| Rule | Violation | Line |
|---|---|---|
| DataClass | The class 'OscalModelConstants' is suspected to be a Data Class (WOC=0.000%, NOPA=17, NOAM=0, WMC=1) | 11 |
dev/metaschema/oscal/lib/metapath/function/library/HasOscalNamespace.java
| Rule | Violation | Line |
|---|---|---|
| OnlyOneReturn | A method should have only one exit point, and that should be the last statement in the method | 200 |
dev/metaschema/oscal/lib/model/control/AbstractParameter.java
| Rule | Violation | Line |
|---|---|---|
| CyclomaticComplexity | The method 'build()' has a cyclomatic complexity of 11. | 160 |
| NPathComplexity | The method 'build()' has an NPath complexity of 1024, current threshold is 200 | 160 |
dev/metaschema/oscal/lib/model/control/AbstractPart.java
| Rule | Violation | Line |
|---|---|---|
| NPathComplexity | The method 'build()' has an NPath complexity of 256, current threshold is 200 | 132 |
dev/metaschema/oscal/lib/model/control/catalog/AbstractCatalogGroup.java
| Rule | Violation | Line |
|---|---|---|
| CyclomaticComplexity | The method 'build()' has a cyclomatic complexity of 10. | 128 |
dev/metaschema/oscal/lib/model/control/catalog/AbstractControl.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryWarningSuppression | Unnecessary PMD suppression comment | 40 |
dev/metaschema/oscal/lib/model/metadata/IProperty.java
| Rule | Violation | Line |
|---|---|---|
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 25 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 28 |
dev/metaschema/oscal/lib/model/util/AllowedValueCollectingNodeItemVisitor.java
| Rule | Violation | Line |
|---|---|---|
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 33 |
dev/metaschema/oscal/lib/profile/resolver/ProfileResolver.java
| Rule | Violation | Line |
|---|---|---|
| CouplingBetweenObjects | A value of 42 may denote a high amount of coupling within the class (threshold: 20) | 6–757 |
| GodClass | Possible God Class (WMC=74, ATFD=109, TCC=0.923%) | 83 |
| LooseCoupling | Avoid using implementation types like 'Stack'; use the interface instead | 205 |
| LooseCoupling | Avoid using implementation types like 'Stack'; use the interface instead | 221 |
| LooseCoupling | Avoid using implementation types like 'Stack'; use the interface instead | 252 |
| LooseCoupling | Avoid using implementation types like 'Stack'; use the interface instead | 312 |
| LooseCoupling | Avoid using implementation types like 'Stack'; use the interface instead | 340 |
| LooseCoupling | Avoid using implementation types like 'Stack'; use the interface instead | 465 |
| LooseCoupling | Avoid using implementation types like 'Stack'; use the interface instead | 475 |
| ExhaustiveSwitchHasDefault | The switch block is exhaustive even without the default case | 512–522 |
dev/metaschema/oscal/lib/profile/resolver/alter/AddVisitor.java
| Rule | Violation | Line |
|---|---|---|
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 299–303 |
| EnumComparison | Enums should be compared using == | 299 |
| NPathComplexity | The method 'handleChild(TargetType, Supplier<? extends List<T>>, Supplier<? extends List<T>>, Function<T, Boolean>, Context)' has an NPath complexity of 358, current threshold is 200 | 329 |
| ExhaustiveSwitchHasDefault | The switch block is exhaustive even without the default case | 353–373 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 386–392 |
| EnumComparison | Enums should be compared using == | 386 |
| EnumComparison | Enums should be compared using == | 389 |
| EnumComparison | Enums should be compared using == | 558 |
| EnumComparison | Enums should be compared using == | 558 |
| EnumComparison | Enums should be compared using == | 663 |
| EnumComparison | Enums should be compared using == | 663 |
| EnumComparison | Enums should be compared using == | 664 |
| EnumComparison | Enums should be compared using == | 665 |
| ExhaustiveSwitchHasDefault | The switch block is exhaustive even without the default case | 684–705 |
| EnumComparison | Enums should be compared using == | 708 |
dev/metaschema/oscal/lib/profile/resolver/alter/RemoveVisitor.java
| Rule | Violation | Line |
|---|---|---|
| CognitiveComplexity | The method 'handle(TargetType, Supplier<? extends Collection<T>>, Function<T, Boolean>, Context)' has a cognitive complexity of 18, current threshold is 15 | 142 |
| CyclomaticComplexity | The method 'handle(TargetType, Supplier<? extends Collection<T>>, Function<T, Boolean>, Context)' has a cyclomatic complexity of 11. | 142 |
| UseObjectForClearerAPI | Rather than using a lot of String arguments, consider using a container object for those values. | 194 |
| DataClass | The class 'Context' is suspected to be a Data Class (WOC=28.571%, NOPA=0, NOAM=5, WMC=24) | 306 |
| ExhaustiveSwitchHasDefault | The switch block is exhaustive even without the default case | 441–471 |
dev/metaschema/oscal/lib/profile/resolver/merge/FlatteningStructuringVisitor.java
| Rule | Violation | Line |
|---|---|---|
| EnumComparison | Enums should be compared using == | 223 |
| EnumComparison | Enums should be compared using == | 232 |
| EnumComparison | Enums should be compared using == | 238 |
dev/metaschema/oscal/lib/profile/resolver/policy/AbstractCustomReferencePolicy.java
| Rule | Violation | Line |
|---|---|---|
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 84–87 |
| UnnecessaryWarningSuppression | Unnecessary PMD suppression comment | 112 |
| UnnecessaryWarningSuppression | Unnecessary PMD suppression comment | 137 |
dev/metaschema/oscal/lib/profile/resolver/policy/ReferenceCountingVisitor.java
| Rule | Violation | Line |
|---|---|---|
| CouplingBetweenObjects | A value of 31 may denote a high amount of coupling within the class (threshold: 20) | 6–646 |
| GodClass | Possible God Class (WMC=63, ATFD=41, TCC=3.333%) | 53 |
| EnumComparison | Enums should be compared using == | 207 |
| EnumComparison | Enums should be compared using == | 241 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 251–253 |
| EnumComparison | Enums should be compared using == | 251 |
| EnumComparison | Enums should be compared using == | 304 |
| EnumComparison | Enums should be compared using == | 510–511 |
| CyclomaticComplexity | The method 'entityDispatch(IEntityItem, Context)' has a cyclomatic complexity of 10. | 526 |
| ExhaustiveSwitchHasDefault | The switch block is exhaustive even without the default case | 528–555 |
dev/metaschema/oscal/lib/profile/resolver/selection/ControlSelectionState.java
| Rule | Violation | Line |
|---|---|---|
| CognitiveComplexity | The method 'getSelectionState(IModelNodeItem<?, ?>)' has a cognitive complexity of 15, current threshold is 15 | 60 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 74–77 |
dev/metaschema/oscal/lib/profile/resolver/selection/DefaultControlSelectionFilter.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryWarningSuppression | Unnecessary @SuppressWarnings annotation | 91 |
dev/metaschema/oscal/lib/profile/resolver/selection/FilterNonSelectedVisitor.java
| Rule | Violation | Line |
|---|---|---|
| EnumComparison | Enums should be compared using == | 155 |
| EnumComparison | Enums should be compared using == | 192 |
| EnumComparison | Enums should be compared using == | 197 |
| EnumComparison | Enums should be compared using == | 205 |
| EnumComparison | Enums should be compared using == | 252 |
| EnumComparison | Enums should be compared using == | 257 |
dev/metaschema/oscal/lib/profile/resolver/support/AbstractCatalogEntityVisitor.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryWarningSuppression | Unnecessary PMD suppression comment | 195 |
| UnnecessaryWarningSuppression | Unnecessary PMD suppression comment | 255 |
| UnnecessaryWarningSuppression | Unnecessary PMD suppression comment | 277 |
| UnnecessaryWarningSuppression | Unnecessary PMD suppression comment | 297 |
| UnnecessaryWarningSuppression | Unnecessary PMD suppression comment | 339 |
dev/metaschema/oscal/lib/profile/resolver/support/BasicIndexer.java
| Rule | Violation | Line |
|---|---|---|
| CouplingBetweenObjects | A value of 22 may denote a high amount of coupling within the class (threshold: 20) | 6–377 |
| GodClass | Possible God Class (WMC=50, ATFD=38, TCC=9.881%) | 41 |
| UnnecessaryWarningSuppression | Unnecessary @SuppressWarnings annotation | 70 |
| CyclomaticComplexity | The method 'isSelected(IEntityItem)' has a cyclomatic complexity of 11. | 101 |
| ExhaustiveSwitchHasDefault | The switch block is exhaustive even without the default case | 103–133 |
| EnumComparison | Enums should be compared using == | 106 |
| EnumComparison | Enums should be compared using == | 111 |
| EnumComparison | Enums should be compared using == | 120 |
| UnnecessaryWarningSuppression | Unnecessary @SuppressWarnings annotation | 194 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 228–230 |
dev/metaschema/oscal/lib/profile/resolver/support/IIdentifierMapper.java
| Rule | Violation | Line |
|---|---|---|
| CyclomaticComplexity | The method 'mapByItemType(ItemType, String)' has a cyclomatic complexity of 10. | 57 |
| ExhaustiveSwitchHasDefault | The switch block is exhaustive even without the default case | 61–84 |
dev/metaschema/oscal/lib/profile/resolver/support/IIndexer.java
| Rule | Violation | Line |
|---|---|---|
| CognitiveComplexity | The method 'logIndex(IIndexer, Level)' has a cognitive complexity of 16, current threshold is 15 | 112 |
Priority 4
dev/metaschema/oscal/lib/model/control/catalog/AbstractCatalogVisitor.java
| Rule | Violation | Line |
|---|---|---|
| TypeParameterNamingConventions | The type parameter name 'RESULT' doesn't match '[A-Z]' | 18 |
| TypeParameterNamingConventions | The type parameter name 'CONTEXT' doesn't match '[A-Z]' | 18 |
dev/metaschema/oscal/lib/model/control/catalog/ICatalogVisitor.java
| Rule | Violation | Line |
|---|---|---|
| TypeParameterNamingConventions | The type parameter name 'RESULT' doesn't match '[A-Z]' | 14 |
| TypeParameterNamingConventions | The type parameter name 'CONTEXT' doesn't match '[A-Z]' | 14 |
dev/metaschema/oscal/lib/profile/resolver/policy/AbstractCustomReferencePolicy.java
| Rule | Violation | Line |
|---|---|---|
| TypeParameterNamingConventions | The type parameter name 'TYPE' doesn't match '[A-Z]' | 20 |
dev/metaschema/oscal/lib/profile/resolver/policy/AbstractIndexMissPolicyHandler.java
| Rule | Violation | Line |
|---|---|---|
| TypeParameterNamingConventions | The type parameter name 'TYPE' doesn't match '[A-Z]' | 13 |
dev/metaschema/oscal/lib/profile/resolver/policy/AbstractMultiItemTypeReferencePolicy.java
| Rule | Violation | Line |
|---|---|---|
| TypeParameterNamingConventions | The type parameter name 'TYPE' doesn't match '[A-Z]' | 14 |
dev/metaschema/oscal/lib/profile/resolver/policy/ICustomReferencePolicy.java
| Rule | Violation | Line |
|---|---|---|
| TypeParameterNamingConventions | The type parameter name 'TYPE' doesn't match '[A-Z]' | 10 |
dev/metaschema/oscal/lib/profile/resolver/policy/ICustomReferencePolicyHandler.java
| Rule | Violation | Line |
|---|---|---|
| TypeParameterNamingConventions | The type parameter name 'TYPE' doesn't match '[A-Z]' | 13 |
Files
dev/metaschema/oscal/lib/OscalBindingContext.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryWarningSuppression | Unnecessary @SuppressWarnings annotation | 3 | 96 |
dev/metaschema/oscal/lib/OscalModelConstants.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| DataClass | The class 'OscalModelConstants' is suspected to be a Data Class (WOC=0.000%, NOPA=17, NOAM=0, WMC=1) | 3 | 11 |
dev/metaschema/oscal/lib/metapath/function/library/HasOscalNamespace.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| OnlyOneReturn | A method should have only one exit point, and that should be the last statement in the method | 3 | 200 |
dev/metaschema/oscal/lib/model/control/AbstractParameter.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CyclomaticComplexity | The method 'build()' has a cyclomatic complexity of 11. | 3 | 160 |
| NPathComplexity | The method 'build()' has an NPath complexity of 1024, current threshold is 200 | 3 | 160 |
dev/metaschema/oscal/lib/model/control/AbstractPart.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| NPathComplexity | The method 'build()' has an NPath complexity of 256, current threshold is 200 | 3 | 132 |
dev/metaschema/oscal/lib/model/control/catalog/AbstractCatalogGroup.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CyclomaticComplexity | The method 'build()' has a cyclomatic complexity of 10. | 3 | 128 |
dev/metaschema/oscal/lib/model/control/catalog/AbstractCatalogVisitor.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TypeParameterNamingConventions | The type parameter name 'RESULT' doesn't match '[A-Z]' | 4 | 18 |
| TypeParameterNamingConventions | The type parameter name 'CONTEXT' doesn't match '[A-Z]' | 4 | 18 |
dev/metaschema/oscal/lib/model/control/catalog/AbstractControl.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryWarningSuppression | Unnecessary PMD suppression comment | 3 | 40 |
dev/metaschema/oscal/lib/model/control/catalog/ICatalogVisitor.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TypeParameterNamingConventions | The type parameter name 'RESULT' doesn't match '[A-Z]' | 4 | 14 |
| TypeParameterNamingConventions | The type parameter name 'CONTEXT' doesn't match '[A-Z]' | 4 | 14 |
dev/metaschema/oscal/lib/model/metadata/IProperty.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 3 | 25 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 3 | 28 |
dev/metaschema/oscal/lib/model/util/AllowedValueCollectingNodeItemVisitor.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 33 |
dev/metaschema/oscal/lib/profile/resolver/ProfileResolver.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CouplingBetweenObjects | A value of 42 may denote a high amount of coupling within the class (threshold: 20) | 3 | 6–757 |
| GodClass | Possible God Class (WMC=74, ATFD=109, TCC=0.923%) | 3 | 83 |
| LooseCoupling | Avoid using implementation types like 'Stack'; use the interface instead | 3 | 205 |
| LooseCoupling | Avoid using implementation types like 'Stack'; use the interface instead | 3 | 221 |
| LooseCoupling | Avoid using implementation types like 'Stack'; use the interface instead | 3 | 252 |
| LooseCoupling | Avoid using implementation types like 'Stack'; use the interface instead | 3 | 312 |
| LooseCoupling | Avoid using implementation types like 'Stack'; use the interface instead | 3 | 340 |
| LooseCoupling | Avoid using implementation types like 'Stack'; use the interface instead | 3 | 465 |
| LooseCoupling | Avoid using implementation types like 'Stack'; use the interface instead | 3 | 475 |
| ExhaustiveSwitchHasDefault | The switch block is exhaustive even without the default case | 3 | 512–522 |
dev/metaschema/oscal/lib/profile/resolver/alter/AddVisitor.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 3 | 299–303 |
| EnumComparison | Enums should be compared using == | 3 | 299 |
| NPathComplexity | The method 'handleChild(TargetType, Supplier<? extends List<T>>, Supplier<? extends List<T>>, Function<T, Boolean>, Context)' has an NPath complexity of 358, current threshold is 200 | 3 | 329 |
| ExhaustiveSwitchHasDefault | The switch block is exhaustive even without the default case | 3 | 353–373 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 3 | 386–392 |
| EnumComparison | Enums should be compared using == | 3 | 386 |
| EnumComparison | Enums should be compared using == | 3 | 389 |
| EnumComparison | Enums should be compared using == | 3 | 558 |
| EnumComparison | Enums should be compared using == | 3 | 558 |
| EnumComparison | Enums should be compared using == | 3 | 663 |
| EnumComparison | Enums should be compared using == | 3 | 663 |
| EnumComparison | Enums should be compared using == | 3 | 664 |
| EnumComparison | Enums should be compared using == | 3 | 665 |
| ExhaustiveSwitchHasDefault | The switch block is exhaustive even without the default case | 3 | 684–705 |
| EnumComparison | Enums should be compared using == | 3 | 708 |
dev/metaschema/oscal/lib/profile/resolver/alter/RemoveVisitor.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CognitiveComplexity | The method 'handle(TargetType, Supplier<? extends Collection<T>>, Function<T, Boolean>, Context)' has a cognitive complexity of 18, current threshold is 15 | 3 | 142 |
| CyclomaticComplexity | The method 'handle(TargetType, Supplier<? extends Collection<T>>, Function<T, Boolean>, Context)' has a cyclomatic complexity of 11. | 3 | 142 |
| UseObjectForClearerAPI | Rather than using a lot of String arguments, consider using a container object for those values. | 3 | 194 |
| DataClass | The class 'Context' is suspected to be a Data Class (WOC=28.571%, NOPA=0, NOAM=5, WMC=24) | 3 | 306 |
| ExhaustiveSwitchHasDefault | The switch block is exhaustive even without the default case | 3 | 441–471 |
dev/metaschema/oscal/lib/profile/resolver/merge/FlatteningStructuringVisitor.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| EnumComparison | Enums should be compared using == | 3 | 223 |
| EnumComparison | Enums should be compared using == | 3 | 232 |
| EnumComparison | Enums should be compared using == | 3 | 238 |
dev/metaschema/oscal/lib/profile/resolver/policy/AbstractCustomReferencePolicy.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TypeParameterNamingConventions | The type parameter name 'TYPE' doesn't match '[A-Z]' | 4 | 20 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 3 | 84–87 |
| UnnecessaryWarningSuppression | Unnecessary PMD suppression comment | 3 | 112 |
| UnnecessaryWarningSuppression | Unnecessary PMD suppression comment | 3 | 137 |
dev/metaschema/oscal/lib/profile/resolver/policy/AbstractIndexMissPolicyHandler.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TypeParameterNamingConventions | The type parameter name 'TYPE' doesn't match '[A-Z]' | 4 | 13 |
dev/metaschema/oscal/lib/profile/resolver/policy/AbstractMultiItemTypeReferencePolicy.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TypeParameterNamingConventions | The type parameter name 'TYPE' doesn't match '[A-Z]' | 4 | 14 |
dev/metaschema/oscal/lib/profile/resolver/policy/ICustomReferencePolicy.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TypeParameterNamingConventions | The type parameter name 'TYPE' doesn't match '[A-Z]' | 4 | 10 |
dev/metaschema/oscal/lib/profile/resolver/policy/ICustomReferencePolicyHandler.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TypeParameterNamingConventions | The type parameter name 'TYPE' doesn't match '[A-Z]' | 4 | 13 |
dev/metaschema/oscal/lib/profile/resolver/policy/ReferenceCountingVisitor.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CouplingBetweenObjects | A value of 31 may denote a high amount of coupling within the class (threshold: 20) | 3 | 6–646 |
| GodClass | Possible God Class (WMC=63, ATFD=41, TCC=3.333%) | 3 | 53 |
| EnumComparison | Enums should be compared using == | 3 | 207 |
| EnumComparison | Enums should be compared using == | 3 | 241 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 3 | 251–253 |
| EnumComparison | Enums should be compared using == | 3 | 251 |
| EnumComparison | Enums should be compared using == | 3 | 304 |
| EnumComparison | Enums should be compared using == | 3 | 510–511 |
| CyclomaticComplexity | The method 'entityDispatch(IEntityItem, Context)' has a cyclomatic complexity of 10. | 3 | 526 |
| ExhaustiveSwitchHasDefault | The switch block is exhaustive even without the default case | 3 | 528–555 |
dev/metaschema/oscal/lib/profile/resolver/selection/ControlSelectionState.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CognitiveComplexity | The method 'getSelectionState(IModelNodeItem<?, ?>)' has a cognitive complexity of 15, current threshold is 15 | 3 | 60 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 3 | 74–77 |
dev/metaschema/oscal/lib/profile/resolver/selection/DefaultControlSelectionFilter.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryWarningSuppression | Unnecessary @SuppressWarnings annotation | 3 | 91 |
dev/metaschema/oscal/lib/profile/resolver/selection/FilterNonSelectedVisitor.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| EnumComparison | Enums should be compared using == | 3 | 155 |
| EnumComparison | Enums should be compared using == | 3 | 192 |
| EnumComparison | Enums should be compared using == | 3 | 197 |
| EnumComparison | Enums should be compared using == | 3 | 205 |
| EnumComparison | Enums should be compared using == | 3 | 252 |
| EnumComparison | Enums should be compared using == | 3 | 257 |
dev/metaschema/oscal/lib/profile/resolver/support/AbstractCatalogEntityVisitor.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryWarningSuppression | Unnecessary PMD suppression comment | 3 | 195 |
| UnnecessaryWarningSuppression | Unnecessary PMD suppression comment | 3 | 255 |
| UnnecessaryWarningSuppression | Unnecessary PMD suppression comment | 3 | 277 |
| UnnecessaryWarningSuppression | Unnecessary PMD suppression comment | 3 | 297 |
| UnnecessaryWarningSuppression | Unnecessary PMD suppression comment | 3 | 339 |
dev/metaschema/oscal/lib/profile/resolver/support/BasicIndexer.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CouplingBetweenObjects | A value of 22 may denote a high amount of coupling within the class (threshold: 20) | 3 | 6–377 |
| GodClass | Possible God Class (WMC=50, ATFD=38, TCC=9.881%) | 3 | 41 |
| UnnecessaryWarningSuppression | Unnecessary @SuppressWarnings annotation | 3 | 70 |
| CyclomaticComplexity | The method 'isSelected(IEntityItem)' has a cyclomatic complexity of 11. | 3 | 101 |
| ExhaustiveSwitchHasDefault | The switch block is exhaustive even without the default case | 3 | 103–133 |
| EnumComparison | Enums should be compared using == | 3 | 106 |
| EnumComparison | Enums should be compared using == | 3 | 111 |
| EnumComparison | Enums should be compared using == | 3 | 120 |
| UnnecessaryWarningSuppression | Unnecessary @SuppressWarnings annotation | 3 | 194 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 3 | 228–230 |
dev/metaschema/oscal/lib/profile/resolver/support/IIdentifierMapper.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CyclomaticComplexity | The method 'mapByItemType(ItemType, String)' has a cyclomatic complexity of 10. | 3 | 57 |
| ExhaustiveSwitchHasDefault | The switch block is exhaustive even without the default case | 3 | 61–84 |
dev/metaschema/oscal/lib/profile/resolver/support/IIndexer.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CognitiveComplexity | The method 'logIndex(IIndexer, Level)' has a cognitive complexity of 16, current threshold is 15 | 3 | 112 |
Suppressed Violations
| Filename | Rule message | Suppression type | Reason |
|---|---|---|---|
| dev/metaschema/oscal/lib/OscalUtils.java | A method should have only one exit point, and that should be the last statement in the method | @suppresswarnings | |
| dev/metaschema/oscal/lib/metapath/function/library/HasOscalNamespace.java | Linguistics Antipattern - The method 'hasNamespace' indicates linguistically it returns a boolean, but it returns 'IBooleanItem' | @suppresswarnings | |
| dev/metaschema/oscal/lib/metapath/function/library/HasOscalNamespace.java | A method should have only one exit point, and that should be the last statement in the method | @suppresswarnings | |
| dev/metaschema/oscal/lib/metapath/function/library/HasOscalNamespace.java | A method should have only one exit point, and that should be the last statement in the method | @suppresswarnings | |
| dev/metaschema/oscal/lib/metapath/function/library/ResolveProfile.java | A method should have only one exit point, and that should be the last statement in the method | @suppresswarnings | |
| dev/metaschema/oscal/lib/metapath/function/library/ResolveProfile.java | A method should have only one exit point, and that should be the last statement in the method | @suppresswarnings | |
| dev/metaschema/oscal/lib/metapath/function/library/ResolveReference.java | Avoid unused method parameters such as 'function'. | @suppresswarnings | |
| dev/metaschema/oscal/lib/metapath/function/library/ResolveReference.java | Avoid unused method parameters such as 'dynamicContext'. | @suppresswarnings | |
| dev/metaschema/oscal/lib/metapath/function/library/ResolveReference.java | Avoid unused method parameters such as 'function'. | @suppresswarnings | |
| dev/metaschema/oscal/lib/metapath/function/library/ResolveReference.java | Avoid unused method parameters such as 'dynamicContext'. | @suppresswarnings | |
| dev/metaschema/oscal/lib/metapath/function/library/ResolveReference.java | A method should have only one exit point, and that should be the last statement in the method | @suppresswarnings | |
| dev/metaschema/oscal/lib/metapath/function/library/ResolveReference.java | A method should have only one exit point, and that should be the last statement in the method | @suppresswarnings | |
| dev/metaschema/oscal/lib/model/control/AbstractPart.java | Avoid using short method names | @suppresswarnings | |
| dev/metaschema/oscal/lib/model/metadata/AbstractProperty.java | Assigning an Object to null is a code smell. Consider refactoring. | @suppresswarnings | |
| dev/metaschema/oscal/lib/profile/resolver/ModifyPhaseUtils.java | A method should have only one exit point, and that should be the last statement in the method | @suppresswarnings | |
| dev/metaschema/oscal/lib/profile/resolver/ModifyPhaseUtils.java | A method should have only one exit point, and that should be the last statement in the method | @suppresswarnings | |
| dev/metaschema/oscal/lib/profile/resolver/ModifyPhaseUtils.java | A method should have only one exit point, and that should be the last statement in the method | @suppresswarnings | |
| dev/metaschema/oscal/lib/profile/resolver/ProfileResolver.java | Exception thrown at line 582 is caught in this block. | @suppresswarnings | |
| dev/metaschema/oscal/lib/profile/resolver/ProfileResolver.java | Exception thrown at line 685 is caught in this block. | @suppresswarnings | |
| dev/metaschema/oscal/lib/profile/resolver/ProfileResolver.java | Exception thrown at line 657 is caught in this block. | @suppresswarnings | |
| dev/metaschema/oscal/lib/profile/resolver/alter/AddVisitor.java | The method 'newContext(Control, Position, String, MarkupLine, List<Parameter>, List<Property>, List<Link>, List<ControlPart>)' has a cognitive complexity of 16, current threshold is 15 | @suppresswarnings | |
| dev/metaschema/oscal/lib/profile/resolver/alter/AddVisitor.java | The method 'handleChild(TargetType, Supplier<? extends List<T>>, Supplier<? extends List<T>>, Function<T, Boolean>, Context)' has a cognitive complexity of 23, current threshold is 15 | @suppresswarnings | |
| dev/metaschema/oscal/lib/profile/resolver/alter/AddVisitor.java | A value of 33 may denote a high amount of coupling within the class (threshold: 20) | @suppresswarnings | |
| dev/metaschema/oscal/lib/profile/resolver/alter/AddVisitor.java | The method 'newContext(Control, Position, String, MarkupLine, List<Parameter>, List<Property>, List<Link>, List<ControlPart>)' has a cyclomatic complexity of 17. | @suppresswarnings | |
| dev/metaschema/oscal/lib/profile/resolver/alter/AddVisitor.java | The method 'handleChild(TargetType, Supplier<? extends List<T>>, Supplier<? extends List<T>>, Function<T, Boolean>, Context)' has a cyclomatic complexity of 16. | @suppresswarnings | |
| dev/metaschema/oscal/lib/profile/resolver/alter/AddVisitor.java | The method 'newContext(Control, Position, String, MarkupLine, List<Parameter>, List<Property>, List<Link>, List<ControlPart>)' has an NPath complexity of 768, current threshold is 200 | @suppresswarnings | |
| dev/metaschema/oscal/lib/profile/resolver/selection/Import.java | Deeply nested if..then statements are hard to read | @suppresswarnings |