Сортировка сущностей в Doctrine 2?

Question

[TrueDrago]

Для последнего проекта я создал сущность, которая должна поддерживать сортировку. Добавил для неё по инструкции Sortable behavior, но появилась следующая проблема. Помимо геттеров и сеттеров позиции мне нужны следующие методы:

<?php<br>
public function moveUp() {<br>
    ++$this->position;<br>
}<br><br>
public function moveDown() {<br>
    if($this->position != 0)<br>
        --$this->position;<br>
}<br>

С сдвигом вниз в принципе всё понятно — ниже нуля оно не будет двигаться. Но как быть с максимальными позициями? Насколько я понял, делать напрямую из сущности запросы к бд, чтобы определить максимальное значение — не лучшее архитектурное решение. Как тогда запретить инкрементировать элемент с максимальной позицией?

Answer 1

[JekaRu]

Если мне память не изменяет, там кажется не имеет особого смысла это делать, все автоматом пересчитывается при сохранении.
Вам точно надо ограничивать максимальное значение?

Comments

[TrueDrago]

Ну смотрите, у меня есть 2 сущности:
(сущность) — (порядок)
item1 — 0
item2 — 1

Если сделать $item2->moveUp(), то тогда всё будет выглядеть не правильно:
item1 — 0
item2 — 2

[JekaRu]

Имхо move{Up,Down} надо не в самой модели, а в сервисном слое делать.
Например, в менеджере модели или репозитории.

[TrueDrago]

Хорошая идея! Не подскажете где толково написано про менеджеры моделей?

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

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

[JekaRu]

Вот тут я недавно отвечал на вопрос по сервисам
habrahabr.ru/qa/33117/#answer_129891
Там примеры есть из известных проектов.

Answer 2

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

Ну мне кажется не слишком удобная реализация. Ибо…
a — 1
b — 2
c — 3

если сделать c.moveUp()

a — 1
b — 2
c — 2

И уже сложно будет пересчитать, вы так не считаете?

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

Comments

[TrueDrago]

Дело в том, что перерасчитывает оно автоматически, если назначить уже существовавшую ранее позицию. Мне нужно чтобы итак уже максимальное значение не инкрементировалось