Есть два класса с многими по бизнесу общими полями, и с парой специфичных:
class ClassA {
public int Field1 {get;set;} // общее поле 1
//...
public int FieldN {get;set;} // общее поле N
public int SpecificA {get;set;} // специфичное поле для A
}
class ClassB {
public int Field1 {get;set;} // общее поле 1
//...
public int FieldN {get;set;} // общее поле N
public int SpecificB {get;set;} // специфичное поле для B
}
Я думаю создать для них общий класс BaseClass
:
class BaseClass {
public int Field1 {get;set;} // общее поле 1
// ...
public int FieldN {get;set;} // общее поле N
}
// И убрать из ClassA и ClassB общие поля в него, оставив только специфичные:
class ClassA : BaseClass {
public int SpecificA {get;set;} // специфичное поле для A
}
class ClassB : BaseClass {
public int SpecificB {get;set;} // специфичное поле для B
}
И создать такую фабрику(не уверен, что этот термин подходит):
public class Factory {
public static ClassA CreateClassA(){
var classA = new ClassA();
// Заполняем поля общие с классом ClassB
UpdateGeneralFields(classA);
// Заполняем специфичные для ClassA поля
classA.SpecificA = 'specificA';
return classA;
}
public static ClassB CreateClassB(){
var classB = new ClassB();
// Заполняем поля общие с классом ClassB
UpdateGeneralFields(classB);
// Заполняем специфичные для ClassB поля
classB.SpecificB = 'specificB';
return classB;
}
private static void UpdateGeneralFields(BaseClass baseClass){
baseClass.Field1 = 1;
// ...
baseClass.FieldN = N;
}
}
Вопросы
Это правильный подход?
Как можно создавать такие типы объектов используя известные порождающие шаблоны проектирования?
Насколько такой код может считаться чистым?
Что тут можно улучшить?
IEnumerable<BaseClass>
или что-то подобное, то лучше использовать композицию вместо наследования. Повторяющиеся свойства - это не причина для наследования.CreateClass*
вырождаются в простой вызов конструктора конкретного класса