Java family tree #20
Java family tree #20
Conversation
src/ru/gb/family_tree/Human.java
Outdated
|
|
||
| public class Human { | ||
| private long id; | ||
| private String _name; |
There was a problem hiding this comment.
В джава не принято называть переменные с символа земли, но с точки зрения синтаксиса возможно
There was a problem hiding this comment.
На сколько я знаю, есть конвенция (по крайней мере в C#) на именование полей объектов через символ нижнего подчеркивания, как по мне оно упрощает взаимодействие с полями и их читаемость.
src/ru/gb/family_tree/Human.java
Outdated
| public String FamilyTree(){ | ||
| return FamilyTree(0); | ||
| } | ||
| public String FamilyTree(int tabsCount){ |
There was a problem hiding this comment.
Очень странно для класса Человек наличие методов связанных с деревом. То есть этот метод добавляет различные отступы, которые важны не для работы текущего класса. Следовательно данная работа должна быть описана в классе дерева. Также названия методов пишутся с маленькой буквы
|
Added serializer class |
| import java.util.ArrayList; | ||
|
|
||
| public class FamilyTree implements Serializable { | ||
| private ArrayList<Human> _tree; |
There was a problem hiding this comment.
такой нейминг ничего не дает в джаве, но снижает читаемость
| import java.io.ObjectOutputStream; | ||
|
|
||
| public class FamilyTreeSerializer { | ||
| public void Write(String path, FamilyTree obj){ |
There was a problem hiding this comment.
название методов с маленькой буквы
There was a problem hiding this comment.
Хорошо, учту в следующем проекте.
|
Leave for review "Basic interfaces" execise |
| sb.append(human.toString() + "\n"); | ||
| return sb.toString(); | ||
| } | ||
| public void WriteTo(String path){ |
There was a problem hiding this comment.
Название методов с маленькой буквы
|
"Genericks" exercise |
| import human.Human; | ||
| import human.comparators.HumanNameComparator; | ||
|
|
||
| public class FamilyTree implements Serializable, Iterable<Human> { |
There was a problem hiding this comment.
По заданию класс семейного дерева должен был использовать параметр (быть обобщенным), но это обычный класс. Вижу, что есть класс FamilyTreeItem, который использует параметр, но этот класс тоже никак не фигурирует в описании класса семейное дерево
|
Pushed final changes |
|
"From simple to practice" |
|
"OOP Desing and SOLID" (part1 ) |
| var human = humanBuilder.build(name); | ||
| tree.add(human); | ||
| } | ||
| public void addHuman(String name, Human mother, Human father){ |
There was a problem hiding this comment.
Надо передавать не Human родителей, а только их id и по id в данном методе найти уже Human в дереве
There was a problem hiding this comment.
Подправил, но не полностью. В будущем буду пристальнее за этим следить.
| import java.util.Scanner; | ||
|
|
||
| import model.human.Gender; | ||
| import model.human.Human; |
| service.addHuman(name, mother, father); | ||
| getFamilyTree(); | ||
| } | ||
|
|
There was a problem hiding this comment.
добавить методы сохранения и загрузки
There was a problem hiding this comment.
Они реализованы в классе FamilyTreeSerializer, я подумал что такое дублирование будет лишним и удалил вызывающие методы
There was a problem hiding this comment.
Они то реализованы, но как то этот функционал не прикручен к основному проекту
There was a problem hiding this comment.
Оно было в этом коммите 51000ce , но впоследствии я их удалил. Теперь буду знать, что не стоит так делать.
Added famiy-tree project