在应用策略模式时，是否有一个好方法来避免某些子类中未使用的方法参数？

Question

我有以下场景，其中我有不同类型的销售算法来计算销售价格。 FixSaleStrategy 不需要 basePrice 参数，而所有其他策略实现都需要它。有没有什么好的方法可以避免这个多余的参数呢？

public abstract class SalesStrategy
{
    public abstract double GetPrice(double basePrice, double saleAmount);
}
public class AmountOffSale : SalesStrategy
{
    public override double GetPrice(double basePrice, double salesAmount)
    {
        return basePrice - salesAmount;
    }
}
public class FixedPriceSale : SalesStrategy
{
    public override double GetPrice(double basePrice, double salesAmount)
    {
        return salesAmount;
    }
}

Answer 1

策略模式的核心是调用代码不知道被调用的实现。

如果您要更改每个实现使用的参数，您会发现您没有获得此模式的全部好处：调用者需要知道将使用哪个实现以及如何调用它。

我倾向于传递一个包含超集信息（例如 PricingInfo）的类，该类始终以相同的方式填充（理想情况下集中在代码中），唯一的区别是策略的实现。

好处之一是我可以向我的 PricingInfo 类添加一个过去不相关的属性（例如 systemDiscount），并且对整个系统的影响不会太大。

Answer 2

不，这不是一个多余的参数；它是一个多余的参数。使用 SalesStrategy 的代码不应该知道它正在使用哪个具体类，因此所有派生类中的方法签名必须相同。

Answer 3

如果您使用的是 c# 4.0，您可以反转参数并使 basePrice 可选，如下所示：

public abstract class SalesStrategy
{
    public abstract double GetPrice(double saleAmount, double basePrice = 0d);
}

public class AmountOffSale : SalesStrategy
{
    public override double GetPrice(double salesAmount, double basePrice)
    {
        return basePrice - salesAmount;
    }
}

public class FixedPriceSale : SalesStrategy
{
    public override double GetPrice(double salesAmount, double basePrice = 0d)
    {
        return salesAmount;
    }
}

这意味着可以执行以下操作...

FixedPriceSale fixedPrice = new FixedPriceSale();
...
fixedPrice.GetPrice(salesAmount);

请注意 AmountOffSale< /code> 的 basePrice 参数不是可选，这意味着以下内容将无法编译：

AmountOffSale amountOffSale = new AmountOffSale();
...
// No overload for method 'GetPrice' takes 1 arguments
amountOffSale.GetPrice(salesAmount); 

Answer 4

在我看来，这不是一个好主意。我会保持原样。您可以使用多种技巧，例如 params （有一个参数 double[]priceData）或 IDynamicObject。但最干净的方法就是让一些策略忽略额外的参数。

Answer 5

从接口中删除不相关参数的一个好方法是在子类的构造函数中传递这些参数。因此，您的设计的另一种选择是：

public interface SalesStrategy
    {
        double CalculatePrice(double basePrice);
    }

public class FixedPriceSale : SalesStrategy
    {
        public double CalculatePrice(double basePrice)
        {
            return basePrice;
        }
    }

public class AmountOffSale : SalesStrategy
    {
        public double SalesAmount { get; set; }

        public AmountOffSale(double salesAmount)
        {
            this.SalesAmount = salesAmount;
        }

        public double CalculatePrice(double basePrice)
        {
            return basePrice - SalesAmount;
        }
    }

在该构造中，您不会用子类中的特定数据污染您的界面。

Answer 6

另一种选择是使用参数对象或Dictionary。通过这种方式，您可以合并每个方法的参数数量，并为将来需求发生变化时添加额外参数留出空间。

一个缺点是 Dictionary 会使在代码中跟踪参数变得更加困难，而参数对象只包含您可以在代码中查看的所有属性。