4

Нужно ли в классах ентити всегда переопределять методы equals() и hashCode() ? И что значит не используйте ссылки на поля в методах equals() и hashCode() если классы задействованны в ORM ?

2 ответа 2

8

Перегружать equals и hashCode в Entity-классах, используемых в ORM нужно по тем же причинам, по которым это нужно в других случаях:

  • корректное сравнение объектов, полученных из разных источников (для ORM - в разных сессиях)
  • корректное хранение объектов в Hash* и *Set коллекциях

Нежелательность использования ссылок на поля в equals и hashCode в классах, используемых ORM, вызвано тем, что большинство ORM возвращают не непосредственно объекты вашего класса, а прокси. Прокси наследуются от ваших классов и перегружают геттеры. Во многих ORM по умолчанию стратегия получения объектов - lazy (ленивая). Это значит, что в момент, когда вы получаете объект, вы получаете пустую прокси. И только когда вы получаете значения полей через геттеры, выполняется запрос к базе, извлекающий значения полей и заполняющий ими объект. Если вы в equals или hashCode будете использовать ссылки на поля, вы можете попасть в ситуацию, когда в сравнении или вычислении хеша участвует ещё не заполненный объект. Поэтому вместо прямого доступа к полям следует использовать геттеры.

Естественно, вышесказанное относится к ORM, умеющим загружать объекты "лениво", например, к Hibernate. А вот ORM вроде MyBatis грузят объект целиком, поэтому там использование ссылок на поля в методах объекта вполне уместно.

4
  • Почему-то мне кажется, что обращения к базе при подсчёте хэша - это в принципе плохая идея.
    – Qwertiy
    29 фев 2016 в 17:11
  • Обращение к базе будет ровно одно - при первом доступе к полям неинициализированного объекта. После этого в полях объекта будут лежать валидные значения, и последующие обращения будут направлены к ним. Впрочем, все ORM, поддерживающие "ленивую" загрузку можно заставить грузить объект явно, в момент запроса. Это спасёт от множества неприятных ошибок вроде "нужно загрузить данные, а сессия уже закрыта", но может отрицательно сказаться на производительности, если объект подтягивает много других объектов по ссылкам OneToMany и ManyToMany.
    – fori1ton
    29 фев 2016 в 17:20
  • Если у нас 100 объектов по 3 поля, то при ленивой загрузке мы 300 раз слазием в базу для подсчёта хешей. Ведь в случае, если мы считаем хеш, то, вероятно, не у одного объекта. Вообще, можно было бы хэшем сделать id. Но если уж у нас туда попали какие-то странные поля, то наверное, стоит обеспечить их неленивую загрузку. Или я где-то неправ?
    – Qwertiy
    29 фев 2016 в 17:26
  • Вы правы. Это стандартная дилемма при работе с ORM - что стоит грузить "лениво", а что - нет. Обычно ORM настраивают так, чтобы свойства самого объекта грузились явно, а связанные с ним объекты/коллекции - лениво. Но это сильно зависит от специфики приложения.
    – fori1ton
    29 фев 2016 в 17:32
3

ORM классы лучше (вот прям очень сильно лучше, если только вам совсем выбора не оставили) не трогать, так как они предназначенны для отображения структуры БД и только, да и нормальные люди их генерируют, если есть возможность, и с каждой генерацией ваши методы вылетать будут. Hash - это "техническая" эквивалентность, как от прокси объекта считать? И самое главное - зачем?

Настоятельно рекомендую ORM и все с ними связанное (репозитории и т.п.) не пускать дальше вашего ДБ слоя/пакета/модуля, а заполнять свои pojo классы (или любые другие модельные классы) на месте, пока транзакция еще бежит и тогда весь бред с прокси и т.п. сразу решается, так сказать. И остальная часть приложения никак не зависит от ORM фреймворка, захотели и заменили провайдера своей модели на xml или messages фреймворк и т.д. и т.п.

А так - да, их имплементация - это часть контракта Жабы, только надо понимать, что hashCode логика сломана! И лучше объекты эти делать имутабельными, т.к. hash коллекции и т.п. не обновят карзину объекта, если его поля поменяются. При сложной иерархии классов смешных эффектов еще больше.

6
  • то есть правильным решением будет делать поля класса-ентити final?
    – calm27
    1 мар 2016 в 16:00
  • я просто после каких либо изменений в классе-ентити удаляю и заново генерирую hashCode() и equals(). .. насколько это правильное решения?
    – calm27
    1 мар 2016 в 16:00
  • Уменьшать mutability полезно и нужно, но не надо забывать, что вы увеличиваете количество созданных объектов и на это тоже уходит время и ресурсы (и собирать их из памяти). Нужно искать баланс в своих решения между идеальным миром теорий и жестокой реальностью. Зачастую хеширование объекта и не нужно, достаточно использовать HashMap<String, Object> и другие подобные техники.
    – Webaib
    1 мар 2016 в 16:26
  • Файнал - это полный запрет на изменение, т.б. это насилие, которое бывает необходимо там, где это нужно. Но если люди понимают, что за каждым изменяемым полем лежит обязаности поддерживать связанные с ним стурктуры целостными - то разницы нет )
    – Webaib
    1 мар 2016 в 16:31
  • кажется, что это очень индивидуальный подход к каждому ентити, вот если с ентити нужно проделывать все операции CRUD тогда, апдейтнут иммутабле объект уже получается, не получится?
    – calm27
    1 мар 2016 в 16:32

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.