Спринт 6 #2
Спринт 6 #2
Conversation
…a-kanban into sprint_6-solution
|
✅ Все автотесты на GIT пройдены |
|
|
||
| import ru.alexgur.kanban.model.Task; | ||
|
|
||
| public class Node { |
There was a problem hiding this comment.
Этот класс можно сделать внутренним прямо в менеджере истории, потому что больше он нигде не нужен.
На усмотрение.
There was a problem hiding this comment.
Сделал класс внутренним, удалил файл.
| @@ -10,6 +10,8 @@ public interface HistoryManager { | |||
|
|
|||
| void clear(); | |||
There was a problem hiding this comment.
clear
По техническому заданию этот метод не требуется добавлять в интерфейс, его можно сделать внутренним методом менеджера истории.
| @Override | ||
| public void add(Task task) { | ||
| addImpl(task); | ||
| } | ||
|
|
||
| @Override | ||
| public List<Task> getHistory() { | ||
| return getHistoryImpl(); | ||
| List<Task> listTasks = new ArrayList<>(); | ||
| for (Node node : getTasks()) { |
There was a problem hiding this comment.
Формирование общего списка задач в истории можно было бы упростить.
На усмотрение.
public ArrayList<Task> getHistory() {
ArrayList<Task> tasks = new ArrayList<>();
Node node = first;
while (node != null) {
tasks.add(node.task);
node = node.next;
}
return tasks;
}
There was a problem hiding this comment.
Пока оставил, чтобы не было дубликования кода с приватным методом getTasks.
There was a problem hiding this comment.
Так в этом случае приватный метод вообще будет не нужен.
test/ru/alexgur/kanban/Tests.java
Outdated
| @@ -1,21 +1,22 @@ | |||
| package ru.alexgur.kanban.tests; | |||
| package test.ru.alexgur.kanban; | |||
There was a problem hiding this comment.
Обычно для тестов создаётся отдельное дерево пакетов, аналогичное структуре пакетов кода.
src
-- tracker
-- controllers
-- model
.......................
test
-- tracker
-- controllers
-- model
.......................
There was a problem hiding this comment.
То есть надо тесты разделить по разным файлам, а не все в один файл ставить?
Например, есть файл с кодом src/ru/alexgur/kanban/model/Task,java
Для методов в этом файле должны быть тесты в другом файле test/ru/alexgur/kanban/model/Task,java
Верно понял? Или как-то по-другому?
There was a problem hiding this comment.
Разделил все тесты на файлы (*Test.java), разбил по папкам.
There was a problem hiding this comment.
Да, совершенно верно, обычно на один тестируемый файл создаётся соответствующий класс-тест
класс внутренним и приватным, удалить clear из интерфейса HistoryManager
Сделать бесконечную историю со скоростью доступа О(1)