Спринт 9,Выполнено ТЗ, Коммит №1 - #5
Conversation
| } else | ||
| writeResponse(exchange, "Ошибка при обработке запроса", 404); | ||
| } catch (IOException e) { | ||
| throw new RuntimeException(e); |
There was a problem hiding this comment.
Не нужно кидать RuntimeException. Либо кидайте какое-то свое иключение, либо из какой-то. библиотеки, но не RuntimeException, так как это слишком общее исключение и его кидает программа, когда что-то не так, а не программист.
| } | ||
| break; | ||
| } | ||
| } catch (Exception e) { |
There was a problem hiding this comment.
Не нужно ловить Exception, а нужно ловить конкретное исключение, которое может кинуть метод в обернутом участке кода.
И не нужно делать два перехвата подряд. Думаю, можно сделать так, чтобы не вкладывать один try/catch в другой.
В других местах, где встречаются аналогичные конструкции, тоже нужно поправить.
| } | ||
|
|
||
| @Override | ||
| public ArrayList<Subtask> getEpicSubtasks(Epic epic) { |
There was a problem hiding this comment.
Возвращайте интерфейс, а не реализацию.
| } | ||
| } | ||
| } catch (IOException e) { | ||
| e.printStackTrace(System.out); |
There was a problem hiding this comment.
Не нужно выводить printStackTrace. Потому что в трейсе может быть персональная информация, которую нельзя писать в лог. Поэтому в лог нужно всегда писать осмысленный текст о том, какая ошибка произошла.
|
|
||
| public void handle(HttpExchange exchange) throws IOException { | ||
| System.out.println("Началась обработка Эпика"); | ||
| try (exchange) { |
There was a problem hiding this comment.
В блок try нужно оборачивать только тот код, который может кинуть исключение. Лишнего кода в этот блок влючать не нужно. Поэтому от вложенного try нужно избавиться.
| } | ||
|
|
||
| @Override | ||
| public LocalDateTime read(final JsonReader jsonReader) throws IOException { |
There was a problem hiding this comment.
Лучше не пробрасывать выше IOException, а обработать эту ошибку там, где она возникла.
| } | ||
|
|
||
| @Override | ||
| public void handle(HttpExchange exchange) throws IOException { |
There was a problem hiding this comment.
Если вы перехватили IOException в 23 строке, то в сигнатуре эту ошибку указывать не нужно.
| } | ||
|
|
||
| break; | ||
| case "DELETE": |
2 participants
No description provided.