牢固的本金违规

Question

以下代码违反了坚实的本金，有人知道我应该如何重构吗？ 我找不到任何解决方案来重构此代码以遵循坚实的主体。

public class Calculator
{
    public int Calculate(int a, int b, string operation)
    {
        int result = 0;
        if (operation.Equals("add"))
        {
            result = a + b;
        }
        else if (operation.Equals("multiply"))
        {
            result = a * b;
        }
        else if (operation.Equals("devide"))
        {
            result = a / b;
        }
        else if (operation.Equals("subtract"))
        {
            result = a - b;
        }
        return result;
    }
}

据我了解，这可能只是违反班级的单一责任。但是我找不到重构的方法。

Answer 1

我猜问题是您有一个单个函数执行许多计算，因此这样做不再简单，并且不仅仅是一个责任。

没有替换或扩展的方法，这通常来自使用实现固定接口的类，然后根据需要创建一个实例。

从其可以执行的计算中隔离的计算类别都不是。

如果您希望添加另一种计算类型，例如Math.pow（a，b）您需要更改伸展switch> switch语句的函数，从而更改您的计算类。

如果您可以拥有一个接口：

public interface ICalculation
{
    public int Calculate(int a, int b);
}

那么您有四个类add，减去，乘数，divide。每个类都实现一个算术公式。

您的计算器类可以实现icalculation类型实例的列表，例如
list＆lt; icalculator＆gt;计算=新列表＆lt; calculator＆gt; = {new add（），new subtract};

，或者您可以将计算器注入施工时间支持的类型列表
例如

class Calculator
{
    private List<ICalculator> _calculations;
    Calculator(List<ICalculator> calculations)
    {
        _calculations = calculations;
    }
}

calculator.calculate可以将索引输入列表中以选择计算类型

Calculate(uint index, int a, int b)
{
    //TODO: remember to bounds check index against _calculations.Count
    return _calculations[index].Calculate(a,b);
}

或calculator.calculate可以更改以允许注入

Calculate(ICalculation calculation, int a, int b)
{
    //TODO: null check required
    return calculation.Calculate(a,b);
}

具有较小更改的 计算实例您可以扩展计算器类，以使其可以将计算串在一起。

您能看到随着进步的方式，计算器类似乎可以获得更多功能，而实际上对其可以执行的计算有任何了解？