基于多个变量状态的条件语句的良好设计模式

Question

我正在制定一种方法来计算用 PHP 编写的购物车的总计，并且希望获得一些关于处理不同条件的良好设计模式的反馈。我试图为管理员提供多种计算折扣的策略。管理员可以选择在税费之前或之后应用折扣，以及是否对运费应用折扣。这概述了我的任务。

变量

对于此任务，我有以下变量以及可能的值：

$tax_option : 'before', 'after'

$shipping_option : ' yes', 'no'

除了这两个变量之外，计算总价值的公式也会根据 $subtotal（购物车中商品的数量）和$reduction（要折扣的总金额）。

一般来说，我的逻辑是测试 $tax_option 和 $shipping_option 变量的 4 种组合。对于 $subtotal 小于或等于 $reduction 的情况，我还需要更改公式。总共，我有8种不同的条件。

可能性

我认为这里确实有 3 种不同的选择：if 语句、switch 语句或策略模式。我将展示 if 语句和策略模式的结构，但排除 switch 的可能性，因为在这种情况下这似乎不正确。

if 语句

这是我正在考虑的 if 语句的一般模式。请注意，我知道我可以稍微调整一下结构，但无论出于何种原因，这对我来说都更具可读性。

if($tax_option == 'after' && $shipping_option == 'yes')
{
    if($subtotal <= $reduction)
    {

    }
    else
    {

    }
}
elseif($tax_option == 'before' && $shipping_option == 'yes')
{
    if($subtotal <= $reduction)
    {

    }
    else
    {

    }
}
elseif($tax_option == 'before' && $shipping_option == 'no')
{
    if($subtotal <= $reduction)
    {

    }
    else
    {

    }
}
elseif($tax_option == 'after' && $shipping_option == 'no')
{
    if($subtotal <= $reduction)
    {

    }
    else
    {

    }
}
else
    $new_total = $total;

策略模式

在网上寻找此问题的解决方案后，我了解到策略模式，看起来棒极了。我在这里开始研究它，不介意一些反馈。到目前为止，它看起来像下面的代码，明显删除了一些例程。

class DPPCalculateTotal
{
    protected $formulas = array();

    public function DPPCalculateTotal($tax_option, $shipping_option, $total, $subtotal, $shipping, $tax, $coupons_amount, $reduction)
    {
        foreach($this->formulas as $formula)
        {
            if($formula->test($tax_option, $shipping_option, $total, $subtotal, $shipping, $tax, $coupons_amount, $reduction))
            {
                return $formula->calculate_total($tax_option, $shipping_option, $total, $subtotal, $shipping, $tax, $coupons_amount, $reduction);
            }
        }
    }

    function add_formula(DPPFormula $formula)
    {
        $this->formulas = $formula;
    }
}

interface DPPFormula
{
    public function test($tax_option, $shipping_option, $total, $subtotal, $shipping, $tax, $coupons_amount, $reduction);

    public function calculate_total($tax_option, $shipping_option, $total, $subtotal, $shipping, $tax, $coupons_amount, $reduction);
}

class AfterTaxesYesShippingGreaterSubotal implements DPPFormula
{
    public function test($tax_option, $shipping_option, $total, $subtotal, $shipping, $tax, $coupons_amount, $reduction)
    {

    }

    public function calculate_total($tax_option, $shipping_option, $total, $subtotal, $shipping, $tax, $coupons_amount, $reduction)
    {

    }
}

class AfterTaxesYesShippingLesserSubotal implements DPPFormula
{
    public function test($tax_option, $shipping_option, $total, $subtotal, $shipping, $tax, $coupons_amount, $reduction)
    {

    }

    public function calculate_total($tax_option, $shipping_option, $total, $subtotal, $shipping, $tax, $coupons_amount, $reduction)
    {

    }
}

class AfterTaxesNoShippingGreaterSubotal implements DPPFormula
{
    public function test($tax_option, $shipping_option, $total, $subtotal, $shipping, $tax, $coupons_amount, $reduction)
    {

    }

    public function calculate_total($tax_option, $shipping_option, $total, $subtotal, $shipping, $tax, $coupons_amount, $reduction)
    {

    }
}

