Сринт 7. Добавить функцию чтения и записи задач в CSV файл #3
Conversation
случае, если часть грузится из файла.
| @@ -0,0 +1,7 @@ | |||
| package ru.alexgur.kanban.service; | |||
|
|
|||
| public class ManageLoadException extends Exception { | |||
There was a problem hiding this comment.
Лучше наследовать от RuntimeException
Из ТЗ:
кидать собственное непроверяемое исключение ManagerSaveException. Благодаря этому можно не менять сигнатуру методов интерфейса менеджера.
public class RuntimeException extends Exception — базовый класс для ошибок во время выполнения. Относится к необрабатываемым исключениям (uncatched\unchecked). Как сказано в описании класса — это суперкласс, исключения которого могут быть выброшены во время нормальной работы JVM.
There was a problem hiding this comment.
Поменял на RuntimeException
| @@ -0,0 +1,7 @@ | |||
| package ru.alexgur.kanban.service; | |||
There was a problem hiding this comment.
Собственные исключения обычно хранят в отдельных пакетах, например, exceptions
There was a problem hiding this comment.
Перенёс исключения в ru.alexgur.kanban.exceptions;
| this.csvSplitter = csvSplitter; | ||
| } | ||
|
|
||
| public static FileBackedTaskManager loadFromFile(File file) throws ManageLoadException { |
There was a problem hiding this comment.
throws ManageLoadException
Если наследовать от RuntimeException, то исключение будет непроверяемым и оно само пробросится, можно будет обойтись без throws
There was a problem hiding this comment.
Поменял на RuntimeException, убрал throws
| } catch (IOException e) { | ||
| throw new ManagerSaveException("Произошла ошибка во время записи файла."); | ||
| } | ||
| } catch (Exception e) { |
There was a problem hiding this comment.
Действительно ли нужно перехватывать Exception ?
There was a problem hiding this comment.
Убрал внешний try-catch
на RuntimeException, убрать лишние throws-try-catch
Добавить функцию чтения и записи задач в CSV файл