HW2.1 #14
HW2.1 #14
Conversation
src/Main.java
Outdated
| @@ -0,0 +1,2 @@ | |||
| package PACKAGE_NAME;public class Main { | |||
src/family_tree/FamilyTree.java
Outdated
| import java.util.Iterator; | ||
| import java.util.List; | ||
|
|
||
| public class FamilyTree implements Serializable { |
There was a problem hiding this comment.
Здесь по заданию ожидался интерфейс Iterable
| import view.FamilyTreeView; | ||
|
|
||
| public class FamilyTreePresenter { | ||
| private final FamilyTree<Human> model; |
There was a problem hiding this comment.
Требуется создать класс Сервис для группировки работы модели. Модель это не только работа с деревом. Например в нашем проекте еще есть работа с файловой системой, с помощью которой мы делаем сохранения и загрузку
| do { | ||
| view.displayFamilyTree(model.getInfo()); |
There was a problem hiding this comment.
Цикл перенести во view. Вы путаете с mvc. Например, если изменится view, то придется и презентер переписывать, что конечно же неправильно. Данный код не проходит по 1 и 2 принципу солид
src/presenter/prompt_user_input.java
Outdated
| package presenter; | ||
| import java.util.Scanner; | ||
|
|
||
| public class prompt_user_input { |
There was a problem hiding this comment.
Название классов всегда с большой буквы
src/presenter/prompt_user_input.java
Outdated
| import java.util.Scanner; | ||
|
|
||
| public class prompt_user_input { | ||
| private Scanner scanner; |
There was a problem hiding this comment.
Общение с пользователем это работа view части проекта, а не презентера, а я вижу, что данный класс находится в пакете презентера...
src/view/FamilyTreeView.java
Outdated
|
|
||
| import java.util.Scanner; | ||
|
|
||
| public class FamilyTreeView { |
There was a problem hiding this comment.
Желательно все таки создать интерфейс будет к следующему дз. В соответствии с 5 принципом солид
|
|
||
| public class Service { | ||
| public FamilyTree<Human> familyTree; | ||
| public FileHandler fileHandler; |
There was a problem hiding this comment.
нарушает пятый принцип солид. Зависимость должна быть через интерфейс.
| public String printFamilyTree() { | ||
| StringBuilder strBuilder = new StringBuilder(); | ||
| strBuilder.append("Family tree:"); | ||
| strBuilder.append("\n"); | ||
| for (Human human : familyTree) { | ||
| strBuilder.append(human); | ||
| strBuilder.append("\n"); | ||
| return strBuilder.toString(); | ||
| } | ||
| return null; |
There was a problem hiding this comment.
Скорее это работа класса Дерево, а точней его метода toString
2 participants
No description provided.