Интерфейс, Аннотация или просто виртуальные методы?

Question

У меня есть группа систем, позвоните по телефону A, B, C, D, E, F, G, H, I, J.

Все они имеют сходные методы и свойства. Некоторые из них содержат тот же метод и свойства, некоторые из них могут незначительно отличаться, а некоторые могут сильно различаться. Прямо сейчас у меня есть много дублированного кода для каждой системы. Например, у меня есть метод, называемый GetPropertyInformation(), который определен для каждой системы. Я пытаюсь выяснить, какой метод будет наилучшим способом уменьшить дублирование кода или, может быть одним из перечисленных ниже способов не путь:

Интерфейс

public Interface ISystem 
{ 
    public void GetPropertyInformation(); 
    //Other methods to implement 
} 

public class A : ISystem 
{ 
    public void GetPropertyInformation() 
    { 
     //Code here 
    } 
} 

Аннотация

public abstract class System 
{ 
    public virtual void GetPropertyInformation() 
    { 
     //Standard Code here 
    } 
} 

public class B : System 
{ 
    public override void GetPropertyInformation() 
    { 
     //B specific code here 
    } 
} 

Виртуальные методы в классе Супер Base

public class System 
{ 
    public virtual void GetPropertyInformation() 
    { 
    //System Code 
    } 
} 

public class C : System 
{ 
    public override void GetPropertyInformation() 
    { 
     //C Code 
    } 
} 

один вопрос, хотя это может быть глупо, это допустим, я пошел с абстрактным подходом, и я хотел, чтобы переопределить GetPropertyInformation, но мне нужно, чтобы передать ему дополнительный параметр, возможно ли это или я должен создать другой метод в абстрактном классе? Например, GetPropertyInformation(x)

Answer 1

Подходы с абстрактным и «супер базовым классом» не слишком разные. Вы всегда должны делать абстрактные абстрактные классы, и вы можете предоставить реализацию по умолчанию (виртуальные методы) или нет (абстрактные методы). Решающим фактором является то, хотите ли вы когда-либо иметь экземпляры базового класса, я думаю, что нет.

Так что это между базовым классом и интерфейсом. Если между вашими классами A, B C существует сильная связь, вы можете использовать базовый класс и, возможно, общую реализацию.

Если классы A, B, C естественно не принадлежат к одному «семейству», то используйте интерфейс.

И System не такое хорошее имя.

И вы не можете изменить список параметров при переопределении. Возможно, параметры по умолчанию могут помочь, иначе вам просто нужно 2 перегрузки для GetPropertyInformation().

Answer 2

Как правило, вы выбираете наследование объектов, когда хотите разделить реализацию и уменьшите то, что в противном случае было бы дублированием. В противном случае интерфейсы выигрывают, потому что они более гибкие, так как нет необходимости в общем базовом классе.

Что касается переопределения метода и изменения списка параметров, это просто невозможно. Представьте, как бы вы назвали этот метод базовым классом или ссылкой на интерфейс?

Answer 3

Я бы пошел с чем-то вроде того, что я добавил ниже. Вы по-прежнему пользуетесь интерфейсом и совместной реализацией.

public Interface ISystem 
{ 
    public void GetPropertyInformation(); 
    //Other methods to implement 
} 

public abstract class System : ISystem 
{ 
    public virtual void GetPropertyInformation() 
    { 
     //Standard Code here 
    } 
} 

public class B : System 
{ 
    public string ExtendedSystemProp {get;set;} 

    public override void GetPropertyInformation() 
    { 
     base.GetPropertyInformation(); 

     var prop = "some extra calculating"; 

     GetExtraPropertyInformation(prop); 
    } 

    public void GetExtraPropertyInformation(string prop) 
    { 
     ExtendedSystemProp = prop; 
    } 
} 

ISystem genericSystem = new B(); 
genericSystem.GetPropertyInformation(); 

(genericSystem as B).ExtendedSystemProp = "value"; 

Answer 4

Вы не можете передать дополнительный параметр в переопределение. Когда вы переопределяете, вы переопределяете метод с точной подписью. Я бы предложил вам передать параметр интерфейса, например IPropertyInformation, который может быть изменен для каждой реализации.

Решение идти с базовым классом или интерфейсом для вашей реализации действительно зависит от вашего использования. У A-I достаточно общего между собой, что они должны действительно все происходить из одного и того же базового класса? Если это так, используйте базовый класс. Неужели это всего лишь GetPropertyInformation является общим, и в противном случае системы полностью функционально отличаются? Тогда вы действительно просто хотите, чтобы они делились интерфейсом.

Answer 5

Другие рассмотрели то, что первоначально было в моем ответе, но о точке «добавления параметра»: не забывайте, что последний C# также позволяет вам иметь необязательные параметры в методах.

Answer 6

Если у вас нет веской причины, я бы пошел с интерфейсом. Публичные виртуальные методы, хотя многое сделано, составляют not ideal.