我有以下Rest Controller类，用于处理与我的Client实体相关的HTTP请求。
对于上下文-一个client（例如锻炼客户端）可以有一个Trainer，但是一个Trainer可以有许多客户端。
我是否以正确的方式从控制器返回实体？我正在使用Spring ResponseEntity类来 Package 客户端相关的响应。为了改进此代码，我是否遗漏了其他控制器实现？

控制器：

@RestController
@RequestMapping(value = "/clients")
public class ClientController {

    private final ClientService clientService;

    public ClientController(ClientService clientService) {
        this.clientService = clientService;
    }

    @GetMapping("/{clientId}")
    ResponseEntity<Client> getClientById(@PathVariable long clientId) {
        Client Client = clientService.getClientById(clientId);
        return new ResponseEntity<>(Client, HttpStatus.OK);
    }

    @GetMapping("/trainer/{trainerId}")
    ResponseEntity<List<Client>> getClientsByTrainerId(@PathVariable long trainerId) {
        List<Client> clients = clientService.getClientsByTrainerId(trainerId);
        return new ResponseEntity<>(clients, HttpStatus.OK);
    }

    @PostMapping
    public ResponseEntity<Client> createClient(@RequestBody ClientDTO clientDTO) {
        Client newClient = clientService.createNewClient(clientDTO);
        return new ResponseEntity<>(newClient, HttpStatus.CREATED);
    }

    @PutMapping("/{clientId}")
    ResponseEntity<Client> updateClient(@PathVariable Long clientId, @RequestBody ClientDTO clientDTO) {
        Client editedClient = clientService.editClient(clientId, clientDTO);
        return new ResponseEntity<>(editedClient, HttpStatus.OK);
    }

}

编辑：添加删除方法

@DeleteMapping("/{clientId}")
long deleteClient(@PathVariable long clientId) {
    clientService.deleteClient(clientId);
    return clientId;
}