оке, на что скидка влияет?

на всю покупку

оке

добавим сервис DiscountCalculator

сделаем там нужные проверки

секу, набросаю gist

да давай гист

p.s. блин надо все же для таких эксперементов сделать проект на гитхабе таки

я ж тебе за это уже хз сколько говорю

предлагаешь мне на недельку взять отпуск что бы попилить магазин?)

та он пилится за пару вечеров на выходных)

один хер бездельничаешь)

дай пол часа

запилю набросок

?

https://github.com/fesor/shop-example

добавить 2 продукта в карт, и цена у них должна стать на 10% меньше у всего карта

$cart->totalPrice($discountCalculator)

не

?

мм.. скажем, по сущностям по каждой нужно посчитать новый ценник

эм... не по каждой

блин у меня нет Cart

надо добавить походу

только Cart может знать что у нас 2 айтема одинаковых в карте

ну вот хочу чтобы в карте на UI у меня показывало по каждому продукту сколько ценник например я добавил Властелин колец 2 книги(по 10$) и Гаррипоттера (20$) тотал без скидки 40$, со скидкой 36$ и по каждому продукту Властелин колец 9$(10$)х2 = 18$ Гаррипоттер 18$(20$)х1 = 18$

у меня за это сейчас отвечает OrderLine

если влоб - то в методе price мы просто добавляем проверку

давай влоб))

public function price(): Money { if ($this->quantity == 1) { return $this->sellPrice; } return $this->sellPrice->multiply($this->quantity)->multiply(0.9); }

влобее не придумаешь)

? норм

но я уже с такой реализацией кучу проблем могу придумать)

ну для текущего варианта сойдет, щас следующее требование напишу

public function price(): Money { if ($this->quantity == 1) { return $this->sellPrice; } return $this->sellPrice->multiply($this->quantity)->multiply(0.9); }

а кстати оно ж проверяет только 1 книгу не?

ну для гаррипоттера не будет скидки

не понял

тип если мы покупаем 1-ую и 4-ую часть книги то всеравно надо получать скидку

?

это ж разные книги, одна серия - да

не, почитай выше кейс

ну вот хочу чтобы в карте на UI у меня показывало по каждому продукту сколько ценник например я добавил Властелин колец 2 книги(по 10$) и Гаррипоттера (20$) тотал без скидки 40$, со скидкой 36$ и по каждому продукту Властелин колец 9$(10$)х2 = 18$ Гаррипоттер 18$(20$)х1 = 18$

щас скидка работает для властелина колец

а для гаррипоттера работает?

погоди....

хм...

не так прочитал требования)

на весь карт надо скидку)

все таки надо формализовать требования, сейчас...

https://github.com/fesor/shop-example/pull/1

блин опять не все требования

да)

еще ж показывать по каждому айтему ценники

ай мне лень, давай завтра

))

? окей

сразу тебе наброшу, чтобы ты прикинул заранее: у книг могут быть свои издатели на каждого издателя мы накидываем свою маржу, индивидуально(где-то 50%, где-то 30%). т.е у нас стоит цена в каталоге 10$, а юзеру показываем 13$(если 30% маржи) на каждого издателя свой набор правил скидок. к примеру наборы: если положили в корзину N продуктов от одного издателя, то даем N% скидку если в карте лежит больше чем на N$, тогда даем N$ скидку если сегодня N день недели даем N% скидку (advanced) если юзер купил в прошлом месяце больше чем на N$, даем N% скидку

То есть мы делаем магазин книг?

да давай книг, вроде все просто

если тут есть желающие присоеденится к реализации то велкам цель: придерживаясь ооп практик реализовать требования)

Ну определенно Cart есть, и есть процессоры, которые дальше этот карт разбирают и сами в нужных местах аплаются

Давайте что ли тоже попробую накидать

можешь к фесору в репос набрасывать)

Я пока свой инитнул, потом посмотрим

@Enleur, ну кстати у тебя в условии не хватает ответа на вопрос, что раньше применять, проценты или суммы

Так что пусть будут суммы, а то жирные больно скидки

проценты от базы считать

точнее, база + маркап

Ну маркап и база это ладно, но если у меня корзина попадает под 2+ условия скидок на разные критерии, я должен что-то применить первым

Ну маркап и база это ладно, но если у меня корзина попадает под 2+ условия скидок на разные критерии, я должен что-то применить первым

да я так и понял. считай от базовой стоимости, без учета других скидок

ну т.е они по сути суммируются, скидки

да я так и понял. считай от базовой стоимости, без учета других скидок

ну такое... обычно выбирается максимальная

ок, давайте решим как лучше

100€cart - 5% - 10€ == 85 100€cart - 10€ - 5% == 85.5

Ну это так, implementation detail

можно сделать проще - "хз, запустим A/B тестирование и решим с чем конверсия выше"

Это да, это всегда работает)

всегда кстати интересовало каким образом считаются скидки, но осбо не задавался вопросом

всегда кстати интересовало каким образом считаются скидки, но осбо не задавался вопросом

это бизнес решать должен) для пользователей выгоднее вариант при котором выходит меньше прайс. Для продукт оунера - где выше прайс

>100€cart - 5% - 10€ давай тогда ASC модель

>100€cart - 5% - 10€ давай тогда ASC модель

wat?

100€cart - 5% - 10€ == 85

Ну в общих чертах типа так https://github.com/kix/shop-example

Щас правила поимплеменчу ради примера

Хочется https://github.com/lstrojny/functional-php для общего развития поюзать, вроде применимо

abstract class DiscountRule почему абстракт класс вместо интерфейса выбрал?

Хм, так-то зря, да

Можно конечно потеоретизировать, что, мол, мы их будем куда-то в реестр собирать, а потом галочкой в интерфейсе инейблить-дизейблить

А, не, гоню

Ну, короче, мне видимо что-то подсказывает, что тут скорее всего внутри какой-нибудь проп при инициализации общий таки повиснет для всех

Но пока необходимости нет, да, надо интерфейс

а заделай examples где будет пример всего в сборе. а то по спекам всю картину не видно

Ага, я пока хочу хотя бы один рул заимплементить примера ради

Ну наверное дабы не копаться в functional-php, пока накостылить можно, а потом и по красоте попробовать

Знаю, кстати, что плохо и что не нравится. Корзина мутабельная, но это тоже можно поправить

Ну вроде того: https://github.com/kix/shop-example/blob/master/example/example.php

>на каждого издателя свой набор правил скидок а щас они общие)

Рулы в мап к издателю сунуть, ну по айдишнику как вариант. И матчить соответственно

А, ну и маржа скорее всего не DiscountRule, а что-то ещё. Хотя с издателем можно и притянуть