PHP - Laravel - 管理条件规则的最佳方式

Question

我有一个使用 Laravel 框架的应用程序，有一些条件规则我不知道什么是最好的编码和维护方式。

用例：有条件地应用促销代码

• 可以在特定日期或日期范围内应用促销代码
• 订单可使用促销代码 >= $100
• 可以为特定项目申请促销代码
• ...

基本的解决方案是编写多个 IF ELSE 语句来逐一检查。例如：

if ($promo->specific_date) {

} 
elseif ($promo->date_range >= 'date' && $promo->specific_date <= 'date') {

}

if ($totalAmount < 100) {
    // Dont allow
}

if (! $promo->allowed_items) {
    // Dont allow
}

// More conditions ...

我可以看到代码在测试和维护时会出现问题。

所以我想知道是否有更好的方法来处理这个问题？例如。使用 OOP 方式？

P/S：澄清我的用例：

• 我需要通过所有规则才能使促销有效
• 我正在考虑创建一个规则模型，以便我可以有一个 CRUD 来管理它们，并且在后端，我可以运行一个查询来获取所有规则，然后调用一个类来管道并检查每个规则......（不确定这是好主意还是坏主意）

谢谢，

Answer 1

您可以利用 Laravel 管道对您的订单进行某种检查。

想象一下，您可以从数据库中正确提取约束配置并构建一个数组（或类似 ConstraintBag 实例），其中包含您需要检查的所有约束：

configuration
$constraints = [
    DateRangeConstraint::class,
    TotalAmountConstraint::class,
    AllowedItemsContraint::class,
];

每个约束都可以遵循相同的接口（本 PoC 中的 Constraint），该接口将定义一个 handle 方法：

use Closure;

class DateRangeConstraint implements Constraint
{
    public function handle($order, Closure $next)
    {
        if ($order->promo->start_date >= 'date' || $order->promo->end_date <= 'date') {
            throw new PromotionConstraintException($this);
        }

        return $next($order);
    }
}

然后在您的控制器/服务方法中，您可以在管道中使用此规则数组，并通过约束传递订单对象（或包含验证所有约束所需的所有部分的对象）。如果其中任何一个失败，您可以触发自定义异常（可能每个约束类别一个/每个约束一个）并返回validation 流程的结果：

// Do not forget to add the use statement
use Illuminate\Pipeline\Pipeline;

class PromotionValidationService
{
    protected $constraints;

    // Pass in the constraints array you have already built
    public function __construct($constraints)
    {
        $this->constraints = $constraints;
    }

    // Start the validation process and cycle through all the constraints
    // I would pass in the order object as you might need to access the total
    // order amount and/or the items in the order
    public function validate($order)
    {
        try {
            app(Pipeline::class)
                ->send($order)
                ->through($this->constraints);
        } catch (PromotionConstraintException $exception) {
            // Handle the exception and return false or rethrow
            // the exception for further handling from the caller.
            return false;
        }

        return true;
    }
}

显然，这仍然是一个概念证明，需要更多的研究和架构规划来处理您可能需要检查的各种约束（例如：传递整个 $order 对象可能不是最好的主意，或者它可能检查促销约束时尚不可用）。但是，这可能是一个灵活的替代方案，可以替代需要针对每次更改进行编辑的固定 if/else 序列。

Answer 2

您可以在 Promotion 模型中定义促销属性（这意味着它们可能应该存储在您的数据库中的某个位置），然后有一个标准化的验证器，您可以调用任何促销。

这里有一些示例/伪代码来解释这个过程（它是 PHP 7.4，只需删除属性类型以使其适用于以前的版本）：

final class Promotion
{
  private DateTime $minDate;
  private DateTime $maxDate;
  private DateTime $minAmount;
  private array $allowedItems;

  public function getMinDate(): DateTime
  {
    return $this->minDate;
  }

  public function getMaxDate(): DateTime
  {
    return $this->maxDate;
  }

  public function getMinAmount(): DateTime
  {
    return $this->minAmount;
  }

  public function getAllowedItems(): array
  {
    return $this->allowedItems;
  }
}

final class PromotionValidator
{
  public function isPromotionValid(Promotion $promo, array $purchasedItems, int $totalAmount): bool
  {
    $now = new \DateTime();

    if ($now < $promo->getMinDate() || $now > $promo->getMaxDate()) {
      return false;
    }

    if ($totalAmount < $promo->getMinAmount()) {
      return false;
    }

    if (count(array_intersect($purchasedItems, $promo->getAllowedItems())) !== count($promo->getAllowedItems())) {
      return false;
    }

    return true;
  }
}

用法：

$promotionValidator = new PromotionValidator();
$promoIsValid = $promotionValidator->isPromotionValid($promo, $cartItems, $cartAmount);

Answer 3

在此处使用 switch 语句，因为它比 if

更快

默认设置promo_code 标志false 因为现在只想我的任何条件它为真然后设置promo_code 标志为真..

如果你想一个一个地使用 if 语句，那很好，因为它易于阅读和维护

$promo_code_flag = false;

if ($promo->specific_date) {
    $promo_code_flag = true;
} 
elseif ($promo->date_range >= 'date' && $promo->specific_date <= 'date') {
     $promo_code_flag = true;
}

if ($totalAmount > 100) {
    $promo_code_flag = true;
}

if ( $promo->allowed_items) {
    $promo_code_flag = true;
}

if($promo_code_flag) { 
 //allow promo code
}//other wise it will not allowe