PMD Results
The following document contains the results of PMD 7.5.0.
Violations By Priority
Priority 3
gov/nist/secauto/metaschema/databind/DefaultBindingContext.java
| Rule | Violation | Line |
|---|---|---|
| CouplingBetweenObjects | High amount of different objects as members denotes a high coupling | 6–317 |
| AvoidSynchronizedStatement | Use ReentrantLock rather than synchronization | 76–80 |
gov/nist/secauto/metaschema/databind/PostProcessingModuleLoaderStrategy.java
| Rule | Violation | Line |
|---|---|---|
| AvoidSynchronizedStatement | Use ReentrantLock rather than synchronization | 45–51 |
gov/nist/secauto/metaschema/databind/SimpleModuleLoaderStrategy.java
| Rule | Violation | Line |
|---|---|---|
| AvoidSynchronizedStatement | Use ReentrantLock rather than synchronization | 44–50 |
| AvoidSynchronizedStatement | Use ReentrantLock rather than synchronization | 57–65 |
gov/nist/secauto/metaschema/databind/codegen/typeinfo/AbstractTypeInfo.java
| Rule | Violation | Line |
|---|---|---|
| AvoidSynchronizedStatement | Use ReentrantLock rather than synchronization | 38–47 |
| AvoidSynchronizedStatement | Use ReentrantLock rather than synchronization | 58–63 |
gov/nist/secauto/metaschema/databind/codegen/typeinfo/ChoiceGroupTypeInfoImpl.java
| Rule | Violation | Line |
|---|---|---|
| CyclomaticComplexity | The method 'buildBindingAnnotation(Builder, Builder, Builder)' has a cyclomatic complexity of 10. | 54 |
| NPathComplexity | The method 'buildBindingAnnotation(Builder, Builder, Builder)' has an NPath complexity of 288, current threshold is 200 | 54 |
gov/nist/secauto/metaschema/databind/codegen/typeinfo/DefaultMetaschemaClassFactory.java
| Rule | Violation | Line |
|---|---|---|
| NPathComplexity | The method 'newClassBuilder(IModule, ClassName)' has an NPath complexity of 288, current threshold is 200 | 244 |
gov/nist/secauto/metaschema/databind/codegen/typeinfo/DefaultTypeResolver.java
| Rule | Violation | Line |
|---|---|---|
| CouplingBetweenObjects | High amount of different objects as members denotes a high coupling | 6–289 |
| AvoidSynchronizedStatement | Use ReentrantLock rather than synchronization | 218–242 |
| AvoidSynchronizedStatement | Use ReentrantLock rather than synchronization | 272–286 |
gov/nist/secauto/metaschema/databind/codegen/typeinfo/FlagInstanceTypeInfoImpl.java
| Rule | Violation | Line |
|---|---|---|
| NPathComplexity | The method 'buildField(Builder, Builder)' has an NPath complexity of 512, current threshold is 200 | 52 |
gov/nist/secauto/metaschema/databind/io/AbstractDeserializer.java
| Rule | Violation | Line |
|---|---|---|
| AvoidSynchronizedStatement | Use ReentrantLock rather than synchronization | 59–64 |
| AvoidSynchronizedStatement | Use ReentrantLock rather than synchronization | 69–71 |
gov/nist/secauto/metaschema/databind/io/DefaultBoundLoader.java
| Rule | Violation | Line |
|---|---|---|
| CouplingBetweenObjects | High amount of different objects as members denotes a high coupling | 6–253 |
gov/nist/secauto/metaschema/databind/io/ModelDetector.java
| Rule | Violation | Line |
|---|---|---|
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 235 |
gov/nist/secauto/metaschema/databind/io/json/DefaultJsonDeserializer.java
| Rule | Violation | Line |
|---|---|---|
| AvoidSynchronizedStatement | Use ReentrantLock rather than synchronization | 62–68 |
gov/nist/secauto/metaschema/databind/io/json/DefaultJsonSerializer.java
| Rule | Violation | Line |
|---|---|---|
| AvoidSynchronizedStatement | Use ReentrantLock rather than synchronization | 56–58 |
| AvoidSynchronizedStatement | Use ReentrantLock rather than synchronization | 63–69 |
gov/nist/secauto/metaschema/databind/io/json/MetaschemaJsonReader.java
| Rule | Violation | Line |
|---|---|---|
| CouplingBetweenObjects | High amount of different objects as members denotes a high coupling | 6–808 |
| GodClass | Possible God Class (WMC=50, ATFD=45, TCC=1.087%) | 57 |
| CloseResource | Ensure that resources like this JsonParser object are closed after use | 146 |
| CyclomaticComplexity | The method 'readObjectRoot(IBoundDefinitionModelComplex, String)' has a cyclomatic complexity of 12. | 169 |
| CloseResource | Ensure that resources like this JsonParser object are closed after use | 172 |
| CloseResource | Ensure that resources like this JsonParser object are closed after use | 436 |
| CloseResource | Ensure that resources like this JsonParser object are closed after use | 487 |
| CloseResource | Ensure that resources like this JsonParser object are closed after use | 524 |
| CloseResource | Ensure that resources like this JsonParser object are closed after use | 687 |
| CloseResource | Ensure that resources like this JsonParser object are closed after use | 717 |
| DataClass | The class 'MetaschemaData' is suspected to be a Data Class (WOC=0.000%, NOPA=0, NOAM=4, WMC=5) | 767 |
gov/nist/secauto/metaschema/databind/io/json/MetaschemaJsonWriter.java
| Rule | Violation | Line |
|---|---|---|
| CouplingBetweenObjects | High amount of different objects as members denotes a high coupling | 6–364 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 110 |
gov/nist/secauto/metaschema/databind/io/xml/DefaultXmlDeserializer.java
| Rule | Violation | Line |
|---|---|---|
| AvoidSynchronizedStatement | Use ReentrantLock rather than synchronization | 71–97 |
| AvoidSynchronizedStatement | Use ReentrantLock rather than synchronization | 108–110 |
gov/nist/secauto/metaschema/databind/io/xml/DefaultXmlSerializer.java
| Rule | Violation | Line |
|---|---|---|
| AvoidSynchronizedStatement | Use ReentrantLock rather than synchronization | 52–62 |
| AvoidSynchronizedStatement | Use ReentrantLock rather than synchronization | 73–75 |
gov/nist/secauto/metaschema/databind/io/xml/MetaschemaXmlReader.java
| Rule | Violation | Line |
|---|---|---|
| CouplingBetweenObjects | High amount of different objects as members denotes a high coupling | 6–643 |
gov/nist/secauto/metaschema/databind/io/xml/MetaschemaXmlWriter.java
| Rule | Violation | Line |
|---|---|---|
| CouplingBetweenObjects | High amount of different objects as members denotes a high coupling | 6–398 |
gov/nist/secauto/metaschema/databind/model/AbstractBoundModule.java
| Rule | Violation | Line |
|---|---|---|
| CouplingBetweenObjects | High amount of different objects as members denotes a high coupling | 6–263 |
gov/nist/secauto/metaschema/databind/model/IBoundDefinitionModelFieldComplex.java
| Rule | Violation | Line |
|---|---|---|
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 103 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 118 |
gov/nist/secauto/metaschema/databind/model/impl/AssemblyModelContainerSupport.java
| Rule | Violation | Line |
|---|---|---|
| CyclomaticComplexity | The constructor 'AssemblyModelContainerSupport(DefinitionAssembly)' has a cyclomatic complexity of 10. | 59 |
gov/nist/secauto/metaschema/databind/model/impl/ConstraintFactory.java
| Rule | Violation | Line |
|---|---|---|
| CouplingBetweenObjects | High amount of different objects as members denotes a high coupling | 6–334 |
| UnusedPrivateField | Avoid unused private fields such as 'LOGGER'. | 55 |
gov/nist/secauto/metaschema/databind/model/impl/DefinitionField.java
| Rule | Violation | Line |
|---|---|---|
| CouplingBetweenObjects | High amount of different objects as members denotes a high coupling | 6–311 |
gov/nist/secauto/metaschema/databind/model/impl/InstanceModelChoiceGroup.java
| Rule | Violation | Line |
|---|---|---|
| CouplingBetweenObjects | High amount of different objects as members denotes a high coupling | 6–346 |
gov/nist/secauto/metaschema/databind/model/impl/InstanceModelFieldComplex.java
| Rule | Violation | Line |
|---|---|---|
| CyclomaticComplexity | The method 'newInstance(Field, DefinitionField, IBoundDefinitionModelAssembly)' has a cyclomatic complexity of 12. | 61 |
| CognitiveComplexity | The constructor 'InstanceModelFieldComplex(Field, BoundField, IGroupAs, DefinitionField, IBoundDefinitionModelAssembly)' has a cognitive complexity of 16, current threshold is 15 | 117 |
gov/nist/secauto/metaschema/databind/model/info/IModelInstanceCollectionInfo.java
| Rule | Violation | Line |
|---|---|---|
| CognitiveComplexity | The method 'of(IBoundInstanceModel<T>)' has a cognitive complexity of 19, current threshold is 15 | 29 |
| CyclomaticComplexity | The method 'of(IBoundInstanceModel<T>)' has a cyclomatic complexity of 17. | 29 |
gov/nist/secauto/metaschema/databind/model/metaschema/BindingConstraintLoader.java
| Rule | Violation | Line |
|---|---|---|
| CouplingBetweenObjects | High amount of different objects as members denotes a high coupling | 6–403 |
| CognitiveComplexity | The method 'parseResource(URI, Deque<URI>)' has a cognitive complexity of 20, current threshold is 15 | 86 |
| CyclomaticComplexity | The method 'parseResource(URI, Deque<URI>)' has a cyclomatic complexity of 12. | 86 |
| CyclomaticComplexity | The method 'parseScopedConstraints(MetaschemaModuleConstraints, ISource)' has a cyclomatic complexity of 10. | 180 |
gov/nist/secauto/metaschema/databind/model/metaschema/BindingModuleLoader.java
| Rule | Violation | Line |
|---|---|---|
| AvoidSynchronizedStatement | Use ReentrantLock rather than synchronization | 94–99 |
gov/nist/secauto/metaschema/databind/model/metaschema/impl/AssemblyModelContainerSupport.java
| Rule | Violation | Line |
|---|---|---|
| CognitiveComplexity | The constructor 'AssemblyModelContainerSupport(AssemblyModel, IBoundInstanceModelAssembly, IBindingDefinitionModelAssembly, INodeItemFactory)' has a cognitive complexity of 15, current threshold is 15 | 100 |
| CyclomaticComplexity | The constructor 'AssemblyModelContainerSupport(AssemblyModel, IBoundInstanceModelAssembly, IBindingDefinitionModelAssembly, INodeItemFactory)' has a cyclomatic complexity of 15. | 100 |
| NPathComplexity | The constructor 'AssemblyModelContainerSupport(AssemblyModel, IBoundInstanceModelAssembly, IBindingDefinitionModelAssembly, INodeItemFactory)' has an NPath complexity of 512, current threshold is 200 | 100 |
gov/nist/secauto/metaschema/databind/model/metaschema/impl/BindingConstants.java
| Rule | Violation | Line |
|---|---|---|
| DataClass | The class 'BindingConstants' is suspected to be a Data Class (WOC=0.000%, NOPA=6, NOAM=0, WMC=1) | 9 |
gov/nist/secauto/metaschema/databind/model/metaschema/impl/BindingModule.java
| Rule | Violation | Line |
|---|---|---|
| CouplingBetweenObjects | High amount of different objects as members denotes a high coupling | 6–332 |
| AvoidInstantiatingObjectsInLoops | Avoid instantiating new objects inside loops | 295–300 |
| AvoidInstantiatingObjectsInLoops | Avoid instantiating new objects inside loops | 307–311 |
| AvoidInstantiatingObjectsInLoops | Avoid instantiating new objects inside loops | 315–319 |
gov/nist/secauto/metaschema/databind/model/metaschema/impl/ChoiceGroupModelContainerSupport.java
| Rule | Violation | Line |
|---|---|---|
| CyclomaticComplexity | The constructor 'ChoiceGroupModelContainerSupport(ChoiceGroup, IBoundInstanceModelGroupedAssembly, IChoiceGroupInstance, INodeItemFactory)' has a cyclomatic complexity of 10. | 81 |
gov/nist/secauto/metaschema/databind/model/metaschema/impl/ChoiceModelContainerSupport.java
| Rule | Violation | Line |
|---|---|---|
| CyclomaticComplexity | The constructor 'ChoiceModelContainerSupport(Choice, IBoundInstanceModelGroupedAssembly, IChoiceInstance, INodeItemFactory)' has a cyclomatic complexity of 11. | 81 |
gov/nist/secauto/metaschema/databind/model/metaschema/impl/ConstraintBindingSupport.java
| Rule | Violation | Line |
|---|---|---|
| CouplingBetweenObjects | High amount of different objects as members denotes a high coupling | 6–436 |
| GodClass | Possible God Class (WMC=70, ATFD=48, TCC=0.000%) | 56 |
| AvoidInstantiatingObjectsInLoops | Avoid instantiating new objects inside loops | 149 |
gov/nist/secauto/metaschema/databind/model/metaschema/impl/FlagContainerSupport.java
| Rule | Violation | Line |
|---|---|---|
| AvoidInstantiatingObjectsInLoops | Avoid instantiating new objects inside loops | 58–62 |
Files
gov/nist/secauto/metaschema/databind/DefaultBindingContext.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CouplingBetweenObjects | High amount of different objects as members denotes a high coupling | 3 | 6–317 |
| AvoidSynchronizedStatement | Use ReentrantLock rather than synchronization | 3 | 76–80 |
gov/nist/secauto/metaschema/databind/PostProcessingModuleLoaderStrategy.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AvoidSynchronizedStatement | Use ReentrantLock rather than synchronization | 3 | 45–51 |
gov/nist/secauto/metaschema/databind/SimpleModuleLoaderStrategy.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AvoidSynchronizedStatement | Use ReentrantLock rather than synchronization | 3 | 44–50 |
| AvoidSynchronizedStatement | Use ReentrantLock rather than synchronization | 3 | 57–65 |
gov/nist/secauto/metaschema/databind/codegen/typeinfo/AbstractTypeInfo.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AvoidSynchronizedStatement | Use ReentrantLock rather than synchronization | 3 | 38–47 |
| AvoidSynchronizedStatement | Use ReentrantLock rather than synchronization | 3 | 58–63 |
gov/nist/secauto/metaschema/databind/codegen/typeinfo/ChoiceGroupTypeInfoImpl.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CyclomaticComplexity | The method 'buildBindingAnnotation(Builder, Builder, Builder)' has a cyclomatic complexity of 10. | 3 | 54 |
| NPathComplexity | The method 'buildBindingAnnotation(Builder, Builder, Builder)' has an NPath complexity of 288, current threshold is 200 | 3 | 54 |
gov/nist/secauto/metaschema/databind/codegen/typeinfo/DefaultMetaschemaClassFactory.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| NPathComplexity | The method 'newClassBuilder(IModule, ClassName)' has an NPath complexity of 288, current threshold is 200 | 3 | 244 |
gov/nist/secauto/metaschema/databind/codegen/typeinfo/DefaultTypeResolver.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CouplingBetweenObjects | High amount of different objects as members denotes a high coupling | 3 | 6–289 |
| AvoidSynchronizedStatement | Use ReentrantLock rather than synchronization | 3 | 218–242 |
| AvoidSynchronizedStatement | Use ReentrantLock rather than synchronization | 3 | 272–286 |
gov/nist/secauto/metaschema/databind/codegen/typeinfo/FlagInstanceTypeInfoImpl.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| NPathComplexity | The method 'buildField(Builder, Builder)' has an NPath complexity of 512, current threshold is 200 | 3 | 52 |
gov/nist/secauto/metaschema/databind/io/AbstractDeserializer.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AvoidSynchronizedStatement | Use ReentrantLock rather than synchronization | 3 | 59–64 |
| AvoidSynchronizedStatement | Use ReentrantLock rather than synchronization | 3 | 69–71 |
gov/nist/secauto/metaschema/databind/io/DefaultBoundLoader.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CouplingBetweenObjects | High amount of different objects as members denotes a high coupling | 3 | 6–253 |
gov/nist/secauto/metaschema/databind/io/ModelDetector.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 235 |
gov/nist/secauto/metaschema/databind/io/json/DefaultJsonDeserializer.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AvoidSynchronizedStatement | Use ReentrantLock rather than synchronization | 3 | 62–68 |
gov/nist/secauto/metaschema/databind/io/json/DefaultJsonSerializer.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AvoidSynchronizedStatement | Use ReentrantLock rather than synchronization | 3 | 56–58 |
| AvoidSynchronizedStatement | Use ReentrantLock rather than synchronization | 3 | 63–69 |
gov/nist/secauto/metaschema/databind/io/json/MetaschemaJsonReader.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CouplingBetweenObjects | High amount of different objects as members denotes a high coupling | 3 | 6–808 |
| GodClass | Possible God Class (WMC=50, ATFD=45, TCC=1.087%) | 3 | 57 |
| CloseResource | Ensure that resources like this JsonParser object are closed after use | 3 | 146 |
| CyclomaticComplexity | The method 'readObjectRoot(IBoundDefinitionModelComplex, String)' has a cyclomatic complexity of 12. | 3 | 169 |
| CloseResource | Ensure that resources like this JsonParser object are closed after use | 3 | 172 |
| CloseResource | Ensure that resources like this JsonParser object are closed after use | 3 | 436 |
| CloseResource | Ensure that resources like this JsonParser object are closed after use | 3 | 487 |
| CloseResource | Ensure that resources like this JsonParser object are closed after use | 3 | 524 |
| CloseResource | Ensure that resources like this JsonParser object are closed after use | 3 | 687 |
| CloseResource | Ensure that resources like this JsonParser object are closed after use | 3 | 717 |
| DataClass | The class 'MetaschemaData' is suspected to be a Data Class (WOC=0.000%, NOPA=0, NOAM=4, WMC=5) | 3 | 767 |
gov/nist/secauto/metaschema/databind/io/json/MetaschemaJsonWriter.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CouplingBetweenObjects | High amount of different objects as members denotes a high coupling | 3 | 6–364 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 110 |
gov/nist/secauto/metaschema/databind/io/xml/DefaultXmlDeserializer.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AvoidSynchronizedStatement | Use ReentrantLock rather than synchronization | 3 | 71–97 |
| AvoidSynchronizedStatement | Use ReentrantLock rather than synchronization | 3 | 108–110 |
gov/nist/secauto/metaschema/databind/io/xml/DefaultXmlSerializer.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AvoidSynchronizedStatement | Use ReentrantLock rather than synchronization | 3 | 52–62 |
| AvoidSynchronizedStatement | Use ReentrantLock rather than synchronization | 3 | 73–75 |
gov/nist/secauto/metaschema/databind/io/xml/MetaschemaXmlReader.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CouplingBetweenObjects | High amount of different objects as members denotes a high coupling | 3 | 6–643 |
gov/nist/secauto/metaschema/databind/io/xml/MetaschemaXmlWriter.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CouplingBetweenObjects | High amount of different objects as members denotes a high coupling | 3 | 6–398 |
gov/nist/secauto/metaschema/databind/model/AbstractBoundModule.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CouplingBetweenObjects | High amount of different objects as members denotes a high coupling | 3 | 6–263 |
gov/nist/secauto/metaschema/databind/model/IBoundDefinitionModelFieldComplex.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 103 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 118 |
gov/nist/secauto/metaschema/databind/model/impl/AssemblyModelContainerSupport.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CyclomaticComplexity | The constructor 'AssemblyModelContainerSupport(DefinitionAssembly)' has a cyclomatic complexity of 10. | 3 | 59 |
gov/nist/secauto/metaschema/databind/model/impl/ConstraintFactory.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CouplingBetweenObjects | High amount of different objects as members denotes a high coupling | 3 | 6–334 |
| UnusedPrivateField | Avoid unused private fields such as 'LOGGER'. | 3 | 55 |
gov/nist/secauto/metaschema/databind/model/impl/DefinitionField.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CouplingBetweenObjects | High amount of different objects as members denotes a high coupling | 3 | 6–311 |
gov/nist/secauto/metaschema/databind/model/impl/InstanceModelChoiceGroup.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CouplingBetweenObjects | High amount of different objects as members denotes a high coupling | 3 | 6–346 |
gov/nist/secauto/metaschema/databind/model/impl/InstanceModelFieldComplex.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CyclomaticComplexity | The method 'newInstance(Field, DefinitionField, IBoundDefinitionModelAssembly)' has a cyclomatic complexity of 12. | 3 | 61 |
| CognitiveComplexity | The constructor 'InstanceModelFieldComplex(Field, BoundField, IGroupAs, DefinitionField, IBoundDefinitionModelAssembly)' has a cognitive complexity of 16, current threshold is 15 | 3 | 117 |
gov/nist/secauto/metaschema/databind/model/info/IModelInstanceCollectionInfo.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CognitiveComplexity | The method 'of(IBoundInstanceModel<T>)' has a cognitive complexity of 19, current threshold is 15 | 3 | 29 |
| CyclomaticComplexity | The method 'of(IBoundInstanceModel<T>)' has a cyclomatic complexity of 17. | 3 | 29 |
gov/nist/secauto/metaschema/databind/model/metaschema/BindingConstraintLoader.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CouplingBetweenObjects | High amount of different objects as members denotes a high coupling | 3 | 6–403 |
| CognitiveComplexity | The method 'parseResource(URI, Deque<URI>)' has a cognitive complexity of 20, current threshold is 15 | 3 | 86 |
| CyclomaticComplexity | The method 'parseResource(URI, Deque<URI>)' has a cyclomatic complexity of 12. | 3 | 86 |
| CyclomaticComplexity | The method 'parseScopedConstraints(MetaschemaModuleConstraints, ISource)' has a cyclomatic complexity of 10. | 3 | 180 |
gov/nist/secauto/metaschema/databind/model/metaschema/BindingModuleLoader.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AvoidSynchronizedStatement | Use ReentrantLock rather than synchronization | 3 | 94–99 |
gov/nist/secauto/metaschema/databind/model/metaschema/impl/AssemblyModelContainerSupport.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CognitiveComplexity | The constructor 'AssemblyModelContainerSupport(AssemblyModel, IBoundInstanceModelAssembly, IBindingDefinitionModelAssembly, INodeItemFactory)' has a cognitive complexity of 15, current threshold is 15 | 3 | 100 |
| CyclomaticComplexity | The constructor 'AssemblyModelContainerSupport(AssemblyModel, IBoundInstanceModelAssembly, IBindingDefinitionModelAssembly, INodeItemFactory)' has a cyclomatic complexity of 15. | 3 | 100 |
| NPathComplexity | The constructor 'AssemblyModelContainerSupport(AssemblyModel, IBoundInstanceModelAssembly, IBindingDefinitionModelAssembly, INodeItemFactory)' has an NPath complexity of 512, current threshold is 200 | 3 | 100 |
gov/nist/secauto/metaschema/databind/model/metaschema/impl/BindingConstants.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| DataClass | The class 'BindingConstants' is suspected to be a Data Class (WOC=0.000%, NOPA=6, NOAM=0, WMC=1) | 3 | 9 |
gov/nist/secauto/metaschema/databind/model/metaschema/impl/BindingModule.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CouplingBetweenObjects | High amount of different objects as members denotes a high coupling | 3 | 6–332 |
| AvoidInstantiatingObjectsInLoops | Avoid instantiating new objects inside loops | 3 | 295–300 |
| AvoidInstantiatingObjectsInLoops | Avoid instantiating new objects inside loops | 3 | 307–311 |
| AvoidInstantiatingObjectsInLoops | Avoid instantiating new objects inside loops | 3 | 315–319 |
gov/nist/secauto/metaschema/databind/model/metaschema/impl/ChoiceGroupModelContainerSupport.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CyclomaticComplexity | The constructor 'ChoiceGroupModelContainerSupport(ChoiceGroup, IBoundInstanceModelGroupedAssembly, IChoiceGroupInstance, INodeItemFactory)' has a cyclomatic complexity of 10. | 3 | 81 |
gov/nist/secauto/metaschema/databind/model/metaschema/impl/ChoiceModelContainerSupport.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CyclomaticComplexity | The constructor 'ChoiceModelContainerSupport(Choice, IBoundInstanceModelGroupedAssembly, IChoiceInstance, INodeItemFactory)' has a cyclomatic complexity of 11. | 3 | 81 |
gov/nist/secauto/metaschema/databind/model/metaschema/impl/ConstraintBindingSupport.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CouplingBetweenObjects | High amount of different objects as members denotes a high coupling | 3 | 6–436 |
| GodClass | Possible God Class (WMC=70, ATFD=48, TCC=0.000%) | 3 | 56 |
| AvoidInstantiatingObjectsInLoops | Avoid instantiating new objects inside loops | 3 | 149 |
gov/nist/secauto/metaschema/databind/model/metaschema/impl/FlagContainerSupport.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AvoidInstantiatingObjectsInLoops | Avoid instantiating new objects inside loops | 3 | 58–62 |
Suppressed Violations
| Filename | Rule message | Suppression type | Reason |
|---|---|---|---|
| gov/nist/secauto/metaschema/databind/IBindingContext.java | Avoid using short method names | @suppresswarnings | |
| gov/nist/secauto/metaschema/databind/codegen/IProduction.java | Avoid using short method names | //nopmd | - intentional |
| gov/nist/secauto/metaschema/databind/codegen/IProduction.java | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | //nopmd | - no concurrency |
| gov/nist/secauto/metaschema/databind/codegen/IProduction.java | Avoid instantiating new objects inside loops | //nopmd | - intentional |
| gov/nist/secauto/metaschema/databind/codegen/ModuleCompilerHelper.java | The method 'compile(JavaCompiler, JavaFileManager, DiagnosticCollector<JavaFileObject>, List<JavaFileObject>, Path)' has a cognitive complexity of 20, current threshold is 15 | @suppresswarnings | |
| gov/nist/secauto/metaschema/databind/codegen/ModuleCompilerHelper.java | The method 'compile(JavaCompiler, JavaFileManager, DiagnosticCollector<JavaFileObject>, List<JavaFileObject>, Path)' has a cyclomatic complexity of 12. | @suppresswarnings | |
| gov/nist/secauto/metaschema/databind/codegen/ProductionImpl.java | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | //nopmd | - immutable |
| gov/nist/secauto/metaschema/databind/codegen/ProductionImpl.java | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | //nopmd | - immutable |
| gov/nist/secauto/metaschema/databind/codegen/impl/AnnotationGenerator.java | High amount of different objects as members denotes a high coupling | @suppresswarnings | |
| gov/nist/secauto/metaschema/databind/codegen/impl/AnnotationGenerator.java | Possible God Class (WMC=72, ATFD=88, TCC=0.735%) | @suppresswarnings | |
| gov/nist/secauto/metaschema/databind/codegen/impl/AnnotationGenerator.java | Assigning an Object to null is a code smell. Consider refactoring. | //nopmd | readability |
| gov/nist/secauto/metaschema/databind/codegen/typeinfo/ChoiceGroupTypeInfoImpl.java | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | @suppresswarnings | |
| gov/nist/secauto/metaschema/databind/codegen/typeinfo/ChoiceGroupTypeInfoImpl.java | Avoid instantiating new objects inside loops | //nopmd | needed |
| gov/nist/secauto/metaschema/databind/codegen/typeinfo/DefaultMetaschemaClassFactory.java | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | //nopmd | - intended |
| gov/nist/secauto/metaschema/databind/codegen/typeinfo/DefaultMetaschemaClassFactory.java | High amount of different objects as members denotes a high coupling | @suppresswarnings | |
| gov/nist/secauto/metaschema/databind/codegen/typeinfo/DefaultMetaschemaClassFactory.java | The method 'newClassBuilder(IModule, ClassName)' has a cyclomatic complexity of 10. | @suppresswarnings | |
| gov/nist/secauto/metaschema/databind/codegen/typeinfo/FlagInstanceTypeInfoImpl.java | The method 'buildField(Builder, Builder)' has a cyclomatic complexity of 11. | @suppresswarnings | |
| gov/nist/secauto/metaschema/databind/io/AbstractDeserializer.java | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | //nopmd | - this is intentional |
| gov/nist/secauto/metaschema/databind/io/DefaultBoundLoader.java | Assigning an Object to null is a code smell. Consider refactoring. | @suppresswarnings | |
| gov/nist/secauto/metaschema/databind/io/DeserializationFeature.java | The class 'DeserializationFeature' is suspected to be a Data Class (WOC=0.000%, NOPA=7, NOAM=0, WMC=1) | @suppresswarnings | |
| gov/nist/secauto/metaschema/databind/io/json/DefaultJsonSerializer.java | Assigning an Object to null is a code smell. Consider refactoring. | @suppresswarnings | |
| gov/nist/secauto/metaschema/databind/io/json/MetaschemaJsonReader.java | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | //nopmd | not concurrent |
| gov/nist/secauto/metaschema/databind/io/json/MetaschemaJsonReader.java | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | @suppresswarnings | |
| gov/nist/secauto/metaschema/databind/io/xml/DefaultXmlSerializer.java | A throw statement in a finally block makes the control flow hard to understand. | //nopmd | - exception handling is needed |
| gov/nist/secauto/metaschema/databind/metapath/function/Model.java | A method should have only one exit point, and that should be the last statement in the method | @suppresswarnings | |
| gov/nist/secauto/metaschema/databind/metapath/function/Model.java | A method should have only one exit point, and that should be the last statement in the method | @suppresswarnings | |
| gov/nist/secauto/metaschema/databind/model/IFeatureJavaField.java | You should not modify visibility of constructors, methods or fields using setAccessible() | //nopmd | - intentional |
| gov/nist/secauto/metaschema/databind/model/IFeatureJavaField.java | You should not modify visibility of constructors, methods or fields using setAccessible() | //nopmd | - intentional |
| gov/nist/secauto/metaschema/databind/model/IFeatureJavaField.java | You should not modify visibility of constructors, methods or fields using setAccessible() | //nopmd | - intentional |
| gov/nist/secauto/metaschema/databind/model/IFeatureJavaField.java | You should not modify visibility of constructors, methods or fields using setAccessible() | //nopmd | - intentional |
| gov/nist/secauto/metaschema/databind/model/annotations/AllowedValues.java | Avoid using short method names | @suppresswarnings | |
| gov/nist/secauto/metaschema/databind/model/annotations/Expect.java | Avoid using short method names | @suppresswarnings | |
| gov/nist/secauto/metaschema/databind/model/annotations/HasCardinality.java | Avoid using short method names | @suppresswarnings | |
| gov/nist/secauto/metaschema/databind/model/annotations/Index.java | Avoid using short method names | @suppresswarnings | |
| gov/nist/secauto/metaschema/databind/model/annotations/IndexHasKey.java | Avoid using short method names | @suppresswarnings | |
| gov/nist/secauto/metaschema/databind/model/annotations/IsUnique.java | Avoid using short method names | @suppresswarnings | |
| gov/nist/secauto/metaschema/databind/model/annotations/Matches.java | Avoid using short method names | @suppresswarnings | |
| gov/nist/secauto/metaschema/databind/model/annotations/ModelUtil.java | Assigning an Object to null is a code smell. Consider refactoring. | //nopmd | - intentional |
| gov/nist/secauto/metaschema/databind/model/impl/AssemblyModelContainerSupport.java | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | @suppresswarnings | |
| gov/nist/secauto/metaschema/databind/model/impl/AssemblyModelContainerSupport.java | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | @suppresswarnings | |
| gov/nist/secauto/metaschema/databind/model/impl/AssemblyModelContainerSupport.java | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | @suppresswarnings | |
| gov/nist/secauto/metaschema/databind/model/impl/AssemblyModelContainerSupport.java | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | @suppresswarnings | |
| gov/nist/secauto/metaschema/databind/model/impl/ConstraintFactory.java | Avoid instantiating new objects inside loops | @suppresswarnings | |
| gov/nist/secauto/metaschema/databind/model/impl/FlagContainerSupport.java | Use asList instead of tight loops | @suppresswarnings | |
| gov/nist/secauto/metaschema/databind/model/info/IFeatureComplexItemValueHandler.java | Avoid throwing raw exception types. | @suppresswarnings | |
| gov/nist/secauto/metaschema/databind/model/info/IFeatureComplexItemValueHandler.java | Avoid throwing raw exception types. | @suppresswarnings | |
| gov/nist/secauto/metaschema/databind/model/info/IModelInstanceCollectionInfo.java | Avoid using short method names | @suppresswarnings | |
| gov/nist/secauto/metaschema/databind/model/metaschema/BindingConstraintLoader.java | Avoid instantiating new objects inside loops | @suppresswarnings | |
| gov/nist/secauto/metaschema/databind/model/metaschema/impl/AssemblyModelContainerSupport.java | Avoid using short method names | @suppresswarnings | |
| gov/nist/secauto/metaschema/databind/model/metaschema/impl/AssemblyModelContainerSupport.java | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | @suppresswarnings | |
| gov/nist/secauto/metaschema/databind/model/metaschema/impl/AssemblyModelContainerSupport.java | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | @suppresswarnings | |
| gov/nist/secauto/metaschema/databind/model/metaschema/impl/AssemblyModelContainerSupport.java | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | @suppresswarnings | |
| gov/nist/secauto/metaschema/databind/model/metaschema/impl/AssemblyModelContainerSupport.java | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | @suppresswarnings | |
| gov/nist/secauto/metaschema/databind/model/metaschema/impl/AssemblyModelContainerSupport.java | Avoid instantiating new objects inside loops | @suppresswarnings | |
| gov/nist/secauto/metaschema/databind/model/metaschema/impl/AssemblyModelContainerSupport.java | Avoid instantiating new objects inside loops | @suppresswarnings | |
| gov/nist/secauto/metaschema/databind/model/metaschema/impl/AssemblyModelContainerSupport.java | Avoid instantiating new objects inside loops | @suppresswarnings | |
| gov/nist/secauto/metaschema/databind/model/metaschema/impl/AssemblyModelContainerSupport.java | Avoid instantiating new objects inside loops | @suppresswarnings | |
| gov/nist/secauto/metaschema/databind/model/metaschema/impl/ChoiceGroupModelContainerSupport.java | Avoid using short method names | @suppresswarnings | |
| gov/nist/secauto/metaschema/databind/model/metaschema/impl/ChoiceGroupModelContainerSupport.java | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | @suppresswarnings | |
| gov/nist/secauto/metaschema/databind/model/metaschema/impl/ChoiceGroupModelContainerSupport.java | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | @suppresswarnings | |
| gov/nist/secauto/metaschema/databind/model/metaschema/impl/ChoiceGroupModelContainerSupport.java | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | @suppresswarnings | |
| gov/nist/secauto/metaschema/databind/model/metaschema/impl/ChoiceGroupModelContainerSupport.java | Avoid instantiating new objects inside loops | @suppresswarnings | |
| gov/nist/secauto/metaschema/databind/model/metaschema/impl/ChoiceGroupModelContainerSupport.java | Avoid instantiating new objects inside loops | @suppresswarnings | |
| gov/nist/secauto/metaschema/databind/model/metaschema/impl/ChoiceModelContainerSupport.java | Avoid using short method names | @suppresswarnings | |
| gov/nist/secauto/metaschema/databind/model/metaschema/impl/ChoiceModelContainerSupport.java | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | @suppresswarnings | |
| gov/nist/secauto/metaschema/databind/model/metaschema/impl/ChoiceModelContainerSupport.java | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | @suppresswarnings | |
| gov/nist/secauto/metaschema/databind/model/metaschema/impl/ChoiceModelContainerSupport.java | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | @suppresswarnings | |
| gov/nist/secauto/metaschema/databind/model/metaschema/impl/ChoiceModelContainerSupport.java | Avoid instantiating new objects inside loops | @suppresswarnings | |
| gov/nist/secauto/metaschema/databind/model/metaschema/impl/ChoiceModelContainerSupport.java | Avoid instantiating new objects inside loops | @suppresswarnings | |
| gov/nist/secauto/metaschema/databind/model/metaschema/impl/FlagContainerSupport.java | A method should have only one exit point, and that should be the last statement in the method | @suppresswarnings |