That have repeated code for ConfigurationSection accessing which can instead be covered by a single piece of code. This could probably be a nested class on the abstract class AttributeTypeConstructor and then normal constructors can take in this object, probably even the parent class can handle this setting.
Better yet, most AttributeTypes' constructors should be able to take in a configuration section and handle it themselves. However this would require a flag to be set during construction or to allow post constructor initialization.
Migsect
commented
4 years ago
The issue is that we have the variables:
That have repeated code for ConfigurationSection accessing which can instead be covered by a single piece of code. This could probably be a nested class on the abstract class AttributeTypeConstructor and then normal constructors can take in this object, probably even the parent class can handle this setting.
Better yet, most AttributeTypes' constructors should be able to take in a configuration section and handle it themselves. However this would require a flag to be set during construction or to allow post constructor initialization.