2.5.14), есть ентити, где много чего понавешано
допустим, кастомер, у которого может быть скидка на отдельные товары:
class Customer {
/**
* @ORM\OneToMany(targetEntity="\ProductDiscount", mappedBy="customer", cascade={"persist"}, orphanRemoval=true)
*/
private $productDiscounts;
public function getProductDiscount(Product $product) {
$criteria = ...
$discount = $this->productDiscounts->matching($criteria)->first();
return $discount->percent();
}
}
для отдельных товаров работает, но если запрашивать скидку для группы товаров, то из-за matching не вся коллекция запрашивается, а только часть
т.е. для n товаров будет N запросов, естесно все тормозит
можно легко пофиксить добавив $this->discounts вначале, но об этом легко забыть, да и мест таких дофига (не только товары, но и сервисы и прочее), хотелось бы подчистить
придумал создать класс со скидками, где каждая скидка в отдельном классе, т.е.
class ProductDiscounts {
/**
* @ORM\OneToMany(targetEntity="ProductDiscount", mappedBy="discounts", cascade={"persist"}, orphanRemoval=true)
*/
private $productDiscounts;
public function getDiscount(Product $product) {
$criteria = ...
$discount = $this->productDiscounts->matching($criteria)->first();
return $discount->percent();
}
}
class Discounts {
/**
* @ORM\OneToOne(targetEntity="ProductDiscounts")
* @ORM\JoinColumn(name="id", referencedColumnName="id")
*/
private $productDiscounts;
...
public function products() {
return $this->productDiscounts;
}
...
}
class Customer {
/**
* @ORM\OneToOne(targetEntity="ProductDiscounts")
* @ORM\JoinColumn(name="id", referencedColumnName="id")
*/
private $discounts;
public function getProductDiscount(Product $product) {
return $this->discounts->products->getDiscount(product)->percent();
}
}
для Discounts и ProductDiscounts мне отдельные таблички не нужны - достаточно существущей кастомера, т.е. те классы я замапил на существующую табичку
все работает, но со скидками можно работать только после того, как запросишь кастомера/скидки с базы:
$customer = $em->find(Customer::class, $customerId);
$customer->discounts()->products()->setDiscount($product, '12.3');
$em->flush();
// или
$discounts = $em->find(Discounts::class, $customerId);
$discounts->products()->setDiscount($product, '12.3');
$em->flush();
а вот если со скидками работать сразу после создания объекта - вылетает ошибка
т.е. следующее не работает:
$customer = new Customer();
$customer->discounts()->products()->setDiscount($product, '12.3');
$em->flush();
пробовал разное, и ошибки разные: нет персиста, или существующий id уже есть и т.д.
на данный момент таких мест с ошибкой нет, но они легко могут появиться
и как здесь быть, чтобы работало всегда, а не только после загрузки кастомера?
можно, конечно, для всего сделать отдельные таблички и там все автоматом будет персиститься/сохраняться, но там, по хорошему, будет только одно поле: customerId, не нравится мне такой оверхед, хотелось бы на одной все разрулить, но какие варианты для этого есть?
ну или по доктрине мож кто канал посоветует
что-то типа такого интересует: https://www.doctrine-project.org/projects/doctrine-orm/en/current/cookbook/decorator-pattern.html#persisting-the-decorator-pattern только чтобы декоратор именно как декоратор был, а не как наследник, которого в базе отдельно храним
Обсуждают сегодня