add DZ1 #11
add DZ1 #11
Conversation
src/family_tree/human/Human.java
Outdated
| public String mother; | ||
| public String father; |
There was a problem hiding this comment.
Почему дети Human, а родители только String? что за странная семья?) родители тоже должны быть Human
| } | ||
|
|
||
| public void sortName() { | ||
| humanList.sort((Comparator<? super E>) new HumanCompName()); |
There was a problem hiding this comment.
Приведение типов не нужно. Нужно переписать компараторы с учетом того, что не всегда людей придется сравнивать и научить их сравнивать объекты типа Е
|
|
||
| @Override | ||
| public Iterator<E> iterator() { | ||
| return (Iterator<E>) new FamilyTreeIter(humanList); |
There was a problem hiding this comment.
Тоже самое. Приведение типов ошибка. Будет работать только если использовать Human, а для Dog например приложение упадет
| public class Presenter { | ||
| private View view; | ||
|
|
||
| public Presenter(View view) { | ||
| this.view = view; | ||
| } | ||
| } No newline at end of file |
There was a problem hiding this comment.
И все?)) Такой презентер не создаст связи между view и моделью
src/family_tree/view/ConsoleUI.java
Outdated
| int age = Integer.parseInt(scanner.nextLine()); | ||
| } | ||
|
|
||
| private static void printMenu() { |
|
|
||
| public class FamilyTree<E extends TreeN<E>> implements Serializable, Iterable<E> { | ||
| private final List<E> humanList; | ||
| private Writer writer; |
There was a problem hiding this comment.
Нарушение первого принципа солид
| } | ||
|
|
||
| public void add(String name, int age){ | ||
| ; |
| group = new FamilyTree<>(); | ||
| } | ||
|
|
||
| public void add(String name, int age){ |
There was a problem hiding this comment.
Очень мало данных для создания человека. Мы там намного больше полей выделяли. А вместо возраста использовали дату рождения и смерти
| public String getHumanInfo() { | ||
| StringBuilder stringBuilder = new StringBuilder(); | ||
| stringBuilder.append("Список Членов семьи:\n"); | ||
|
|
||
| // Iterator<Student> iterator = group.iterator(); | ||
| // while (iterator.hasNext()){ | ||
| // Student student = iterator.next(); | ||
| // stringBuilder.append(student); | ||
| // stringBuilder.append("\n"); | ||
| // } | ||
| for (Human human: group){ | ||
| stringBuilder.append(human); | ||
| stringBuilder.append("\n"); | ||
| } | ||
| return stringBuilder.toString(); | ||
| } |
There was a problem hiding this comment.
Скорее это работа семейного дерева, а точнее его метод toString. А здесь только вызов этого метода у дерева. Нарушение первого принципа солид
| } | ||
| return stringBuilder.toString(); | ||
| } | ||
|
|
There was a problem hiding this comment.
Не хватает методов сохранения и загрузки
| public void addHuman(String name, int age) { | ||
| service.add(name, age); | ||
| //какой-то ответ | ||
| getHumansListInfo(); | ||
| } |
There was a problem hiding this comment.
Нет возможности создания родственных связей, а это основная идея нашего приложения
2 participants
No description provided.