HomeWork1 for Seminar1 #4

Rinat-A-For-GeekBrains · 2023-11-27T19:32:17Z

No description provided.

Liberate520 · 2023-11-30T12:50:06Z

src/ru/gb/family_tree/Person.java

+ }
+
+
+ private final List<Person> children = new ArrayList<>();


Все поля описываются вверху класса, а только потом конструкторы и методы

Liberate520 · 2023-11-30T12:51:42Z

src/ru/gb/family_tree/Person.java

+
+ public void setMother(Person mother) {
+ this.mother = mother;
+ mother.addChild(this);


Это не очень хорошо. Класс Person должен отвечать только за свое состояние. Не должен менять состояния других объектов. Вот класс Дерева вполне может содержать такие операции, потому что его задача это работа со списком людей

Liberate520 · 2023-11-30T12:52:17Z

src/ru/gb/family_tree/Main.java

+ Person person3 = new Person("Petya");
+
+ person2.setFather(person1);
+


А где вариант использования дерева?

Liberate520 · 2023-12-04T20:48:04Z

src/ru/gb/family_tree/writer/FileHandler.java

+ }
+
+ @Override
+ public void load(Serializable serializable) {


Метод загрузки ничего не должен принимать в качестве аргумента, но должен возвращать ранее сохраненный объект, то есть не должен быть void

Liberate520 · 2023-12-04T20:48:34Z

src/ru/gb/family_tree/writer/FileHandler.java

+ FamilyTree familyTree = (FamilyTree) objectInputStream.readObject();
+ System.out.println(familyTree);


Нет смысла выводить в консоль. Надо вернуть древо для дальнейшей работы

Liberate520 · 2023-12-14T21:30:18Z

src/ru/gb/family_tree/FamilyTree.java

+ public void save(Writable writable) {
+ writable.save(this);
+ }
+
+ public void load(Writable writable) {
+ writable.load();
+ }


Класс дерева не должен выполнять действия по сохранению и загрузке. Наоборот над деревом должны выполнять эти действия. Из за этого непонимание того, как реализовать загрузку) Потому что вроде как метод load возвращает объект, но что делать с этим объектом в рамках текущего класса непонятно

Liberate520 · 2023-12-14T21:32:09Z

src/ru/gb/family_tree/FamilyTree.java

+ public void sortByName(Comparator<T> comparator) {
+ familyTree.sort(comparator);
+ }


Компаратор стоит создавать именно в методе сортировки. Иначе нет смысла например во втором методе сортировки. Ведь методы получились идентичны, за исключением названия метода

Liberate520 · 2023-12-14T21:33:18Z

src/ru/gb/family_tree/comparators/PersonComparatorByAge.java

