Как верно заархитектурить заказ?

Question

[Андрей Прозоров]

Есть заказ. Есть метод калькуляции заказа. Метод нужен во многих местах. Размешать метод можно в сущности заказа или в сервисе заказы (или калькулятор заказа, или свой вариант).
И две ситуации:
1) Стоимость заказа зависит только от собственных свойств
2) Стоимость заказа зависит от всякой лабуды например внешние апи, скидки, день недели

Где бы вы разместили метод расчета стоимости заказа в этих двух разных ситуациях?

Comments

[sim3x]

KISS

Answer 1

[Северное Сияние]

Есть метод калькуляции заказа. Метод нужен во многих местах.

в этом и ошибка - все в кучу. Надо сделать, что бы калькуляция была не черным ящиком, а настраиваемым удобным интерфейсом:

$discountCalculator = new discountCalculator(\ListItems $items);
$discountCalculator->addCondition(new SimpleDiscount());
$discountCalculator->addCondition(new BlackFridayDiscount());
$discountCalculator->addCondition(new ByDateDiscount('last day of this month'));
$discountCalculator->addCondition(new ApiDiscount());
$discountCalculator->calculate();

Comments

[Андрей Прозоров]

Это очень интересно. Вы могли бы привести примерный код класса discountCalculator ? Я буду чертовски признателен. Даже могу вам в знак благодарности закинуть денег на телефон))

[Северное Сияние]

Андрей Прозоров, только деньги слать не надо)))

Надо плясать от того, как у вас все реализовано - скидка дается на товары или на заказ в целом?
А может ли даваться скидка на несколько товаров в заказе? Все очень индивидуально.
Я лишь показал, что вы можете сделать настраиваемый калькулятор скидок. Тогда у вас не будет какого-то огромного супер-метода с расчетами. Но это решение будет ограничивать и создавать сложности.

Вы меня заморочили, вот такой говнокод получился, всё, заканчиваю это писать... это лишь прототип, не до это использовать, это наброски....

class Product
{
    public $price = 100;

    public $priceWithDiscount = 100;

    /**
     * @var DiscountCalculator
     */
    protected $calc;

    /**
     * @param DiscountCalculator $calc
     * @return $this
     */
    public function setDiscountCalculator(DiscountCalculator $calc)
    {
        $this->calc = $calc;

        return $this;
    }

    public function discountCalculate()
    {
        $this->calc->calculate($this);

        return $this;
    }
}


class DiscountCalculator
{
    protected $discounts;

    /**
     * @param Discount $discount
     * @return $this
     */
    public function addCondition(Discount $discount)
    {
        $this->discounts[] = $discount;

        return $this;
    }

    /**
     * @param Product $product
     */
    public function calculate(Product $product)
    {
        $discountsList = [];

        /* @var $discount Discount */
        foreach ($this->discounts as $discount) {
            $discount->setProduct($product);
            if ($discount->checkCondition()) {
                $discountsList[] = $discount->getPercent();
            }
        }

        if ($sum = array_sum($discountsList)) {
            $product->priceWithDiscount =
                $product->priceWithDiscount -
                ($product->priceWithDiscount / 100) * $sum;
        }
    }
}

abstract class Discount
{
    protected $product;

    protected $discount;

    abstract public function checkCondition();

    public function setProduct(Product $product)
    {
        $this->product = $product;
    }

    public function getPercent()
    {
        return $this->discount;
    }
}

class SimpleDiscount extends Discount
{
    // 5% скидка по этой акции
    protected $discount = 5;

    public function checkCondition()
    {
        return true;
    }
}

class ByDateDiscount extends Discount
{
    // 30% скидкапо этой акции
    protected $discount = 30;

    public function checkCondition()
    {
        // тут м не используем объект продукта для какой-то проверки,
        // просто проверим дату, что это вторник этой недели
        return new \DateTime() == (new DateTime('this week'))->add(new DateInterval('P1D'));
    }
}

$calc = (new DiscountCalculator())
                ->addCondition(new ByDateDiscount());
$product = (new Product())
                ->setDiscountCalculator($calc);

// в любой момент можем добавить еще условие калькуляции
$calc->addCondition(new SimpleDiscount());

echo $product->discountCalculate()->priceWithDiscount;

[BoShurik]

Василий Берестов, немного развил идею: https://github.com/BoShurik/discounts-example
Добавил обработку ситуации, когда надо применить только одну скидку из заданных.
К примеру, если есть скидки для 3 и 5 итемов в заказе, должна применяться только максимальная скидка

[Северное Сияние]

BoShurik, ну ты маньяк. зачет!

[Андрей Прозоров]

Василий Берестов, Огромное спасибо!

[Андрей Прозоров]

BoShurik, Спасибо!