dz гб #2
dz гб #2
Conversation
| import java.util.List; | ||
|
|
||
| public class Human { | ||
| private int id = 1; |
There was a problem hiding this comment.
инициализацию лучше делать в конструкторе класса
| private int age; | ||
| private String name; | ||
| private String surname; | ||
| private MotFath MotFath; |
There was a problem hiding this comment.
имена переменных с маленькой буквы. И это видимо родители? Так и назовите Parents
There was a problem hiding this comment.
Создать поля типа Human для ссылки на родителей
| @Override | ||
| public String toString() { | ||
| return ", name: " + name + ", surname: " + surname + ", age: " + age + ", gender: " + gender | ||
| + ", status: " + MotFath + ", children: " + children; | ||
| } |
| public class Human { | ||
| private int id = 1; | ||
| private Gender gender; | ||
| private int age; |
There was a problem hiding this comment.
говорил на семинаре, что это неверно. Возраст постоянно меняется. Правильней указать дату рождения и смерти, а возраст высчитывать в методе
| public enum MotFath { | ||
| Father, Mother, Child; | ||
| } |
There was a problem hiding this comment.
А... это не только родители родители? Тогда надо написать, что это роль в семье или вроде того, но честно не знаю как вы будете это использовать, сейчас глянем
There was a problem hiding this comment.
Да, посмотрел. Это ошибка. Данные понятия относительны. Для кого то человек является матерью, а для кого то тот же самый человек является внучкой, а объект один и вы не можете присвоить ему сразу несколько значений enum
| private String name; | ||
| private String surname; | ||
| private MotFath MotFath; | ||
| private List<String> children; |
There was a problem hiding this comment.
Список детей должен быть не String содержать, а Human
| StringBuilder stringBuilder = new StringBuilder(); | ||
| stringBuilder.append("Список человек в генеалогическом древе: \n"); | ||
| for (Human Human : humanList){ | ||
| stringBuilder.append("id " + id++); |
There was a problem hiding this comment.
Этого тоже не понял. id это же значение дерева? Почему оно вдруг увеличивается? и зачем каждый раз писать перед выдачей информации по людям. И это значение не обнуляется, значит если несколько раз попросить выписку по семье, то id будет меняться у тех же записей, а это уж совсем неправильно
2 participants
Александр Шугалей