+
+import java.util.Comparator;
+
+public class PersonComparatorByAge implements Comparator<Person> {


Классы должны также использовать обобщения. Для возможности использовать в рамках класса СемейноеДерево

Liberate520 · 2023-12-19T10:35:00Z

src/ru/gb/family_tree/presenter/PresenterService.java

+
+public class PresenterService {
+ private FamilyTree<Person> familyTree = new FamilyTree<>();
+ private Scanner scanner;


Презентер не может работать со сканером. нет у него таких задач. Класс должен только связывать между собой модель и вью

Liberate520 · 2023-12-19T10:35:40Z

src/ru/gb/family_tree/presenter/PresenterService.java

+import java.util.Scanner;
+
+public class PresenterService {
+ private FamilyTree<Person> familyTree = new FamilyTree<>();


Семейное дерево это только часть модели. например есть еще Filehandler. Для объединения работы модели требуется создать сервис.

Liberate520 · 2023-12-19T10:37:03Z

src/ru/gb/family_tree/presenter/PresenterService.java

+ switch (choice) {
+ case 1:
+ UI.print("Введите имя");
+ String name = UI.read();
+ UI.print("Введите пол. Если женский то Ж, если мужской то Ж");
+ String sGender = UI.read();
+ Gender gender;
+ if (sGender.equals("M"))
+ gender = Gender.MALE;
+ else


Содержание логики приложения также не входит в круг обязанностей презентера. Диалог должна осуществлять view составляющая проекта полностью. Пример реализации смотри на семинаре

Liberate520 · 2023-12-23T17:54:09Z

src/ru/gb/family_tree/model/service/Service.java

+ StringBuilder stringBuilder = new StringBuilder();
+ for (Person person : familyTree) {
+ stringBuilder.append(person);
+ }
+ return stringBuilder.toString();


Скорее это будет метод дерева. Его метод toString. Считаю это нарушением первого принципа солид

Liberate520 · 2023-12-23T17:56:10Z

src/ru/gb/family_tree/model/service/Service.java

+ switch (savingType) {
+ case FILE -> {
+ writable = new FileHandler();
+ }


Свич кейс будет нарушать второй принцип солид. При попытке добавить новый вариант сохранения нам придется переписывать данный кусок кода. А если без свич кейса, то будет нарушен пятый принцип солид и при попытке использовать другой вариант сохранения опять же получим необходимость переписывать код. Два варианта решения показаны в 6 семинаре во второй задаче

Liberate520 · 2023-12-23T17:56:57Z

src/ru/gb/family_tree/presenter/SavingType.java

@@ -0,0 +1,6 @@
+package ru.gb.family_tree.presenter;
+
+public enum SavingType {


Странно, что этот enum лежит в пакете презентера. логичней было бы в пакете writable

Liberate520 · 2023-12-23T17:58:03Z

src/ru/gb/family_tree/view/ConsoleUI.java

+ private void printMenu() {
+ N = 0;
+ System.out.printf("%d. Добавить члена семьи \n", ++N);
+ System.out.printf("%d. Вывести информацию о семейном дереве\n", ++N);
+ System.out.printf("%d. Сортировать членов семейного дерева по имени\n", ++N);
+ System.out.printf("%d. Сортировать членов семейного дерева по возрасту\n", ++N);
+ System.out.printf("%d. Сохранить в файл]\n", ++N);
+ System.out.printf("%d. Загрузить из файла\n", ++N);
+ System.out.printf("%d. Закончить работу\n", ++N);


нарушает второй принцип солид

Liberate520 · 2023-12-23T17:58:35Z

src/ru/gb/family_tree/view/MainMenu.java

+ switch (choice) {
+ case 1:
+ UI.print("Введите имя");


также нарушает второй принцип солид

Liberate520 · 2023-12-23T17:59:19Z

src/ru/gb/family_tree/view/MainMenu.java

+ gender = Gender.MALE;
+ else
+ gender = Gender.FEMALE;
+ Person person = new Person(name, gender);


View часть проекта не должна создавать Person это нарушение первого принципа солид. Person является частью модели и должен создаваться в модели

В package Service попытался создать классы по паттерну Builder. Пока нигде не задействованы.

доработан метод загрузки и выгрузки файла

доработан метод загрузки и выгрузки файла немного изменено MAinMenu

Liberate520 · 2023-12-27T00:46:40Z

src/ru/gb/family_tree/model/service/Service.java

+ Gender gender;
+ if (sGender.equals("M"))
+ gender = Gender.MALE;
+ else
+ gender = Gender.FEMALE;


пол можно было определять на стороне view

Liberate520 · 2023-12-27T00:47:39Z

src/ru/gb/family_tree/model/service/Service.java

+ void Personmake() {
+ PersonBuilder personBuilder = new PersonBuilder();
+ Person person = personBuilder
+ .create()
+ .buildName("Petya")
+ .buildDateOfBirth(LocalDate.now())
+ .buildDateOfDeath(LocalDate.now())
+ .build();
+ }


Так а смысл в методе, если он не возвращает результат?

Ну и странно, что дата рождения и смерти одинаковые)

Liberate520 · 2023-12-27T00:48:45Z

src/ru/gb/family_tree/presenter/Presenter.java

+import ru.gb.family_tree.view.View;
+
+public class Presenter {
+ private View UI;


Капслоком обозначают обычно константы. В данном случае это не константа)

Liberate520 · 2023-12-27T00:51:57Z

src/ru/gb/family_tree/model/service/Service.java

+ familyTree.sortByAge();
+ }
+
+ public void addParent(Person person, Person child) {


Не увидел соответствующей команды в консоли. Лучше было бы передавать не Person, а например имя человека (считаем, что все имена уникальны). По этому имени можно найти объект Person и уже потом между объектами создать связь

HomeWork1 for Seminar1

d04769e

Liberate520 reviewed Nov 30, 2023

View reviewed changes

HomeWork2 for Seminar2

241d261

Liberate520 reviewed Dec 4, 2023

View reviewed changes

Rinat-A-For-GeekBrains added 2 commits December 12, 2023 23:35

HomeWork3 for Seminar2

4081c7c

HomeWork 4 for Seminar2 with question

2042d56

Liberate520 reviewed Dec 14, 2023

View reviewed changes

HomeWork 5

0d6c625

Liberate520 reviewed Dec 19, 2023

View reviewed changes

HomeWork 6

31bfafa

Liberate520 reviewed Dec 23, 2023

View reviewed changes

Rinat-A-For-GeekBrains added 3 commits December 26, 2023 21:44

HomeWork 7

2e438bd

В package Service попытался создать классы по паттерну Builder. Пока нигде не задействованы.

HomeWork 7

f2b3a0e

доработан метод загрузки и выгрузки файла

HomeWork 7

5b32eba

доработан метод загрузки и выгрузки файла немного изменено MAinMenu

Liberate520 reviewed Dec 27, 2023

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

HomeWork1 for Seminar1 #4

HomeWork1 for Seminar1 #4

Rinat-A-For-GeekBrains commented Nov 27, 2023

Liberate520 Nov 30, 2023

Liberate520 Nov 30, 2023

Liberate520 Nov 30, 2023

Liberate520 Dec 4, 2023

Liberate520 Dec 4, 2023

Liberate520 Dec 14, 2023

Liberate520 Dec 14, 2023

Liberate520 Dec 14, 2023

Liberate520 Dec 19, 2023

Liberate520 Dec 19, 2023

Liberate520 Dec 19, 2023

Liberate520 Dec 23, 2023

Liberate520 Dec 23, 2023

Liberate520 Dec 23, 2023

Liberate520 Dec 23, 2023

Liberate520 Dec 23, 2023

Liberate520 Dec 23, 2023

Liberate520 Dec 27, 2023

Liberate520 Dec 27, 2023

Liberate520 Dec 27, 2023

Liberate520 Dec 27, 2023

Liberate520 Dec 27, 2023

		Person person3 = new Person("Petya");

		person2.setFather(person1);

		FamilyTree familyTree = (FamilyTree) objectInputStream.readObject();
		System.out.println(familyTree);


		import java.util.Comparator;

		public class PersonComparatorByAge implements Comparator<Person> {

		@@ -0,0 +1,6 @@
		package ru.gb.family_tree.presenter;

		public enum SavingType {

HomeWork1 for Seminar1 #4

Are you sure you want to change the base?

HomeWork1 for Seminar1 #4

Conversation

Rinat-A-For-GeekBrains commented Nov 27, 2023

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment