Д/З к семинару 1 #7
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Реализовал для Human конструкторы, геттеры, сеттеры, и переопределил toString; - Реализовал для Human конструкторы и переопределил toString;
- Реализовал для Human конструкторы, геттеры, сеттеры, и переопределил toString; - Реализовал для Human конструкторы и переопределил toString;
…сти от пола при меняются соответсвующие значения и в самих вызовах. - Создал 6 экземпляров класса типа Human.
- Добавил метод поиска по имени; - Исправил ошибки.
Liberate520
reviewed
Mar 31, 2024
Comment on lines
47
to
64
| public Human(String name, LocalDate dob, Gender gender, List<Human> children, Human mother, Human father) { | ||
| this.name = name; | ||
| this.dob = dob; | ||
| this.gender = gender; | ||
| this.children = children; | ||
| this.mother = mother; | ||
| this.father = father; | ||
| } | ||
|
|
||
| public Human(String name, LocalDate dob, Gender gender) { | ||
| this.name = name; | ||
| this.dob = dob; | ||
| this.gender = gender; | ||
| Human temp1 = new Human(); | ||
| Human temp2 = new Human(); | ||
| this.mother = temp1; | ||
| this.father = temp2; | ||
| } |
There was a problem hiding this comment.
И не нудо в 60 строчке создавать человека. Пусть null остается
Comment on lines
70
to
72
| public Human(){ | ||
| this(null); | ||
| } |
There was a problem hiding this comment.
Максимально странная запись)) Почему вы решили имя сделать null, а все остальное не делать null? По факту и так все поля изначально имеют значение null
| ", dod=" + dod + | ||
| ", gender=" + gender + | ||
| ", children=" + getChildrenNames() + | ||
| ", mother='" + this.getMother().getName() + |
There was a problem hiding this comment.
А) вот для чего вы хотели, чтобы родители были хотя бы пустым Human объектом) можно делать проверку на null
src/ru/gb/family_tree/tree/Tree.java
Outdated
Comment on lines
38
to
46
| private String getLikeTable (List<Human> familyList){ | ||
|
|
||
| return null; | ||
| } | ||
|
|
||
| private String getLikeTree (List<Human> familyList) { | ||
|
|
||
| return null; | ||
| } |
+ gitignor изменён; - куча ошибок.
Всё сделано как в семинаре, но без статичного метода тестового дерева. Данные для теста, загружаются из файла "tree.sav".
Всё сделано как в семинаре, но без статичного метода тестового дерева. Данные для теста, загружаются из файла "tree.sav".
- добавил компараторы для возраста и имени.
# Conflicts: # src/ru/gb/family_tree/Main.java # src/ru/gb/family_tree/human/Human.java # src/ru/gb/family_tree/writer/tree.sav
- добавил общую структуру MVP.
+ Реализовал почти весь полезный функционал семейного древа на основе MVP; + Есть возможность сохранения и загрузки древа; - Не прописана, почти нигде "Защита от дурака" и другие исключения; - Нет возможности удаления человека из древа;
+ Реализовал почти весь полезный функционал семейного древа на основе MVP; + Есть возможность сохранения и загрузки древа; + Исправил синтаксические ошибки. - Не прописана, почти нигде "Защита от дурака" и другие исключения; - Нет возможности удаления человека из древа; - Нет реализована возможность контроля за несколькими древами.
Заприватил поля.
- Исправил замечания к прошлому PR.
- Исправил замечания к прошлому PR.
+ Попробовал реализовать принципы SOLID в проекте. Кажется получились, если не брать в счёт исключения.
Liberate520
reviewed
Apr 21, 2024
| } else if (viewGender == ViewGender.FEMALE) { | ||
| gender = Gender.FEMALE; | ||
| } | ||
| service.addHuman(name, LocalDate.of(year, month, day), gender); |
There was a problem hiding this comment.
странно, что именно презентер занимается созданием даты. Логичней было бы в модели или во view
There was a problem hiding this comment.
Тоже самое касается пола. Либо это вопрос корректного ввода и тогда это view, либо это работа с информацией и тогда это модель
| public class Service { | ||
| private int genId; | ||
| private Tree<Human> tree; | ||
| private final FileHandler fH; |
There was a problem hiding this comment.
Нарушает пятый принцип солид. Должна быть зависимость от интерфейса
- Исправлен класс Presenter по замечанию.
- Исправлен класс Presenter по замечанию. - Реализовал принцип "D" в модели.
- Переименовал некоторые методы по назначению.
Надеюсь всё верно.
Liberate520
reviewed
Apr 23, 2024
|
|
||
| public HumanFamilyTree() { | ||
| tree = new Tree<>(); | ||
| fH = new FileHandler(); |
There was a problem hiding this comment.
Надо было передать через аргументы конструктора Writable или через сеттер и тогда упоминаний FileHandler не было бы
| if ((findByID(parentID) != null) && findByID(childID) != null){ | ||
| findByID(parentID).addChild(findByID(childID)); | ||
| } else { | ||
| System.out.println("Данные ID не существуют."); |
There was a problem hiding this comment.
Вывод текста пользователю должна осуществлять view
- Исправил крайние замечания.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.