Sprint 8 solution time and duration by Max-Browckin · Pull Request #3 · Max-Browckin/java-kanban

Max-Browckin · 2025-02-17T22:31:55Z

No description provided.

avfyodorov

Добрый день, Максим!

Практически всё сделано, единственное нужно будет немного поправить работу с отсортированным по времени списком.

avfyodorov · 2025-02-18T09:42:15Z

src/model/Epic.java

 public class Epic extends Task {
    private ArrayList<Subtask> subtaskList = new ArrayList<>();
+    private Duration duration;
+    private LocalDateTime startTime;


private Duration duration; private LocalDateTime startTime;

Эти два поля лишние, потому что они дублируют соответствующие поля в классе предке - Task.
Нужно использовать поля класса Task
Поля в Task лучше сделать protected, чтобы они были доступны напрямую.

avfyodorov · 2025-02-18T09:47:40Z

src/manager/ManagerSaveException.java

@@ -0,0 +1,7 @@
+package manager;


Собственные исключения принято хранить в отдельном пакете, например, exceptions

avfyodorov · 2025-02-18T09:59:59Z

src/manager/InMemoryTaskManager.java

-    private int getNextID() {
-        return nextID++;
+    @Override
+    public List<Task> getPrioritizedTasks() {


К сожалению, данная реализация не соответствует требованиям ТЗ:

.=============
Предполагается, что пользователь будет часто запрашивать этот список, поэтому подберите подходящую структуру данных для хранения. Сложность получения должна быть уменьшена ... до O(n)
...
Если сортировать список заново каждый раз, сложность получения будет O(n*log(n)). Можно хранить все задачи заранее отсортированными с помощью класса TreeSet.
.===========
Метод получения списка отсортированных задач должен выглядеть так:

@Override public List<Task> getPrioritizedTasks() { return new ArrayList<>(prioritizedTasks); }

Нужно будет завести переменную - отсортированный список и поддерживать её в актуальном состоянии.

protected final Set<Task> prioritizedTasks = new TreeSet<>(Comparator.comparing(Task::getStartTime));

avfyodorov

Добрый вечер, Максим!

Ещё несколько небольших уточнений.
И ставьте, пожалуйста, пометки, чтобы было понятно, что комментарии замечены. )

avfyodorov · 2025-02-18T17:27:44Z

src/manager/InMemoryTaskManager.java

    @Override
    public void deleteTasks() {
        tasks.clear();
+        prioritizedTasks.clear();


Не совсем так. Лучше как-то так -

@Override public void deleteTasks() { for (Task task : tasks.values()) { historyManager.remove(task.getId()); prioritizedTasks.remove(task); } tasks.clear(); }

Все комментарии замечены, все исправил
Надеюсь правильно

avfyodorov · 2025-02-18T17:29:22Z

src/manager/InMemoryTaskManager.java

    public void deleteEpics() {
        epics.clear();
        subtasks.clear();
+        prioritizedTasks.clear();


И здесь чистить из истории и из отсортированного списка, но ТОЛЬКО ПОДЗАДАЧИ.

avfyodorov · 2025-02-18T17:29:57Z

src/manager/InMemoryTaskManager.java

    @Override
    public void deleteSubtasks() {
        subtasks.clear();
+        prioritizedTasks.clear();


и здесь

avfyodorov · 2025-02-18T17:30:25Z

src/manager/InMemoryTaskManager.java

-        tasks.remove(id);
+        Task task = tasks.remove(id);
+        if (task != null) {
+            prioritizedTasks.remove(task);


и из истории

avfyodorov · 2025-02-18T17:30:45Z

src/manager/InMemoryTaskManager.java

-            subtasks.remove(subtask.getId());
+        Epic epic = epics.get(id);
+        if (epic != null) {
+            ArrayList<Subtask> epicSubtasks = epic.getSubtaskList();


и из истории

avfyodorov · 2025-02-18T17:31:43Z

src/manager/InMemoryTaskManager.java

+            int epicID = subtask.getEpicID();
+            subtasks.remove(id);
+            prioritizedTasks.remove(subtask);
+            historyManager.remove(id);


а здесь уже есть :-)

avfyodorov

Добрый день, Максим!

Замечаний нет.
Работа принята.

Max added 25 commits December 4, 2024 18:15

commit 7

0087d5b

commit 8

25f1cef

commit 9

a983b45

commit 10

10a757e

commit 11

9b3d3d1

commit 12

2c3695b

commit 13

c08dbf8

commit 14

73316fd

commit 15

b781730

Add saving tasks to a file

46194f8

Add saving tasks to a file

f0747a1

Add saving tasks to a file

a3c8ae8

Error correction

d5603a6

Error correction

f9a8384

Error correction

4a37c98

implemented methods with conservation

2e78efb

sprint 8

ac4a7ed

solution time and duration

ed0d0c1

solution time and duration

e0dd1d9

solution time and duration

f27b481

solution time and duration

11ee931

solution time and duration

09608fb

solution time and duration

512c446

solution time and duration

d042219

solution time and duration

fd5507e

avfyodorov reviewed Feb 18, 2025

View reviewed changes

correcting errors and reducing the difficulty of getting to O(n)

f0fffdc

avfyodorov reviewed Feb 18, 2025

View reviewed changes

correcting errors

918d520

avfyodorov reviewed Feb 19, 2025

View reviewed changes

Conversation

Max-Browckin commented Feb 17, 2025

Uh oh!

avfyodorov left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

avfyodorov left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

avfyodorov left a comment

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants