Спринт 6. Изменения хранения истории просмотра задач на LinkedList и … #1
Conversation
…удаление дублей в истории
…ban into sprint_6-solution
| tail.setNext(newNode); | ||
| } | ||
| this.tail = newNode; | ||
| removeFromHistory(task.getId()); |
There was a problem hiding this comment.
Обязательно
Сначала удаляем, затем линкуем и добавляем в список. В целом в вашем варианте, тоже верно, но логичнее сначала удалить старую запись, а затем добавить новую.
| private void linkLast(Task task) { | ||
| Node newNode = new Node(task, null, this.tail); | ||
| if (this.tail == null) { | ||
| this.head = newNode; |
There was a problem hiding this comment.
Можно лучше
Писать this не обязательно везде, если нет противоречий с названий переменных. Во вторых у вас например в 32 строчке без this указано, в других с this, лучше чтобы было одинаково и без this
| } | ||
|
|
||
| @Override | ||
| public void removeFromHistory(int id) { |
There was a problem hiding this comment.
Обязательно
Также нужно доработать сам таск менеджер и вызывать этот метод когда удаляются сами задачи, подзадачи и эпик не только удаление по id, но и при удалении всех задач, подзадач, эпиков. Согласно заданию
Добавьте вызов метода при удалении задач, чтобы они удалялись также из истории просмотров.
Добавила удаление из истории при удалении задач в таск менеджере, добавила JUnit проверку, в Main реализовала пользовательский сценарий для просмотра истории при различных случаях удаления задач в таск менеджере. |
Изменение хранения истории просмотра задач на LinkedList, удаление дублей в истории и удаление из истории при удалении задач в "Менеджер задач".
Новые тесты на класс InMemoryHistoryManager с учетом нового функционала и проверка на удаление ID сабтасок в списке у соответствующего эпика.