Создана база семейного древа. #16
Conversation
src/ru/gb/family_tree/Human.java
Outdated
| } | ||
|
|
||
| public Human(String name, Gender gender) { | ||
| this(name, gender, LocalDate.of(1900,12,12)); |
There was a problem hiding this comment.
Почему вдруг именно такая дата? Ошибка писать такие даты, потом непонятно откуда берутся артефакты)
src/ru/gb/family_tree/Human.java
Outdated
| private List<Human> children; | ||
| private Human father, mather; |
There was a problem hiding this comment.
Ни в какой момент не изменяется состояние этих полей
| return false; | ||
| } | ||
|
|
||
| public boolean remuve(long humansId){ |
| getHumanInfo(); | ||
| } | ||
|
|
||
| public void addChildren(int nameParent, int nameChild) { |
There was a problem hiding this comment.
странно, что переменная называется nameParent и при этом имеет тип int
There was a problem hiding this comment.
Хотел по имени добавлять родителей, потом вспомнил, имена могут повторяться.
| import java.util.List; | ||
|
|
||
| public class Service { | ||
| private FamilyTree familyTree; |
There was a problem hiding this comment.
угловые скобочки и тип Human
|
|
||
| public class Service { | ||
| private FamilyTree familyTree; | ||
| private String filePath = "src/ru/gb/family_tree/model/save/tree.txt"; |
There was a problem hiding this comment.
не имеет отношения к работе Сервис. Перенести в класс, где используется
| public void addChildren(int nameParent, int nameChild){ | ||
| familyTree.getById(nameParent).addChild(familyTree.getById(nameChild)); | ||
| } |
There was a problem hiding this comment.
А ребенку добавить связь с родителем?
| public String getHumanInfo(){ | ||
| String answer = familyTree.getInfo(); | ||
| return answer; | ||
| } |
There was a problem hiding this comment.
Опять же название метода не сочетается с телом метода
| private static FamilyTree load(String filePath){ | ||
| Writable writable = new FileHandler(); | ||
| return (FamilyTree) writable.read(filePath); | ||
| } | ||
|
|
||
| private static void save(FamilyTree familyTree, String filePath){ | ||
| Writable writable = new FileHandler(); | ||
| writable.save(familyTree, filePath); | ||
| } |
There was a problem hiding this comment.
Убрать статику. Сделать обычным методом. Writable сделать полем. Убрать нарушение 5 принципа. Зависимость должна быть от абстракции
…, исправлен метод записи и считывания данных
2 participants
No description provided.