PMD Results
The following document contains the results of PMD 7.19.0.
Violations By Priority
Priority 3
gov/nist/secauto/oscal/lib/OscalBindingContext.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryWarningSuppression | Unnecessary @SuppressWarnings annotation | 97 |
gov/nist/secauto/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) | 12 |
gov/nist/secauto/oscal/lib/model/control/AbstractParameter.java
| Rule | Violation | Line |
|---|---|---|
| CyclomaticComplexity | The method 'build()' has a cyclomatic complexity of 11. | 161 |
| NPathComplexity | The method 'build()' has an NPath complexity of 1024, current threshold is 200 | 161 |
gov/nist/secauto/oscal/lib/model/control/AbstractPart.java
| Rule | Violation | Line |
|---|---|---|
| NPathComplexity | The method 'build()' has an NPath complexity of 256, current threshold is 200 | 133 |
gov/nist/secauto/oscal/lib/model/control/catalog/AbstractCatalogGroup.java
| Rule | Violation | Line |
|---|---|---|
| CyclomaticComplexity | The method 'build()' has a cyclomatic complexity of 10. | 129 |
gov/nist/secauto/oscal/lib/model/control/catalog/AbstractControl.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryWarningSuppression | Unnecessary PMD suppression comment | 41 |
gov/nist/secauto/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 |
gov/nist/secauto/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 | 34 |
gov/nist/secauto/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–758 |
| GodClass | Possible God Class (WMC=75, ATFD=110, TCC=0.923%) | 84 |
| LooseCoupling | Avoid using implementation types like 'Stack'; use the interface instead | 206 |
| LooseCoupling | Avoid using implementation types like 'Stack'; use the interface instead | 222 |
| LooseCoupling | Avoid using implementation types like 'Stack'; use the interface instead | 253 |
| LooseCoupling | Avoid using implementation types like 'Stack'; use the interface instead | 313 |
| LooseCoupling | Avoid using implementation types like 'Stack'; use the interface instead | 341 |
| LooseCoupling | Avoid using implementation types like 'Stack'; use the interface instead | 466 |
| LooseCoupling | Avoid using implementation types like 'Stack'; use the interface instead | 476 |
| ExhaustiveSwitchHasDefault | The switch block is exhaustive even without the default case | 513–523 |
gov/nist/secauto/oscal/lib/profile/resolver/alter/AddVisitor.java
| Rule | Violation | Line |
|---|---|---|
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 300–304 |
| EnumComparison | Enums should be compared using == | 300 |
| 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 | 330 |
| ExhaustiveSwitchHasDefault | The switch block is exhaustive even without the default case | 354–374 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 387–393 |
| EnumComparison | Enums should be compared using == | 387 |
| EnumComparison | Enums should be compared using == | 390 |
| NPathComplexity | The method 'visitControl(Control, Context)' has an NPath complexity of 256, current threshold is 200 | 400 |
| EnumComparison | Enums should be compared using == | 593 |
| EnumComparison | Enums should be compared using == | 593 |
| EnumComparison | Enums should be compared using == | 698 |
| EnumComparison | Enums should be compared using == | 698 |
| EnumComparison | Enums should be compared using == | 699 |
| EnumComparison | Enums should be compared using == | 700 |
| ExhaustiveSwitchHasDefault | The switch block is exhaustive even without the default case | 719–740 |
| EnumComparison | Enums should be compared using == | 743 |
gov/nist/secauto/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 | 143 |
| CyclomaticComplexity | The method 'handle(TargetType, Supplier<? extends Collection<T>>, Function<T, Boolean>, Context)' has a cyclomatic complexity of 11. | 143 |
| UseObjectForClearerAPI | Rather than using a lot of String arguments, consider using a container object for those values. | 195 |
| DataClass | The class 'Context' is suspected to be a Data Class (WOC=28.571%, NOPA=0, NOAM=5, WMC=24) | 307 |
| ExhaustiveSwitchHasDefault | The switch block is exhaustive even without the default case | 442–472 |
gov/nist/secauto/oscal/lib/profile/resolver/merge/FlatteningStructuringVisitor.java
| Rule | Violation | Line |
|---|---|---|
| EnumComparison | Enums should be compared using == | 224 |
| EnumComparison | Enums should be compared using == | 233 |
| EnumComparison | Enums should be compared using == | 239 |
gov/nist/secauto/oscal/lib/profile/resolver/policy/AbstractCustomReferencePolicy.java
| Rule | Violation | Line |
|---|---|---|
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 85–88 |
| UnnecessaryWarningSuppression | Unnecessary PMD suppression comment | 113 |
| UnnecessaryWarningSuppression | Unnecessary PMD suppression comment | 138 |
gov/nist/secauto/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–647 |
| GodClass | Possible God Class (WMC=63, ATFD=41, TCC=3.333%) | 54 |
| EnumComparison | Enums should be compared using == | 208 |
| EnumComparison | Enums should be compared using == | 242 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 252–254 |
| EnumComparison | Enums should be compared using == | 252 |
| EnumComparison | Enums should be compared using == | 305 |
| EnumComparison | Enums should be compared using == | 511–512 |
| CyclomaticComplexity | The method 'entityDispatch(IEntityItem, Context)' has a cyclomatic complexity of 10. | 527 |
| ExhaustiveSwitchHasDefault | The switch block is exhaustive even without the default case | 529–556 |
gov/nist/secauto/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 | 61 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 75–78 |
gov/nist/secauto/oscal/lib/profile/resolver/selection/DefaultControlSelectionFilter.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryWarningSuppression | Unnecessary @SuppressWarnings annotation | 92 |
gov/nist/secauto/oscal/lib/profile/resolver/selection/FilterNonSelectedVisitor.java
| Rule | Violation | Line |
|---|---|---|
| EnumComparison | Enums should be compared using == | 156 |
| EnumComparison | Enums should be compared using == | 193 |
| EnumComparison | Enums should be compared using == | 198 |
| EnumComparison | Enums should be compared using == | 206 |
| EnumComparison | Enums should be compared using == | 253 |
| EnumComparison | Enums should be compared using == | 258 |
gov/nist/secauto/oscal/lib/profile/resolver/support/AbstractCatalogEntityVisitor.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryWarningSuppression | Unnecessary PMD suppression comment | 196 |
| UnnecessaryWarningSuppression | Unnecessary PMD suppression comment | 256 |
| UnnecessaryWarningSuppression | Unnecessary PMD suppression comment | 278 |
| UnnecessaryWarningSuppression | Unnecessary PMD suppression comment | 298 |
| UnnecessaryWarningSuppression | Unnecessary PMD suppression comment | 340 |
gov/nist/secauto/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–378 |
| GodClass | Possible God Class (WMC=50, ATFD=38, TCC=9.881%) | 42 |
| UnnecessaryWarningSuppression | Unnecessary @SuppressWarnings annotation | 71 |
| CyclomaticComplexity | The method 'isSelected(IEntityItem)' has a cyclomatic complexity of 11. | 102 |
| ExhaustiveSwitchHasDefault | The switch block is exhaustive even without the default case | 104–134 |
| EnumComparison | Enums should be compared using == | 107 |
| EnumComparison | Enums should be compared using == | 112 |
| EnumComparison | Enums should be compared using == | 121 |
| UnnecessaryWarningSuppression | Unnecessary @SuppressWarnings annotation | 195 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 229–231 |
gov/nist/secauto/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 |
gov/nist/secauto/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 | 113 |
Priority 4
gov/nist/secauto/oscal/lib/model/control/catalog/AbstractCatalogVisitor.java
| Rule | Violation | Line |
|---|---|---|
| TypeParameterNamingConventions | The type parameter name 'RESULT' doesn't match '[A-Z]' | 19 |
| TypeParameterNamingConventions | The type parameter name 'CONTEXT' doesn't match '[A-Z]' | 19 |
gov/nist/secauto/oscal/lib/model/control/catalog/ICatalogVisitor.java
| Rule | Violation | Line |
|---|---|---|
| TypeParameterNamingConventions | The type parameter name 'RESULT' doesn't match '[A-Z]' | 15 |
| TypeParameterNamingConventions | The type parameter name 'CONTEXT' doesn't match '[A-Z]' | 15 |
gov/nist/secauto/oscal/lib/profile/resolver/policy/AbstractCustomReferencePolicy.java
| Rule | Violation | Line |
|---|---|---|
| TypeParameterNamingConventions | The type parameter name 'TYPE' doesn't match '[A-Z]' | 21 |
gov/nist/secauto/oscal/lib/profile/resolver/policy/AbstractIndexMissPolicyHandler.java
| Rule | Violation | Line |
|---|---|---|
| TypeParameterNamingConventions | The type parameter name 'TYPE' doesn't match '[A-Z]' | 14 |
gov/nist/secauto/oscal/lib/profile/resolver/policy/AbstractMultiItemTypeReferencePolicy.java
| Rule | Violation | Line |
|---|---|---|
| TypeParameterNamingConventions | The type parameter name 'TYPE' doesn't match '[A-Z]' | 15 |
gov/nist/secauto/oscal/lib/profile/resolver/policy/ICustomReferencePolicy.java
| Rule | Violation | Line |
|---|---|---|
| TypeParameterNamingConventions | The type parameter name 'TYPE' doesn't match '[A-Z]' | 10 |
gov/nist/secauto/oscal/lib/profile/resolver/policy/ICustomReferencePolicyHandler.java
| Rule | Violation | Line |
|---|---|---|
| TypeParameterNamingConventions | The type parameter name 'TYPE' doesn't match '[A-Z]' | 14 |
Files
gov/nist/secauto/oscal/lib/OscalBindingContext.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryWarningSuppression | Unnecessary @SuppressWarnings annotation | 3 | 97 |
gov/nist/secauto/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 | 12 |
gov/nist/secauto/oscal/lib/model/control/AbstractParameter.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CyclomaticComplexity | The method 'build()' has a cyclomatic complexity of 11. | 3 | 161 |
| NPathComplexity | The method 'build()' has an NPath complexity of 1024, current threshold is 200 | 3 | 161 |
gov/nist/secauto/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 | 133 |
gov/nist/secauto/oscal/lib/model/control/catalog/AbstractCatalogGroup.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CyclomaticComplexity | The method 'build()' has a cyclomatic complexity of 10. | 3 | 129 |
gov/nist/secauto/oscal/lib/model/control/catalog/AbstractCatalogVisitor.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TypeParameterNamingConventions | The type parameter name 'RESULT' doesn't match '[A-Z]' | 4 | 19 |
| TypeParameterNamingConventions | The type parameter name 'CONTEXT' doesn't match '[A-Z]' | 4 | 19 |
gov/nist/secauto/oscal/lib/model/control/catalog/AbstractControl.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryWarningSuppression | Unnecessary PMD suppression comment | 3 | 41 |
gov/nist/secauto/oscal/lib/model/control/catalog/ICatalogVisitor.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TypeParameterNamingConventions | The type parameter name 'RESULT' doesn't match '[A-Z]' | 4 | 15 |
| TypeParameterNamingConventions | The type parameter name 'CONTEXT' doesn't match '[A-Z]' | 4 | 15 |
gov/nist/secauto/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 |
gov/nist/secauto/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 | 34 |
gov/nist/secauto/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–758 |
| GodClass | Possible God Class (WMC=75, ATFD=110, TCC=0.923%) | 3 | 84 |
| LooseCoupling | Avoid using implementation types like 'Stack'; use the interface instead | 3 | 206 |
| LooseCoupling | Avoid using implementation types like 'Stack'; use the interface instead | 3 | 222 |
| LooseCoupling | Avoid using implementation types like 'Stack'; use the interface instead | 3 | 253 |
| LooseCoupling | Avoid using implementation types like 'Stack'; use the interface instead | 3 | 313 |
| LooseCoupling | Avoid using implementation types like 'Stack'; use the interface instead | 3 | 341 |
| LooseCoupling | Avoid using implementation types like 'Stack'; use the interface instead | 3 | 466 |
| LooseCoupling | Avoid using implementation types like 'Stack'; use the interface instead | 3 | 476 |
| ExhaustiveSwitchHasDefault | The switch block is exhaustive even without the default case | 3 | 513–523 |
gov/nist/secauto/oscal/lib/profile/resolver/alter/AddVisitor.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 3 | 300–304 |
| EnumComparison | Enums should be compared using == | 3 | 300 |
| 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 | 330 |
| ExhaustiveSwitchHasDefault | The switch block is exhaustive even without the default case | 3 | 354–374 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 3 | 387–393 |
| EnumComparison | Enums should be compared using == | 3 | 387 |
| EnumComparison | Enums should be compared using == | 3 | 390 |
| NPathComplexity | The method 'visitControl(Control, Context)' has an NPath complexity of 256, current threshold is 200 | 3 | 400 |
| EnumComparison | Enums should be compared using == | 3 | 593 |
| EnumComparison | Enums should be compared using == | 3 | 593 |
| EnumComparison | Enums should be compared using == | 3 | 698 |
| EnumComparison | Enums should be compared using == | 3 | 698 |
| EnumComparison | Enums should be compared using == | 3 | 699 |
| EnumComparison | Enums should be compared using == | 3 | 700 |
| ExhaustiveSwitchHasDefault | The switch block is exhaustive even without the default case | 3 | 719–740 |
| EnumComparison | Enums should be compared using == | 3 | 743 |
gov/nist/secauto/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 | 143 |
| CyclomaticComplexity | The method 'handle(TargetType, Supplier<? extends Collection<T>>, Function<T, Boolean>, Context)' has a cyclomatic complexity of 11. | 3 | 143 |
| UseObjectForClearerAPI | Rather than using a lot of String arguments, consider using a container object for those values. | 3 | 195 |
| DataClass | The class 'Context' is suspected to be a Data Class (WOC=28.571%, NOPA=0, NOAM=5, WMC=24) | 3 | 307 |
| ExhaustiveSwitchHasDefault | The switch block is exhaustive even without the default case | 3 | 442–472 |
gov/nist/secauto/oscal/lib/profile/resolver/merge/FlatteningStructuringVisitor.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| EnumComparison | Enums should be compared using == | 3 | 224 |
| EnumComparison | Enums should be compared using == | 3 | 233 |
| EnumComparison | Enums should be compared using == | 3 | 239 |
gov/nist/secauto/oscal/lib/profile/resolver/policy/AbstractCustomReferencePolicy.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TypeParameterNamingConventions | The type parameter name 'TYPE' doesn't match '[A-Z]' | 4 | 21 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 3 | 85–88 |
| UnnecessaryWarningSuppression | Unnecessary PMD suppression comment | 3 | 113 |
| UnnecessaryWarningSuppression | Unnecessary PMD suppression comment | 3 | 138 |
gov/nist/secauto/oscal/lib/profile/resolver/policy/AbstractIndexMissPolicyHandler.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TypeParameterNamingConventions | The type parameter name 'TYPE' doesn't match '[A-Z]' | 4 | 14 |
gov/nist/secauto/oscal/lib/profile/resolver/policy/AbstractMultiItemTypeReferencePolicy.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TypeParameterNamingConventions | The type parameter name 'TYPE' doesn't match '[A-Z]' | 4 | 15 |
gov/nist/secauto/oscal/lib/profile/resolver/policy/ICustomReferencePolicy.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TypeParameterNamingConventions | The type parameter name 'TYPE' doesn't match '[A-Z]' | 4 | 10 |
gov/nist/secauto/oscal/lib/profile/resolver/policy/ICustomReferencePolicyHandler.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TypeParameterNamingConventions | The type parameter name 'TYPE' doesn't match '[A-Z]' | 4 | 14 |
gov/nist/secauto/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–647 |
| GodClass | Possible God Class (WMC=63, ATFD=41, TCC=3.333%) | 3 | 54 |
| EnumComparison | Enums should be compared using == | 3 | 208 |
| EnumComparison | Enums should be compared using == | 3 | 242 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 3 | 252–254 |
| EnumComparison | Enums should be compared using == | 3 | 252 |
| EnumComparison | Enums should be compared using == | 3 | 305 |
| EnumComparison | Enums should be compared using == | 3 | 511–512 |
| CyclomaticComplexity | The method 'entityDispatch(IEntityItem, Context)' has a cyclomatic complexity of 10. | 3 | 527 |
| ExhaustiveSwitchHasDefault | The switch block is exhaustive even without the default case | 3 | 529–556 |
gov/nist/secauto/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 | 61 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 3 | 75–78 |
gov/nist/secauto/oscal/lib/profile/resolver/selection/DefaultControlSelectionFilter.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryWarningSuppression | Unnecessary @SuppressWarnings annotation | 3 | 92 |
gov/nist/secauto/oscal/lib/profile/resolver/selection/FilterNonSelectedVisitor.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| EnumComparison | Enums should be compared using == | 3 | 156 |
| EnumComparison | Enums should be compared using == | 3 | 193 |
| EnumComparison | Enums should be compared using == | 3 | 198 |
| EnumComparison | Enums should be compared using == | 3 | 206 |
| EnumComparison | Enums should be compared using == | 3 | 253 |
| EnumComparison | Enums should be compared using == | 3 | 258 |
gov/nist/secauto/oscal/lib/profile/resolver/support/AbstractCatalogEntityVisitor.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryWarningSuppression | Unnecessary PMD suppression comment | 3 | 196 |
| UnnecessaryWarningSuppression | Unnecessary PMD suppression comment | 3 | 256 |
| UnnecessaryWarningSuppression | Unnecessary PMD suppression comment | 3 | 278 |
| UnnecessaryWarningSuppression | Unnecessary PMD suppression comment | 3 | 298 |
| UnnecessaryWarningSuppression | Unnecessary PMD suppression comment | 3 | 340 |
gov/nist/secauto/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–378 |
| GodClass | Possible God Class (WMC=50, ATFD=38, TCC=9.881%) | 3 | 42 |
| UnnecessaryWarningSuppression | Unnecessary @SuppressWarnings annotation | 3 | 71 |
| CyclomaticComplexity | The method 'isSelected(IEntityItem)' has a cyclomatic complexity of 11. | 3 | 102 |
| ExhaustiveSwitchHasDefault | The switch block is exhaustive even without the default case | 3 | 104–134 |
| EnumComparison | Enums should be compared using == | 3 | 107 |
| EnumComparison | Enums should be compared using == | 3 | 112 |
| EnumComparison | Enums should be compared using == | 3 | 121 |
| UnnecessaryWarningSuppression | Unnecessary @SuppressWarnings annotation | 3 | 195 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 3 | 229–231 |
gov/nist/secauto/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 |
gov/nist/secauto/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 | 113 |
Suppressed Violations
| Filename | Rule message | Suppression type | Reason |
|---|---|---|---|
| gov/nist/secauto/oscal/lib/OscalUtils.java | A method should have only one exit point, and that should be the last statement in the method | @suppresswarnings | |
| gov/nist/secauto/oscal/lib/metapath/function/library/HasOscalNamespace.java | Linguistics Antipattern - The method 'hasNamespace' indicates linguistically it returns a boolean, but it returns 'IBooleanItem' | @suppresswarnings | |
| gov/nist/secauto/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 | |
| gov/nist/secauto/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 | |
| gov/nist/secauto/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 | |
| gov/nist/secauto/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 | |
| gov/nist/secauto/oscal/lib/metapath/function/library/ResolveReference.java | Avoid unused method parameters such as 'function'. | @suppresswarnings | |
| gov/nist/secauto/oscal/lib/metapath/function/library/ResolveReference.java | Avoid unused method parameters such as 'dynamicContext'. | @suppresswarnings | |
| gov/nist/secauto/oscal/lib/metapath/function/library/ResolveReference.java | Avoid unused method parameters such as 'function'. | @suppresswarnings | |
| gov/nist/secauto/oscal/lib/metapath/function/library/ResolveReference.java | Avoid unused method parameters such as 'dynamicContext'. | @suppresswarnings | |
| gov/nist/secauto/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 | |
| gov/nist/secauto/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 | |
| gov/nist/secauto/oscal/lib/model/control/AbstractPart.java | Avoid using short method names | @suppresswarnings | |
| gov/nist/secauto/oscal/lib/model/metadata/AbstractProperty.java | Assigning an Object to null is a code smell. Consider refactoring. | @suppresswarnings | |
| gov/nist/secauto/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 | |
| gov/nist/secauto/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 | |
| gov/nist/secauto/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 | |
| gov/nist/secauto/oscal/lib/profile/resolver/ProfileResolver.java | Exception thrown at line 658 is caught in this block. | @suppresswarnings | |
| gov/nist/secauto/oscal/lib/profile/resolver/ProfileResolver.java | Exception thrown at line 686 is caught in this block. | @suppresswarnings | |
| gov/nist/secauto/oscal/lib/profile/resolver/ProfileResolver.java | Exception thrown at line 583 is caught in this block. | @suppresswarnings | |
| gov/nist/secauto/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 | |
| gov/nist/secauto/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 | |
| gov/nist/secauto/oscal/lib/profile/resolver/alter/AddVisitor.java | A value of 33 may denote a high amount of coupling within the class (threshold: 20) | @suppresswarnings | |
| gov/nist/secauto/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 | |
| gov/nist/secauto/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 | |
| gov/nist/secauto/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 | |
| gov/nist/secauto/oscal/lib/profile/resolver/selection/Import.java | Deeply nested if..then statements are hard to read | @suppresswarnings |