Как обрабатывать изменяемые параметры запроса?

Question

[PenguinVasily]

Здравствуйте, есть задача реализовать RESTful API сервис, который обрабатывает определенные изображения на пк. Информация об изображении передается в json формате, в теле запроса. Пример запроса:

{
    "image": "D:/images/[название изображения].png"
}

Но проблема заключается в том, что тип операции, которую необходимо произвести с изображением, может передаваться как с URL, так и с json запросом - добавляя поле "operation": "flip".

Если с url всё понятно, просто создаем необходимые методы в Rest-контроллере, то со вторым вариантом я туплю.

Мне необходимо создать метод, который принимает расширенный вариант запроса и с помощью switch-case вызывать определенные методы сервиса?

@PostMapping(value = "/", consumes = MediaType.APPLICATION_JSON_VALUE, 
             produces = {MediaType.APPLICATION_XML_VALUE, MediaType.APPLICATION_JSON_VALUE})
public ResponseEntity<RequestWithOperation> common(@RequestBody RequestWithOperation request) {
    switch (request.getOperation()) {
        case "flip":
        imageService.flip(request.getImage());
        break;

        // Остальные вызовы case...
    }

    return ResponseEntity.ok(new RequestWithOperation(request.getImage(), request.getOperation()));
}

Мозг мне подсказывает, что это неправильный вариант. Как минимум, код начинает повторятся в местах вызова методов сервиса.

Также хочется узнать, куда будет правильней вынести определение POJO классов Request и RequestWithOperation? В пакет models? С бд работы тут нет.

Answer 1

[Dmitry Roo]

В контроллере не должно быть логики точно.
Я бы в контроллере как-то получал необходимые параметры изображение и метод и передавал в сервис.
Сервис пусть там уже разбирается.

Comments

[PenguinVasily]

У меня в методах контроллера берется информация об изображении и передается методам сервиса, где уже всё обрабатывается.

[PenguinVasily]

У меня проблема с тем, чтобы вызвать правильный метод сервиса в контроллере. С URL то понятно localhost:8080/flip/ -> в контроллере отрабатывает метод flip, который передает информацию об изображении сервису.

[PenguinVasily]

Я так понял, что "расширенный" запрос, который содержит название нужной операции - приходит просто на localhost:8080/

[Dmitry Roo]

PenguinVasily, ну так вы сделайте несколько методов.
Один на эндпоинт / , который будет обрабатывать запрос с указанием метода в теле запроса.
И другой эндпоинт, который будет обрабатывать PathVariable

[PenguinVasily]

Dmitry Roo, сейчас всё так и есть. У меня сомнения, что правильно вызывать методы сервиса, через switch-case. Или вы предлагаете сделать отдельный метод в сервисе, где в параметрах я должен буду передать строку на изображение и название операции?

[Dmitry Roo]

PenguinVasily, да. Я предлагаю логику вынести в сервис, как я и написал в ответе "в контроллере не должно быть логики"