c# - 避免更改多个类的设计建议

Question

我正在尝试弄清楚如何更优雅地设计一个小型应用程序，并使其更能抵抗变化。

基本上它是一种项目价格计算器，问题是有很多参数会影响定价。我试图避免为每个参数使用大量 if 子句使代码困惑，但我仍然有例如两个地方的 if 子句检查 size 参数的值。

我有一本 Head First Design Patterns 书，并试图在那里寻找想法，但我得到的最接近的是装饰器模式，它有一个例子，星巴兹咖啡首先根据添加的调味品设定价格，然后再根据通过添加尺寸参数(Tall、Grande、Venti)进行练习。但这似乎没有帮助，因为添加该参数似乎仍然在很多地方增加了 if 子句的复杂性(这是一个练习，他们没有进一步解释)。

我试图避免的是，如果要更改一个参数或添加一个新参数，或者至少在尽可能少的地方进行更改，则必须更改几个类(对此我有一些花哨的设计原则词)请记住 :-))。

下面是代码。基本上，它计算具有“写作”和“分析”任务以及大小参数和不同定价模型的项目的价格。稍后还会有其他参数，例如“产品有多新？” (新的、1-5 岁的、6-10 岁的)等等。任何关于最佳设计的建议都将不胜感激，无论是“设计模式”还是使其能够抵抗变化的良好面向对象原则(例如添加另一种尺寸，或更改其中一个尺寸值，并且只需在一个地方而不是在几个 if 子句中进行更改):

public class Project
{
    private readonly int _numberOfProducts;
    protected Size _size;
    public Task Analysis { get; set; }
    public Task Writing { get; set; }

    public Project(int numberOfProducts)
    {
        _numberOfProducts = numberOfProducts;
        _size = GetSize();
        Analysis = new AnalysisTask(numberOfProducts, _size);
        Writing = new WritingTask(numberOfProducts, _size);

    }

    private Size GetSize()
    {
        if (_numberOfProducts <= 2)
            return Size.small;
        if (_numberOfProducts <= 8)
            return Size.medium;
        return Size.large;
    }
    public double GetPrice()
    {
        return Analysis.GetPrice() + Writing.GetPrice();
    }
}

public abstract class Task
{
    protected readonly int _numberOfProducts;
    protected Size _size;
    protected double _pricePerHour;
    protected Dictionary<Size, int> _hours;
    public abstract int TotalHours { get; }

    public double Price { get; set; }

    protected Task(int numberOfProducts, Size size)
    {
        _numberOfProducts = numberOfProducts;
        _size = size;
    }

    public double GetPrice()
    {
        return _pricePerHour * TotalHours;
    }
}

public class AnalysisTask : Task
{
    public AnalysisTask(int numberOfProducts, Size size)
        : base(numberOfProducts, size)
    {
        _pricePerHour = 850;
        _hours = new Dictionary<Size, int>() { { Size.small, 56 }, { Size.medium, 104 }, { Size.large, 200 } };
    }

    public override int TotalHours
    {
        get { return _hours[_size]; }
    }
}

public class WritingTask : Task
{
    public WritingTask(int numberOfProducts, Size size)
        : base(numberOfProducts, size)
    {
        _pricePerHour = 650;
        _hours = new Dictionary<Size, int>() { { Size.small, 125 }, { Size.medium, 100 }, { Size.large, 60 } };
    }

    public override int TotalHours
    {
        get
        {
            if (_size == Size.small)
                return _hours[_size] * _numberOfProducts;
            if (_size == Size.medium)
                return (_hours[Size.small] * 2) + (_hours[Size.medium] * (_numberOfProducts - 2));
            return (_hours[Size.small] * 2) + (_hours[Size.medium] * (8 - 2)) + (_hours[Size.large] * (_numberOfProducts - 8));
        }
    }
}

public enum Size
{
    small, medium, large
}

public partial class Form1 : Form
{
    public Form1()
    {
        InitializeComponent();
        List<int> quantities = new List<int>();

        for (int i = 0; i < 100; i++)
        {
            quantities.Add(i);
        }
        comboBoxNumberOfProducts.DataSource = quantities;
    }

    private void comboBoxNumberOfProducts_SelectedIndexChanged(object sender, EventArgs e)
    {
        Project project = new Project((int)comboBoxNumberOfProducts.SelectedItem);
        labelPrice.Text = project.GetPrice().ToString();
        labelWriterHours.Text = project.Writing.TotalHours.ToString();
        labelAnalysisHours.Text = project.Analysis.TotalHours.ToString();
    }
}

最后是设置大小的组合框的更改事件中的一个简单的当前调用代码...(顺便说一句，我不喜欢这样的事实，我必须在最后使用几个点才能到达 TotalHours据我所知，这里要么违反了“最少知识原则”或“得墨忒耳法则”，因此对此的投入也将不胜感激，但这不是问题的重点)

问候，

安德斯

Answer 1

首先，我认为您应该重新考虑您的设计。项目看起来不像那样，据我查看您的代码，您无法向项目添加更多任务。还要考虑将项目和计算奖金的方式分开。如果你有不同的计算方法怎么办？这也与责任有关，很快您的项目可能会增长，并且很难将您计算价格的方式与项目结构分开。通常避免“如果”是使用多态性来完成的——也许你会喜欢根据它们的参数有不同的项目类型。这可以使用工厂方法来实现，它将接受参数，执行一次“如果”，然后创建一些项目子类型，它将知道如何正确计算其奖金。如果您将项目和计算分开，则考虑改为使用策略模式来计算奖金。在这里对得墨忒耳法则的关注就足够了，因为您公开了任务。尝试使用一种将返回总价并进行委托(delegate)的方法。原因是，那个方法所在的类(项目或计算策略)可以决定如何计算它，它也可以从其他任务中获取信息。如果您打算添加更多任务，则必须调整方法，可能使用一种带有字符串或枚举参数的方法来选择具体任务来计算奖品。顺便提一句。你为什么这么用下划线？