Рефакторинг класс дизайна для передачи намерений дизайна

Question

У меня есть следующий класс дизайн. Полный код доступен в «How to achieve this functionality using Generics?». Код прекрасно работает и решает проблему литья, упомянутую в «Refactoring Code to avoid Type Casting»

В RetailInvestmentReturnCalculator классе метод GetInvestmentProfit() использует CalculateBaseProfit() метод, присутствующий в InvestmentReturnCalculator абстрактного базового класса. Этот факт не проявляется в дизайне класса.

ВОПРОС

1. Как реорганизовать этот дизайн класса передать вышеупомянутый факт?
2. Что такое руководство по дизайну, которое предотвратит подобные ошибки дизайна?

Примечание: Martin Fowler: Is Design Dead? говорит

Что мы подразумеваем под архитектуру программного обеспечения? Для меня термин архитектура передает понятие основных элементов системы, которые трудно изменить. Основа, на которой остальные должны быть построены

Class Diagram

Аннотация

public abstract class InvestmentReturnCalculator 
{ 
    #region Public 

    public double ProfitElement { get; set; } 
    public abstract double GetInvestmentProfit(); 

    #endregion 

    #region Protected 

    protected double CalculateBaseProfit() 
    { 
     double profit = 0; 
     if (ProfitElement < 5) 
     { 
      profit = ProfitElement * 5/100; 
     } 
     else 
     { 
      profit = ProfitElement * 10/100; 
     } 
     return profit; 
    } 

    #endregion 
} 

public abstract class InvestmentReturnCalculator<T> : InvestmentReturnCalculator where T : IBusiness 
{ 
    public T BusinessType { get; set; } 
} 

Бетон

public class RetailInvestmentReturnCalculator : InvestmentReturnCalculator<IRetailBusiness> 
{ 
    public RetailInvestmentReturnCalculator(IRetailBusiness retail) 
    { 
     BusinessType = retail; 
     //Business = new BookShop(100); 
    } 

    public override double GetInvestmentProfit() 
    { 
     ProfitElement = BusinessType.GrossRevenue; 
     return CalculateBaseProfit(); 
    } 
} 

Answer 1

Поле ProfitElement довольно уродливое. Я бы сделал его абстрактным свойством на InvestmentReturnCalculator и реализовал его в базовых классах (вместо установки значения) - это называется template method pattern. Тогда вам не нужен метод GetInvestmentProfit().

public abstract class InvestmentReturnCalculator 
{ 
    #region Public 

    public abstract double ProfitElement { get; } 

    #endregion 

    #region Protected 

    public double GetInvestmentProfit() 
    { 
     double profit = 0; 
     if (ProfitElement < 5) 
     { 
      profit = ProfitElement * 5/100; 
     } 
     else 
     { 
      profit = ProfitElement * 10/100; 
     } 
     return profit; 
    } 

    #endregion 
} 

public abstract class InvestmentReturnCalculator<T> : InvestmentReturnCalculator where T : IBusiness 
{ 
    public T BusinessType { get; set; } 
} 

public class RetailInvestmentReturnCalculator : InvestmentReturnCalculator<IRetailBusiness> 
{ 
    public RetailInvestmentReturnCalculator(IRetailBusiness retail) 
    { 
     BusinessType = retail; 
     //Business = new BookShop(100); 
    } 

    public override double ProfitElement {get { return BusinessType.GrossRevenue;}} 

}