Sprint 9 solution http api#5
Conversation
avfyodorov
left a comment
avfyodorov
left a comment
There was a problem hiding this comment.
Добрый день, Константин!
Хорошая работа, от меня всего пара уточнений.
Просьба обратить дополнительное внимание на возвращаемые коды ошибок:
400 BAD_REQUEST - Ошибка в самом запросе, не удаётся его правильно разобрать.
404 NOT FOUND - это правильный запрос, просто нет данных
405 METHOD_NOT_ALLOWED - Пришло что-то непонятное, с этим мы не работаем
https://developer.mozilla.org/en-US/docs/Web/HTTP/Status/405
| httpServer.createContext("/prioritized", new PrioritizedHandler(manager)); | ||
| } | ||
|
|
||
| public static Gson getGson() { |
There was a problem hiding this comment.
Можно было бы вынести создание Gson в класс BaseHttpHandler, чтобы был доступ прямо в наследниках.
| System.out.println("возвращаем епик " + idPath); | ||
| } else { | ||
| sendNotFound(httpExchange); | ||
| System.out.println("нет такого действия"); |
There was a problem hiding this comment.
Если пришло непонятно что, то возвращать нужно
405 METHOD_NOT_ALLOWED
Почему возвращается статус 405 METHOD_NOT_ALLOWED можно посмотреть тут: https://developer.mozilla.org/en-US/docs/Web/HTTP/Status/405
| } | ||
| } | ||
| default: | ||
| sendNotFound(httpExchange); |
There was a problem hiding this comment.
Если пришло непонятно что, то возвращать нужно либо
400 BAD_REQUEST
либо
405 METHOD_NOT_ALLOWED
| @Override | ||
| public void handle(HttpExchange httpExchange) throws IOException { | ||
| final String path = httpExchange.getRequestURI().getPath(); | ||
| System.out.println(path); |
There was a problem hiding this comment.
Отладочную печать всё-таки лучше удалять после окончания разработки.
На усмотрение.
There was a problem hiding this comment.
какие то убрал, какие то сократил. С ними пока что удобнее ориентироваться в коде
avfyodorov
left a comment
avfyodorov
left a comment
There was a problem hiding this comment.
Добрый вечер, Константин!
Отметил момент, который можно было бы поправить, но, думаю, что Вы справитесь и вне ревью. 🙂
Работа принята.
| public class EpicTaskHandler extends BaseHttpHandler implements HttpHandler { | ||
|
|
||
| private final TaskManager manager; | ||
| private final Gson gson; |
There was a problem hiding this comment.
Переменную также можно было бы перенести в базовый класс, один раз создать и использовать.
2 participants
ТЗ 9