add basic family tree #1
Conversation
| public FamilyTree() { | ||
| this.treeId = new Random().nextInt(10000); | ||
| this.humanList = new ArrayList<>(); | ||
| System.out.println("Зарегистрировано семейно древо: " + this); |
There was a problem hiding this comment.
В теле конструктора, цель которого создать объект класса, вы выводите информацию о всем объекте? То есть конструктор еще не отработал, но объект уже в печати? Это не логично
| private String name; | ||
| private String surname; | ||
| private Gender gender; | ||
| private Human spouse; |
There was a problem hiding this comment.
Просто улыбки ради пишу, не придирка) странно, что для родителей вы создали список, имея в виду, что неизвестно сколько их будет, но вот для супругов... а как же культуры, где разрешены гаремы?)
|
add FileHandler |
| import java.io.*; | ||
|
|
||
| public class FileHandler implements Writable { | ||
| String path; |
| if ((getHuman(getHuman(id).getSpouse().getId()).getSpouse() != null) | ||
| && ((getHuman(getHuman(id).getSpouse().getId()).getSpouse().getId() == id))) { |
There was a problem hiding this comment.
Код плохо читаем и непонятен. Стоит как то упростить или хотя бы разбить на приватные методы
There was a problem hiding this comment.
или как то использовать промежуточные действия, например получение объекта супруга в переменную, а уже потом использовать эту переменную в коде
| } | ||
|
|
||
| public List<Human> getHumanWithSortByName() { | ||
| return humanList.stream().sorted(new HumanComparatorByName()).collect(Collectors.toList()); |
There was a problem hiding this comment.
Зачем так сложно то? Разве запись humanList.sort(new HumanComparatorByName()) не проще?
There was a problem hiding this comment.
хотел возвращать список, если делаю через humanList.sort, то он не возвращает значение
|
|
||
| public void removeSpouse(long id) { | ||
| if (getElement(id).getSpouse() != null) { | ||
| Human spouse = (Human) getElement(id).getSpouse(); |
There was a problem hiding this comment.
После обобщения не должно быть упоминания класса Human, код должен быть описан с использованием параметра
|
|
||
| public class TreeService<E extends Fundamental<E>> implements BaseFunctions<E>, SortFunctions<E> { | ||
|
|
||
| FamilyTree<E> familyTree; |
| @@ -0,0 +1,144 @@ | |||
| package ru.gb.family_tree.service.tree_service; | |||
There was a problem hiding this comment.
Сервис это не отдельная сущность. Это тоже часть модели и соответственно должен лежать в пакете модели
|
|
||
| public class BaseTerminal<E extends TreeService> implements Vew { | ||
|
|
||
| private CommandsService<Item> commandsService; |
There was a problem hiding this comment.
Паттерн mvp предполагает, что view никак не зависит от модели, но в вашем есть четкая зависимость. Вам не хватает презентера
|
|
||
| import java.util.Scanner; | ||
|
|
||
| public class BaseTerminal<E extends TreeService> implements Vew { |
| private CommandsService<Item> commandsService; | ||
| private Scanner scanner; | ||
|
|
||
| public BaseTerminal(Writable writable) { |
There was a problem hiding this comment.
writable также часть модели и view никак не должно от него зависеть
| if (command == 1) { | ||
| commandsService.doCommand("Активировать новое дерево"); | ||
| } else if (command == 2) { | ||
| commandsService.doCommand("Добавить элемент дерева"); | ||
| } else if (command == 3) { | ||
| commandsService.doCommand("Показать все элементы дерева"); | ||
| } else if (command == 4) { | ||
| commandsService.doCommand("Сохранить изменения"); | ||
| } else if (command == 5) { | ||
| commandsService.doCommand("Загрузить последние сохраненные изменения"); | ||
| } else if (command == 0) { | ||
| System.out.println("Выход из меню."); | ||
| break; | ||
| } else { | ||
| System.out.println("Некорректная команда, попробуйте ещё раз."); | ||
| } |
| private void menu() { | ||
| System.out.println("--------------------------------------------"); | ||
| System.out.println("Введите команду:"); | ||
| System.out.println("1 - Активировать новое дерево"); | ||
| System.out.println("2 - Добавить элемент дерева"); | ||
| System.out.println("3 - Показать все элементы дерева"); | ||
| System.out.println("4 - Сохранить изменения"); | ||
| System.out.println("5 - Загрузить последние сохраненные изменения"); | ||
| System.out.println("0 - Выход"); | ||
| System.out.println("--------------------------------------------"); | ||
| } |
There was a problem hiding this comment.
И это нарушение второго принципа солид
| import java.util.List; | ||
| import java.util.stream.Collectors; | ||
|
|
||
| public class TreeService<E extends Fundamental<E>> implements BaseFunctions<E>, SortFunctions<E> { |
There was a problem hiding this comment.
Много раз говорил, что не надо делать класс сервиса обобщенным...
| e.setId(id); | ||
| familyTree.setElementId(id); | ||
| familyTree.getElementList().add(e); | ||
| System.out.println("В дерево добавлен: " + e); |
There was a problem hiding this comment.
Работа с консолью должна выполняться View составляющей проекта
There was a problem hiding this comment.
нарушение первого принципа солид
| @Override | ||
| public List<E> SortByName() { | ||
| return familyTree.getElementList().stream().sorted(new ElementsComparatorByName<>()).collect(Collectors.toList()); | ||
| } | ||
|
|
||
| @Override | ||
| public List<E> SortByBirthday() { | ||
| return familyTree.getElementList().stream().sorted(new ElementsComparatorByBirthday<>()).collect(Collectors.toList()); | ||
| } | ||
|
|
||
| @Override | ||
| public List<E> SortByChildren() { | ||
| return familyTree.getElementList().stream().sorted(new ElementsComparatorByChildren<>()).collect(Collectors.toList()); | ||
| } |
There was a problem hiding this comment.
Методы и их описание должны быть в классе дерева, а отсюда только вызываться
There was a problem hiding this comment.
Сейчас это нарушение первого принципа солид
| import java.time.format.DateTimeFormatter; | ||
| import java.util.*; | ||
|
|
||
| public class Controller<E extends Fundamental<E>> implements Functions { |
There was a problem hiding this comment.
Презентер и контроллер используются в разных паттернах. В задании ожидалась реализация именно MVP паттерна
|
|
||
| public class Controller<E extends Fundamental<E>> implements Functions { | ||
| private TreeService<E> treeService; | ||
| private Scanner scanner; |
There was a problem hiding this comment.
Сканнер также не должен быть даже в контроллере. Сканнер используется для работы с консолью, а работа с консолью должна выполняться во View составляющей проекта. В данном случае нарушен первый принцип солид
| private TreeService<E> treeService; | ||
| private Scanner scanner; | ||
| private DateTimeFormatter formatter; | ||
| private Writable writable; |
There was a problem hiding this comment.
Writable это часть модели, а не презентера и не контроллера
There was a problem hiding this comment.
Именно сервис должен взаимодействовать с объектом Writable
| switch (commands) { | ||
| case NEW_TREE: | ||
| newTree(); | ||
| break; | ||
| case ADD_ITEM: | ||
| addItem(); | ||
| break; | ||
| case REMOVE_ITEM: | ||
| removeItem(); | ||
| break; | ||
| case ADD_PARENTS: | ||
| addParents(); | ||
| break; | ||
| case GET_PARENTS: | ||
| getParents(); | ||
| break; |
There was a problem hiding this comment.
свич кейс нарушает второй принцип солид
| } else if (i == 2) { | ||
| gender = Gender.Female; | ||
| } | ||
| Item item = new Item(name, surname, birthday, gender); |
There was a problem hiding this comment.
Создание объектов модели это работа модели, а не контроллера. Нарушен первый принцип
| } | ||
|
|
||
| private void getItem() { | ||
| System.out.println("Введите ID элемента дерева, информацию по которому хотите получить: "); |
There was a problem hiding this comment.
Работа с консолью это работа View
| private List<String> createCommands() { | ||
| List<String> list = new ArrayList<>(); | ||
| list.addAll(this.controller.getCommands().values()); | ||
| return list; | ||
| } |
There was a problem hiding this comment.
То есть команды хранятся в контроллере, но пользоваться ими будет View? В этом есть смысл? Команды это по сути строчки либо кнопочки, которые находятся на UI и являются частью View
created packages and base classes