Skip to content

Sprint 6 solution Доработана функциональность по техническому заданию спринта №6.#2

Merged
Alexander-skipper merged 5 commits intomainfrom
sprint_6-solution
Jul 1, 2025
Merged

Sprint 6 solution Доработана функциональность по техническому заданию спринта №6.#2
Alexander-skipper merged 5 commits intomainfrom
sprint_6-solution

Conversation

@Alexander-skipper
Copy link
Copy Markdown
Owner

Сделать историю посещений неограниченной по размеру.
Избавиться от повторных просмотров в истории.
Протестировать новую функциональность менеджера истории.

Comment thread src/manager/InMemoryHistoryManager.java Outdated
private static final int MAX_HISTORY_SIZE = 10;
private final LinkedList<Task> history = new LinkedList<>();
// Внутренний класс для узла двусвязного списка.
private static class Node {
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Вложенный класс лучше располагать в конце внешнего класса

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Сделано.

Comment thread src/manager/InMemoryHistoryManager.java Outdated
private static final int MAX_HISTORY_SIZE = 10;
private final LinkedList<Task> history = new LinkedList<>();
// Внутренний класс для узла двусвязного списка.
private static class Node {
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Класс можно сделать параметризованным

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Сделано.

@@ -190,6 +197,11 @@ public void deleteAllTasks() {
subtasks.clear();
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

метод deleteAllTasks не совсем корректен, пользователь может захотеть удалить только все подзадачи, или только все задачи. Его стоит разделить на три

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Сделано.

@Alexander-skipper Alexander-skipper merged commit b52b8c9 into main Jul 1, 2025
1 check passed
@Alexander-skipper Alexander-skipper deleted the sprint_6-solution branch July 1, 2025 15:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants