Как переопределить значение свойства сущности при помощи сервиса в Symfony2?

Question

[Qixing]

Добрый день.
Мне нужно выводить разную конечную цену на продукт в зависимости от условий ( скажем группы пользователя).
Если у меня везде прописано уже в шаблонах. {{ item.price }}
Как мне реализовать сервис, что бы он перехватывал $item->getPrice и возвращал нужное значение.
Ссылки на примеры приветствуются

Накидал пока так

<?php
namespace Bundle\Service;

class Price
{
    public function __construct($ratio , $minCommission) 
    {
        $this->ratio = $ratio;
        $this->minCommission = $minCommission; 
    }

    public function actualPrice($price)
    {
        return $price > 20000 ? $price * $this->ratio : $price + $this->minCommission;
    }
}

crbr.price:
        class: CrbrBundle\Service\Price
        arguments:
            - %price_ratio%
            - %price_minCommission

parameters:
     price_ratio : 1.1
     price_minCommission: 1500

Answer 1

[Юрий]

В вашем случае, я считаю, можно поступить иначе - реализовать twig фильтр, и в самом шаблоне уже делать так {{ item.price| actualPrice }} .

Comments

[Qixing]

Спасибо. Тоже хороший вариант, но мне тут больше сервис подойдет.

Answer 2

[Алексей Павлов]

прямо так - никак не сделать. Нужно достаточно сильно изменять работу с сущностью.
1) Можно сделать прокси-класс, имеющий расширенную логику в части некоторых свойств объекта, и передающий остальные свойства без изменений.
2) Можно обрабатывать нужные свойства отдельным вызовом перед передачей объекта в шаблон - передать объект сущности в метод сервиса, и там изменить значения свойств.
3) Можно поставить обработчик событий, который будет перехватывать системные события загрузки сущности и в обработчике изменять объект.
Сущности доктрины - это просто обычный класс. И в идеале не стоит передавать в неё что-то внешнее, привносить лишнюю зависимость (например, не надо передавать текущего пользователя, имеющего роли).

Comments

[Qixing]

Алексей, спасибо за ответ. Можно поподробней о способе 2)

[Алексей Павлов]

в контроллере:
$item = $this->get('item')->load($id); // тут уже всё подготовлено

в сервисе item загружать сущность из базы, сразу же после этого изменять нужные поля в соответствие с ролью пользователя (или что там по бизнес-логике) и отдавать уже готовую изменённую сущность в контроллер.

Если использовать тот сервис, который у вас описан в вопросе, то делать так (в контроллере):
$item = $this->get('item')->load($id); // тут сущность, полученная из базы, можно и репозиторий использовать
$price = $item->getPrice();
$price = $this->get('crbr.price')->actualPrice($price);
$item->setPrice($price);
Но лучше спрятать этот код в сервис.

[Алексей Павлов]

и не бойтесь рефакторить код! (изменять код для улучшения его архитектуры).
Чем раньше вы поймёте, как можно сделать код лучше, тем меньшими усилиями получится произвести эти изменения в коде. А если отложить рефакторинг на тот момент, когда кода станет в разы больше, то и желание рефакторинга уменьшится, и код так и останется неоптимальным.

[Qixing]

Алексей Павлов: у меня все получилось. Действительно сервисы это сила. Как я понял, даже все запросы к бд лучше держать там. У меня осталось два вопроса. Я подключаю
Проблема с $entityManager . У меня две бд, в аргументах к сервису передаю вот
- "@doctrine.orm.entity_manager" но тогда
use Doctrine\ORM\EntityManager;
public function __construct($ratio, $minCommission, EntityManager $entityManager)

Он не понимает где искать. Как мне передать entity_manager с нужным orm (у меня их два crbr и main)

Работает пока так
public function __construct($ratio, $minCommission, $doctrine) {}
$this->em = $doctrine->getManager('crbr');
Конфиг
- "@doctrine" #- "@doctrine"

Второй вопрос. А если у меня передается не одно значение item, а целая выборка для списка?
Как быть с обработкой - самый простой вариант какой?

[Алексей Павлов]

>Действительно сервисы это сила. Как я понял, даже все запросы к бд лучше держать там.
Как говорил и Сергей Протько, запросы к БД лучше держать в репозитории сущности. Чтобы всё, связанное с конкретной реализацией хранения, лежало в одном месте.
Получается трёхслойная архитектура:
- высокий уровень - контроллер - логика вывода страницы
- средний уровень - сервис - основная логика
- низкий уровень - репозиторий - связь с БД

Насчёт двух EntityManager-ов - попробуйте передавать их отдельными зависимостями в сервис. Вам про это уже говорил Сергей Протько в предыдущем вопросе. Вообще, Сергей очень дельные советы даёт, он гораздо лучше меня разбирается в Symfony и Doctrine.

По второму вопросу - сильно зависит от того, как БД спроектирована. Если есть возможность переложить работу на БД с помощью JOIN-ов, то это будет хорошим вариантом. А если нет - то цикл, старый добрый цикл.

[Сергей Протько]

Алексей Павлов: вообще по поводу слоев - есть такой термин как "шестигранная архитектура". То есть ваше приложение или его часть представляет собой шестигранник (на самом деле на количество граней пофигу просто так звучит круче). Каждая грань - это какой-то вариант взаимодействия - база данных, http, rest, связь с другими частями приложения... короче связь с внешним миром. Первым слоем идет в этом случае - слой фреймворка. Это собственно контроллеры и доктрина и т.д. Затем идет слой приложения, это наш сервисный слой. Репозитории относятся туда же. Далее идет слой предметной области где содержится вся бизнес логика (стоит разделять бизнес логику и логику приложения, к которой относятся такие вещи как "послать email кому-нибудь если все удалось"). Ну и в самом центре - энтити и value object-ы в которых большая часть бизнес логики приложений.

[Сергей Протько]

Алексей Павлов: в этом случае можно разделить приложение на много много шестигранников которые в сумме дают приложение.

[Алексей Павлов]

Сергей Протько: Похоже на модульную архитектуру, но взгляд на неё с точки зрения другой метафоры. Интересно, спасибо.

[Qixing]

Спасибо. Пытаюсь реализовать.
crbr.item_repository:
class: ParserBundle\Repository\CrbrItemRepository
factory-service: doctrine.orm.entity_manager
factory-method: getRepository
public: false
arguments: [ParserBundle:CrbrItem]

crbr.item:
class: CrbrBundle\Service\Item
arguments:
- %price_ratio%
- %price_minCommission%
- '@crbr.item_repository'

Вот беда, полчаса уже мучаюсь.
В моем репозитарии
public function findOne($crbrId)
{

$qb= $this->createQueryBuilder('i')
->where('i.number = :id')
->setParameter('id', $crbrId);

return $qb->getQuery()->getResult();

}

Получаю ошибку Error: Call to a member function createQueryBuilder() on null

[Алексей Павлов]

а какой класс получился у этого сервиса? напишите сюда, что выдаст команда die(get_class());

[Qixing]

ParserBundle\Repository\CrbrItemRepository и CrbrBundle\Service\Item . Как я понял проблема с $this->_entityName . Она пустая.

[Qixing]

joxi.ru/ZrJV8GPsW6L5rj

[Алексей Павлов]

ну, тут конкретная проблема не в $this->_entityName, а в $this->_em. Она должна устанавливаться сервис-контейнером, но не устанавливается. Не знаю даже, почему. Может, потому что у вас нет дефолтного EntityManager-а? попробуйте поставить в описание сервиса не
factory-service: doctrine.orm.entity_manager
а
factory-service: doctrine.orm
У ORM-а тоже есть свой getRepository.

[Qixing]

У меня же 2а orm для двух БД. crbr и main.
Ну вообщем беда. Я запутался, что у нас должно быть в конструкторе?

crbr.item_repository:
class: ParserBundle\Repository\CrbrItemRepository
factory-service: doctrine.orm
factory-method: getRepository
#public: false
arguments: [ParserBundle:CrbrItem]

<?php

namespace ParserBundle\Repository;

use Doctrine\ORM\EntityRepository;
use ParserBundle\Entity\CrbrItem;

class CrbrItemRepository extends EntityRepository
{
public function __construct($em, $classMetadata)
{

}

public function findOne($crbrId)
{

$qb = $this->createQueryBuilder('item')
->where('i.number = :id')
->setParameter('id', $crbrId);

return $qb->getQuery()->getResult();

}
}

Catchable Fatal Error: Argument 2 passed to ParserBundle

[Алексей Павлов]

ничего не должно быть в конструкторе - он наследуется из родительского класса. Вообще не должно быть конструктора в этом классе. А если есть, то нужно вызывать parent::__construct(...)

[Qixing]

Алексей Павлов: вообще не хочет. joxi.ru/E2pvRXOsXbGyrY
atchable Fatal Error: Argument 2 passed to Doctrine\ORM\EntityRepository::__construct() must be an instance of Doctrine\ORM\Mapping\ClassMetadata, none given,
Не передаются ему и точка.

services:
crbr.item_repository:
class: ParserBundle\Repository\CrbrItemRepository
factory-service: doctrine.orm.entity_manager # тут я пробовал по разному
factory-method: getRepository
#public: false
arguments: [ ParserBundle:CrbrItem ]

<?php

namespace ParserBundle\Repository;

use Doctrine\ORM\EntityRepository;

class CrbrItemRepository extends EntityRepository
{

public function findOne($crbrId)
{

$qb = $this->createQueryBuilder('item')
->where('i.number = :id')
->setParameter('id', $crbrId);

return $qb->getQuery()->getResult();

}
}

Конфиг

orm:
default_entity_manager: crbr
entity_managers:
main:
connection: main
mappings:
RedCodeCurrencyRateBundle: ~
CrbrBundle: ~
crbr:
connection: crbr
mappings:
ParserBundle: ~

P.S. Проблем с сущностями не было, до этого без сервисов все было ок.

[Qixing]

Была ошибка в service конфиге. Но не помогло все равно

services:
crbr.item_repository:
class: Doctrine\ORM\EntityRepository
factory-service: doctrine.orm
factory-method: getRepository
#public: false
arguments: [ ParserBundle:CrbrItem ]

Catchable Fatal Error: Argument 2 passed to Doctrine\ORM\EntityRepository::__construct() must be an instance of Doctrine\ORM\Mapping\ClassMetadata, none given, called in

[Qixing]

Вообщем нашел ответ.
factory-service: doctrine.orm
factory-method: getRepository

Нужно заменить на factory_service и factory_service . Мелочь то какая