Sprint 9 solution http api #4
Conversation
src/Managers.java
Outdated
| public static TaskManager getDefault() { | ||
| return new FileBackedTaskManager(); | ||
| /* FileBackedTaskManager manager = | ||
| FileBackedTaskManager.loadFromFile(new File (".\\data\\tasks.csv")); |
src/BaseHttpHandler.java
Outdated
| public class BaseHttpHandler { | ||
| protected static final Charset DEFAULT_CHARSET = StandardCharsets.UTF_8; | ||
|
|
||
| public void sendText(HttpExchange h, String text, int retCode) throws IOException { |
There was a problem hiding this comment.
Классы, которые реализуют интерфейс AutoCloseable можно обрабатывать с помощью try with resources
There was a problem hiding this comment.
Добавил обработку исключений.
src/BaseHttpHandler.java
Outdated
| public class BaseHttpHandler { | ||
| protected static final Charset DEFAULT_CHARSET = StandardCharsets.UTF_8; | ||
|
|
||
| public void sendText(HttpExchange h, String text, int retCode) throws IOException { |
There was a problem hiding this comment.
Более оптимальным вариантом обработки исключений является использование блоков try catch, так как это позволяет сделать более гибкую обработку ошибок(например добавить логи) и избавляет от необходимости пробрасывать исключения по цепочке
There was a problem hiding this comment.
Добавил try catch в обработчики http запросов.
src/BaseHttpHandler.java
Outdated
| * | ||
| * @return - объект конвертера gson | ||
| */ | ||
| public Gson getGson() { |
There was a problem hiding this comment.
Gson лучше сделать статическим, это избавит от необходимости каждый раз создавать новый объект
| import java.time.LocalDateTime; | ||
| import java.util.Optional; | ||
|
|
||
| public class BaseHttpHandler { |
There was a problem hiding this comment.
Было бы хорошо структурировать проект, например все классы связанные с обработкой http вынести в отдельную папку
There was a problem hiding this comment.
Распределил все классы по пакетам.
No description provided.