class AfterTaxesNoShippingLesserSubotal implements DPPFormula
{
    public function test($tax_option, $shipping_option, $total, $subtotal, $shipping, $tax, $coupons_amount, $reduction)
    {

    }

    public function calculate_total($tax_option, $shipping_option, $total, $subtotal, $shipping, $tax, $coupons_amount, $reduction)
    {

    }
}

class BeforeTaxesYesShippingGreaterSubotal implements DPPFormula
{
    public function test($tax_option, $shipping_option, $total, $subtotal, $shipping, $tax, $coupons_amount, $reduction)
    {

    }

    public function calculate_total($tax_option, $shipping_option, $total, $subtotal, $shipping, $tax, $coupons_amount, $reduction)
    {

    }
}

class BeforeTaxesYesShippingLesserSubotal implements DPPFormula
{
    public function test($tax_option, $shipping_option, $total, $subtotal, $shipping, $tax, $coupons_amount, $reduction)
    {

    }

    public function calculate_total($tax_option, $shipping_option, $total, $subtotal, $shipping, $tax, $coupons_amount, $reduction)
    {

    }
}

class BeforeTaxesNoShippingGreaterSubotal implements DPPFormula
{
    public function test($tax_option, $shipping_option, $total, $subtotal, $shipping, $tax, $coupons_amount, $reduction)
    {

    }

    public function calculate_total($tax_option, $shipping_option, $total, $subtotal, $shipping, $tax, $coupons_amount, $reduction)
    {

    }
}

class BeforeTaxesNoShippingLesserSubotal implements DPPFormula
{
    public function test($tax_option, $shipping_option, $total, $subtotal, $shipping, $tax, $coupons_amount, $reduction)
    {

    }

    public function calculate_total($tax_option, $shipping_option, $total, $subtotal, $shipping, $tax, $coupons_amount, $reduction)
    {

    }
}

问题

1) 您认为，进行此操作的最佳方法是什么？

2）本例中策略模式的优点是什么？

3）由于这是我第一次尝试策略模式，看起来我的方向正确吗？

非常感谢您的投入！我很高兴学习这种模式，并且非常感谢我能得到的任何反馈！

Answer 1

我认为你的方向是正确的。使用策略模式，当应用程序变得更大或需求发生变化时，以后添加新策略真的很容易（根据我的经验，十有八九是这种情况）。

我建议再创建一个名为 Order 的类，将所有订单详细信息封装在其中，然后传递对象。您的测试和计算方法会更简洁一些，

interface DPPFormula
{
    public function test(OrderInteface $order);

    public function calculate_total(OrderInterface $order);
}

interface OrderInterface
{
    function setTotal($total);
    function getTotal();
}

然后您可以执行类似的操作，

$order->setTotal($calculator->DPPCalculateTotal());

具体取决于您的复杂性，您可能希望也可能不想使用Order 的接口。我强烈建议您使用一个，因为这进一步增加了抽象性。

Answer 2

我认为可以将总额分为可折扣金额和不可折扣金额。然后，您将折扣应用于可折扣金额，并添加其余部分，如下伪代码：

extra = 0
if (tax_option)
  subtotal += tax(subtotal)
else
  extra += tax(subtotal)
if (delivery_option)
  subtotal += delivery
else
  extra += delivery
if (subtotal > reduction)
  subtotal -= reduction
else
  // stuff here
subtotal += extra

Answer 3

您是否考虑过使用查找表/数组？可能会让其他人更清楚一些。数组键可以是各种组合的字符串，然后您可以将所需的任何值与查找键相关联。

$lookup = array(
    'taxo-after:shipo-yes'=> array('reduction'=>100),
    'taxo-after:shipo-no'=> array('reduction'=>100),
    'taxo-before:shipo-yes'=> array('reduction'=>100),
    'taxo-before:shipo-no'=> array('reduction'=>100),
    ...
 );

 $lookup_key = 'taxo-'.$tax_option.':'.'shipo-'.$shipping_option;
 if ( $subtotal < $lookup[$lookup_key]['reduction'] ) {
 } else {
 }

这几乎取代了您的整个第一个代码示例。 1 个数组声明和 1 个 if 语句。您可以添加一百种可能性，而不会造成任何性能损失或更多代码。