Как сделать два метода на один эндпоинт в REST-контроллере Spring?

Question

[Eugenue Cesarevich]

У меня есть несколько ролей и один эндпоинт. Столкнулся с такой ситуацией, что для каждой роли нужен свой набора параметров. Ну вот, например, есть роль CONSUMER и есть роль PRODUCER. И есть эндпоинт, к которому обе роли имеют доступ:

@PostMapping("/order")
public void pay(@RequestParam(value = "consumer-id") Integer consumerId,
                         @RequestParam(value = "consumer-region-code") Integer regionCode,
                         @AuthenticationPrincipal AuthorizedUser user) {
    ...
}

На самом деле для пользователя CONSUMER здесь было бы достаточно иметь только принципал. А вот для пользователя PRODUCER нужен только consumerId и regionCode, зато не нужен принципал. Из-за этого метод очень сильно разрастается, каждая роль привносит что-то новое. В идеале было бы сделать два отдельных метода на один эндпоинт:

@PostMapping("/order")
public void pay(@AuthenticationPrincipal AuthorizedUser user) {
    ...
}

@PostMapping("/order")
public void pay(@RequestParam(value = "consumer-id") Integer consumerId,
                         @RequestParam(value = "consumer-region-code") Integer regionCode) {
    ...
}

Но тут Spring ошибку бросит. Как я могу решить данную проблему с лишними параметрами и разрастанием методов?

Answer 1

[BorLaze]

Вот это не оно?

Comments

[Eugenue Cesarevich]

Именно оно. Вариант конечно дикий, и я уже другой способ нашёл, более адекватный (разбил эндпоинты на /order и /my-order, что вполне корректно), но любопытно, что решение действительно есть.

[BorLaze]

Eugenue Cesarevich, только хотел добавить, что более логично было бы просто иметь два реста типа consumer-order + producer-order :-)

Answer 2

[Орхан Гасанлы]

А почему бы вместо этого:

@PostMapping("/order")
public void pay(@AuthenticationPrincipal AuthorizedUser user) {
    ...
}

@PostMapping("/order")
public void pay(@RequestParam(value = "consumer-id") Integer consumerId,
                         @RequestParam(value = "consumer-region-code") Integer regionCode) {
    ...
}

не использовать:

@PostMapping("/order")
public void pay(
@AuthenticationPrincipal AuthorizedUser user,
@RequestParam(value = "consumer-id", required = false) Integer consumerId,
@RequestParam(value = "consumer-region-code", required = false) Integer regionCode
) {
    ...
}

Comments

[Eugenue Cesarevich]

Ну так я так и делаю сейчас. А потом ещё и ещё роли добавляю. И им ещё и ещё данные нужны. Так вот параметры и разрастаются, получается мегаметод

[сергей кузьмин]

Eugenue Cesarevich, что и продю и консюмсер Get ? почему не разные методы ? в Spring обычно для разделения логики указывают какой метод делвет что

@PostMapping("/tutorials")

@DeleteMapping("/tutorials")

@GetMapping("/tutorials")

и т д

по смыслу вроде consumer быть GET а producer PUT

[Eugenue Cesarevich]

сергей кузьмин, ну я вообще для примера привёл, в реальности там другие сущности. Чтобы проще было, такие сущности привёл. Типа суть в том, что Producer хочет получить заказы, такое же тоже может быть. Это get.

[Орхан Гасанлы]

Eugenue Cesarevich, если честно, то идея с разделением контроллера на 2 мне нравится больше,нежели объединение или перегрузка методов.

[Eugenue Cesarevich]

Орхан Гасанлы, ну я так и решил сделать в итоге. /order и /my-order. Довольно логично

[Орхан Гасанлы]

Eugenue Cesarevich, да, я увидел предыдущий ответ