Войти Регистрация
Код-ревью

5. Класс Person: восемь замечаний

Средняя

Java · 42 строк

Класс клиента из сервиса онбординга. Объекты кладут в HashSet для дедупликации заявок и передают между потоками. Найдите проблемы.

В работе: Доменная модель клиента: от неё зависят дедупликация заявок, кэши и логи.

Как проверяем Нажмите на строку с ошибкой и коротко напишите, что не так. Баг засчитан, если отмечена хотя бы одна его строка. Зачёт — найдено не меньше 60 % багов с учётом серьёзности (критичный весит 3, обычный 2, мелкий 1), а лишних отметок не больше, чем багов в коде. Пустые строки, скобки и комментарии лишними не считаем. Ответ один на подход: после него показываем разбор.
equals и hashCodeнеизменяемостьдатыперсональные данные

Источник: обзор транскриптов интервью: Сбер, Java — ревью класса Person. Условие пересказано своими словами.

Давали на собеседовании? Ответов пока мало — ваш поможет.
  1. public class Person {
  2. private String name;
  3. private String passport;
  4. private Date birthDate;
  5. private List<String> phones;
  7. public Person(String name, String passport, Date birthDate, List<String> phones) {
  8. this.name = name;
  9. this.passport = passport;
  10. this.birthDate = birthDate;
  11. this.phones = phones;
  12. }
  14. public Date getBirthDate() {
  15. return birthDate;
  16. }
  18. public List<String> getPhones() {
  19. return phones;
  20. }
  22. public int getAge() {
  23. return new Date().getYear() - birthDate.getYear();
  24. }
  26. @Override
  27. public boolean equals(Object o) {
  28. if (this == o) return true;
  29. Person person = (Person) o;
  30. return name == person.name && passport.equals(person.passport);
  31. }
  33. @Override
  34. public int hashCode() {
  35. return Objects.hash(name, birthDate);
  36. }
  38. @Override
  39. public String toString() {
  40. return "Person{name=" + name + ", passport=" + passport + ", phones=" + phones + "}";
  41. }
  42. }