Нормальная ли такая практика?

Question

[maaestr0]

Здравствуйте. У меня есть продукт и скидка на него. В карточке продукта необходимо отобразить эту скидку. Я делаю следующим образом:

<div class="courses-body__prices">
                
                    @if (!$course->discount_exists)
                        <span class="courses-body__price">
                             {{$course->price->price}}$
                        </span>
                    @else
                        <s class="courses-body__price_crossed">
                            {{$course->price->price}}$
                        </s>  
                        <span class="courses-body__price">
                            {{$course->price->price - ($course->price->price / 100 * $course->discount->first()->size_discount)}}$
                        </span>
                    @endif
    
            </div>

Нормальная ли практика писать такие большие условия в шаблоне?

Comments

[Василий Банников]

Условие вроде не большое.
Почему нет?

Я бы правда вот этот кусок лучше бы в метод какой вынес:

$course->price->price - ($course->price->price / 100 * $course->discount->first()->size_discount)

Answer 1

[Сергей delphinpro]

@if ($course->discount_exists)
    <s class="courses-body__price_crossed">
        {{$course->price->price}}$
    </s>  
@endif
<span class="courses-body__price">
    {{$course->price->price_with_discount}}$
</span>

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

Comments

[maaestr0]

Не совсем понял. Свойство price_with_discount нужно вручную добавить как-нибудь?

[Сергей delphinpro]

maaestr0, Вот такой метод

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

Сделает доступным в модели свойство
$model -> price_with_discount

Это основы. Написано в документации по Eloquent. Обязательно почитайте.

Вам нужно будет как-то пробросить в объект Price скидку из Course, сразу посчитать ее и потом только возвращать результаты.

[maaestr0]

Сергей delphinpro, price with discount не отображалась у меня. Сделал как написано в документации: "If you would like these computed values to be added to the array / JSON representations of your model, you will need to append them."

public function getPriceWithDiscountAttribute(){
        return $this->attributes['price_with_discoutn'] = 
               $this->price->price - ($this->price->price / 100 * $this->discount->size_discount);
    }

Надеюсь не очень костыльно...

[Сергей delphinpro]

maaestr0, нет!
Зачем вы это пишете: $this->attributes['price_with_discoutn'] ?
https://docs.rularavel.com/docs/8.x/eloquent-mutat...

Answer 2

[pLavrenov]

Защиты от дурака нет)
Если $course->discount_exists === true но $course->discount->first() нет то страница отлетит.

Answer 3

[Василий Банников]

Моё мнение: всё ок, но вот это лучше вынести в функцию/метод:

$course->price->price - ($course->price->price / 100 * $course->discount->first()->size_discount)