Sprint 9 solution http api

[Max-Browckin]

No description provided.

[avfyodorov]

Добрый день, Максим!

Отличная работа, есть всего одно уточнение, но и оно на Ваше усмотрение.

[avfyodorov]

1. Можно было бы провести рефакторинг, в базовый класс (BaseHttpHandler) перенести обработчик и там же методы-заглушки, а в классах наследниках переопределять только эти методы (processGet, processPost….), если они нужны. Такое решение лучше соответствует принципам OOP.
   Такой подход уменьшает количество повторяющегося кода и позволяет проще вносить в него изменения.

На усмотрение.

[Max-Browckin]

Я их изучил, но пока решил не вносить изменения, так как они не критичны для проекта.

[avfyodorov]

public  class BaseHandler implements HttpHandler {
......................

public void handle(HttpExchange exchange) throws IOException { 
       String method = exchange.getRequestMethod();
           switch (method) {
               case "GET":
                   processGet(exchange);
                   break;
               case "POST":
                   processPost(exchange);
                   break;
               case "DELETE":
                   processDelete(exchange);
                   break;
               default:
                   writeToUser(exchange, "Данный метод не предусмотрен");
           }
           
    protected void processGet(....) {
        sendMethodNotAllowed(...);
   }
   protected void processPost(....) {
        sendMethodNotAllowed(...);
   }
.......................
И здесь же общие методы для классов-наследников. 

[avfyodorov]

3. Далее в каждом из наследников мы будем переопределять только нужные для этого наследника методы processGet, processPost и т.д. и реализовывать в них нужную нам логику)

Для задач приблизительно так:

public class TasksHandler extends BaseHandler {
......................
    public TasksHandler(........................
    
    @Override
    protected void processGet(HttpExchange exchange, String path) throws IOException {
  .........................
    @Override
    protected void processPost(HttpExchange exchange) throws IOException {
 ...............
    @Override
    protected void processDelete(HttpExchange exchange, String path) throws IOException {
.......................

[avfyodorov]

А для отсортированного по времени списка достаточно будет переопределить всего один метод:

public class PrioritizedHandler extends BaseHandler {

    @Override
    protected void processGet(HttpExchange exchange, String path) throws IOException {
..................

.---------------------

Аналогично определяются классы-наследники и для эпиков, подзадач, истории.

[avfyodorov]

Добрый день, Максим!

Замечаний нет.
Работа принята.