Skip to content

Реализация задач ТЗ спринта 7#5

Merged
h0ttab merged 21 commits intomainfrom
sprint_7-solution-in-file-manager
Jun 10, 2025
Merged

Реализация задач ТЗ спринта 7#5
h0ttab merged 21 commits intomainfrom
sprint_7-solution-in-file-manager

Conversation

@h0ttab
Copy link
Owner

@h0ttab h0ttab commented Jun 8, 2025

Что было сделано

Согласно ТЗ финального задания спринта 7 в приложение были внесены следующие изменения:

  • Создана новая реализация менеджера задач - класс FileBackedTaskManager.
    • Сохраняет задачи в CSV-файл, обеспечивая их восстановление при следующем запуске.
    • Поддерживает работу с разными файлами — можно создавать несколько независимых менеджеров.
    • Метод загрузки из файла реализован с перегрузкой: можно передавать путь к файлу как String, Path или File.
  • Добавлены unchecked-исключения ManagerSaveException и ManagerLoadException для обработки ошибок записи/чтения.
  • Обновлены и расширены unit-тесты.
  • В классе FileBackedTaskManager было выполнено дополнительное задание "Реализуем пользовательский сценарий"

Дополнительно:

  • Рефакторинг и мелкие улучшения ранее созданных классов.
  • Добавлен класс Utils с вспомогательными методами (вынесены в целях декомпозиции).
  • Для некоторых unit-тестов применён Java Reflection API — это позволило протестировать приватные поля и методы без изменения их модификаторов доступа.

Комментарии для ревьюера:

  • Старался писать гибкий и масштабируемый код, с возможностью дальнейшего расширения и минимальными точками жёсткой связки (например, при изменении структуры CSV).
  • Возможно, в некоторых местах решения избыточны или неоптимальны — буду рад замечаниям по этому поводу и готов внести правки.
  • Если какие-то архитектурные решения вызывают сомнения или можно сделать проще — с удовольствием приму критику и пересмотрю их.

h0ttab and others added 20 commits June 6, 2025 03:23
…/java-kanban into sprint_7-solution-in-file-manager
@h0ttab h0ttab merged commit c71e067 into main Jun 10, 2025
1 check passed
@h0ttab h0ttab deleted the sprint_7-solution-in-file-manager branch June 10, 2025 20:48
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