TreeSet #11
TreeSet #11
Conversation
| @@ -0,0 +1,2 @@ | |||
| #Tue Nov 14 16:51:37 MSK 2017 | |||
There was a problem hiding this comment.
папку .gradle не нужно выкладывать, я вроде бы про это писал уже
-0.5
|
|
||
| import static org.junit.Assert.*; | ||
|
|
||
| public class MindTreeSetTest { |
There was a problem hiding this comment.
тесты всё же надо дописать
-1
| */ | ||
| public class MindTreeSet<E> implements MyTreeSet<E>{ | ||
|
|
||
| Node start = null; |
There was a problem hiding this comment.
мне кажется, их лучше сделать закрытыми
| Node finish = null; | ||
| Node root = null; | ||
| int size = 0; | ||
| boolean ret = false; |
There was a problem hiding this comment.
не самое удачное имя для поля, оно мало что говорит о своём назначении
| * Class with realising interface MyTreeSet | ||
| * @param <E> type of elements | ||
| */ | ||
| public class MindTreeSet<E> implements MyTreeSet<E>{ |
There was a problem hiding this comment.
тут бы не помешали какие-нибудь подходящие wilcard'ы генерикам
| * @return result of compare | ||
| */ | ||
| private int compare(E e1, E e2) { | ||
| if (comparator == null) { |
There was a problem hiding this comment.
давайте в конструкторе проинициализируем компаратор, чтобы он использовал compareTo(). тогда не нужно будет постоянно эти проверки делать
| Node<E> cur; | ||
| boolean direction; | ||
|
|
||
| TreeSetIterator(boolean r2) { |
There was a problem hiding this comment.
r2 тоже не особо удачное имя для аргумента
| * @return true, if it's new element, false, if element is already exist | ||
| */ | ||
| @Override | ||
| public boolean add(E e) { |
There was a problem hiding this comment.
это слишком длинный и сложный метод, давайте разобьём его на несколько
|
|
||
| Node start = null; | ||
| Node finish = null; | ||
| Node root = null; |
There was a problem hiding this comment.
давайте разделим сущности множества и дерева, которое оно использует для своей работы
| **/ | ||
| @Override | ||
| public MyTreeSet<E> descendingSet() { // | ||
| return null; |
There was a problem hiding this comment.
это как-то не доделано
-0.5
No description provided.