Спринт 8 #4
Спринт 8 #4
Conversation
список приоритетных задач, переделать тесты
AlekseyGur
changed the title
| } | ||
| if (startTime != null) { | ||
| epic.setStartTime(startTime); | ||
| epic.setEndTime(epic.getStartTime().plus(duration)); |
There was a problem hiding this comment.
Здесь немного не так. По техническому заданию:
А вот с классом Epic нужно поработать дополнительно. Продолжительность эпика — сумма продолжительностей всех его подзадач. Время начала — дата старта самой ранней подзадачи,
а время завершения — время окончания самой поздней из задач.
There was a problem hiding this comment.
Поправил, добавил тесты на эти три условия
| public List<Task> getPrioritizedTasks() { | ||
| return Stream.concat(getTasks().stream(), getSubTasks().stream()) | ||
| .filter(x -> x.getStartTime() == null) | ||
| .sorted(Task::compareToStartTimeAsc) |
There was a problem hiding this comment.
Сортировать задачи лучше при добавлении в этот список. По техническому заданию:
Предполагается, что пользователь будет часто запрашивать этот список, поэтому подберите подходящую структуру данных для хранения. Сложность получения должна быть уменьшена ... до O(n).
То есть метод должен быть приблизительно таким:
@Override
public List<Task> getPrioritizedTasks() {
return new ArrayList<>(prioritizedTasks);
}There was a problem hiding this comment.
Исправил, добавил в интерфейс getPrioritizedTasks и тест для него
| tasks.put(task.id, task); | ||
| addTreeOfTasksAndSubTasks(task); | ||
| return task.id; | ||
| } | ||
|
|
||
| @Override | ||
| public int addSubTask(SubTask subtask) { | ||
| subTasks.put(subtask.id, subtask); |
There was a problem hiding this comment.
Перед добавлением подзадачи также нужно проверить нет ли пересечения с другими задачами и подзадачами, уже добавленными в отсортированный список.
if (isTaskCrossesSavedTasks(subtask)) {
.................| return epic.id; | ||
| } | ||
|
|
||
| @Override | ||
| public void updateTask(Task task) { | ||
| updateTreeOfTasksAndSubTasks(task); |
There was a problem hiding this comment.
У обновляемой задачи могли измениться и время начала и продолжительность, поэтому здесь тоже нужно проверить на пересечение по времени.
There was a problem hiding this comment.
Добавил проверку для обновлений, дополнил тесты
Добавить продолжительность и дату старта задач, список приоритетных задач, переделать тесты