kosnvny / absolutely_new_lab3

0 stars 0 forks source link

(E) не ленись #12

Open G0ld3nDr4g0n3301 opened 9 months ago

G0ld3nDr4g0n3301 commented 9 months ago

У тебя все хэшкоды сделаны через Objects.hash(). Это вообще не ошибка(наверное?) но я бы на твоём месте в целях обучения сделал бы свой хэшкод. Который брал бы поля, делал с ними что-то, домножал на простые числа, снова делал и возвращал. Так будет лучше. Потому что в задаче желательно,чтобы ты разобралась в вопросе,и сделала кастомную хэш-функцию с нуля,а не использовала готовую.

k1b24 commented 9 months ago

Открою маленькую тайну, никто не пользуется хешкодами и уж тем более не передопределяет их поведение с помощью математики. Если есть готовое и рабочее решение через Objects.hash(), то зачем изобретать велосипед?

G0ld3nDr4g0n3301 commented 9 months ago

Просто чтобы удостовериться,что ученик понимает,что это, а не просто скопипастил одну строчку во все переопределения. С практической точки зрения — заниматься этим незачем,но когда ты видишь,что человек сам написал хэш-функцию,то ты понимаешь,что он как минимум понимает,что такое хэш и коллизии.  Поэтому в лабе это стоит проверять. Никто не будет запрещать после сдачи третьей лабы, в следующих лабах или на работе, юзать Objects.hash(), но для обучающих целей в третьей лабе это надо.    

Ain Soph Отправлено из Почты Mail.ru    

Понедельник, 8 января 2024, 10:45 +03:00 от Лазеев Сергей @.***>:     Открою маленькую тайну, никто не пользуется хешкодами и уж тем более не передопределяет их поведение с помощью математики. Если есть готовое и рабочее решение через Objects.hash(), то зачем изобретать велосипед? — Reply to this email directly, view it on GitHub , or unsubscribe . You are receiving this because you authored the thread. Message ID: <kosnvny/absolutely_new_lab3/issues/12/1880512302 @ github . com>