Всем привет, подскажите, является ли плохим тоном возвращать напрямую объект

коллекции из сущности
```
public function getCustomCollection(): Collection
{
return $this->customCollection;
}
```
Если да, то почему? Это чисто религиозная тема, или есть конкретные уязвимости связанные с этим?

Да, могут сайт взломать

есть еще холивар, что геттеро-сеттеры приводят к разрушению инкапсуляции =)

да, скорее про это, что мы работаем с "запчастью" сущности вне ее контекста

мы работаем не с сущностью а просто с структурой данных. это не хорошо и не плохо, это совершенно другой подход

Это может привести к неявному изменению сущности

Вы получаете объект Collection, который потом передаёте в какой-нибудь сервис, а там делаете add или remove. При flush эти изменения улетят в базу.

Поэтому не надо так делать 🙈 Нужно предусмотреть все возможные функции изменения в самой сущности, а не менять её в другом сервисе.

Это не холивар, это реальная проблема

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

Да, я с тобой согласен полностью, проще запретить изначально, а то потом тратишь время на поиски проблемы

Вот, теперь ты уже понимаешь нашу боль 🥲

закон деметры, information expert