Я пытаюсь объяснить своей команде, почему это плохая практика, и я ищу ссылку на анти-шаблон, чтобы помочь в моих объяснениях. Это очень крупное корпоративное приложение, поэтому здесь приведен простой пример, чтобы проиллюстрировать, что было реализовано:
public void ControlStuff()
{
var listOfThings = LoadThings();
var listOfThingsThatSupportX = new string[] {"ThingA","ThingB", "ThingC"};
foreach (var thing in listOfThings)
{
if(listOfThingsThatSupportX.Contains(thing.Name))
{
DoSomething();
}
}
}
Я предлагаю добавить свойство к базовому классу "Вещи", чтобы сообщить нам, поддерживает ли он X, поскольку подкласс Thing должен будет реализовать эту функциональность. Что-то вроде этого:
public void ControlStuff()
{
var listOfThings = LoadThings();
foreach (var thing in listOfThings)
{
if (thing.SupportsX)
{
DoSomething();
}
}
}
class ThingBase
{
public virtual bool SupportsX { get { return false; } }
}
class ThingA : ThingBase
{
public override bool SupportsX { get { return true; } }
}
class ThingB : ThingBase
{
}
Итак, довольно очевидно, почему первый подход - это плохая практика, но что это называется? Кроме того, есть ли образец, более подходящий для этой проблемы, чем тот, который я предлагаю?
