dudoso q los services devuelvan clases http

[maurociancio]

ClienteService ejemplo devuelve return ResponseEntity.ok().body("Cliente creado con éxito.");

eso es responsabilidad del controller, no del service

[LGhosn]

Perfecto, ya lo cambiamos. Gracias!

[LGhosn]

solucionado en los siguietnes commits: ClienteService NegocioService PedidoService PlanService

[maurociancio]

los binarios hacen que los diffs sean imposibles de ver, podrían volver a revisar si tienen que subirlos?

On Wed, Feb 21, 2024 at 2:32 AM Lautaro Ghosn @.***> wrote:

solucionado en los siguietnes commits: ClienteService https://github.com/LGhosn/TakeAwayNow-DCNT/commit/2c3db17cd103f34ecf4c4a706d6b22aea502a386 NegocioService https://github.com/LGhosn/TakeAwayNow-DCNT/commit/aac294c146a164d1e4c07c8c82d14d0407e49666 PedidoService https://github.com/LGhosn/TakeAwayNow-DCNT/commit/400aa66ee61ef44b178149a328f162754672cbc8 PlanService https://github.com/LGhosn/TakeAwayNow-DCNT/commit/29ca4ae9f08f10061926f261c70708e41d71fa00

— Reply to this email directly, view it on GitHub https://github.com/LGhosn/TakeAwayNow-DCNT/issues/12#issuecomment-1955927526, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABGBDP4VQN2J447ETM5RSDYUWBIDAVCNFSM6AAAAABDSGMGKCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNJVHEZDONJSGY . You are receiving this because you authored the thread.Message ID: @.***>

-- Mauro Ciancio http://about.me/maurociancio

[LGhosn]

lo que se hizo fue en todos los services que en vez de devolver el ResponseEntity se lanzara un error el cual es cathcheado en el controller y ahí se devuelve la response correspondiente. por ejemplo:

ClienteService:

    public void crearCliente(String usuario) {
        Optional<Cliente> optionalCliente = clienteRepository.findByUsuario(usuario);
        if (optionalCliente.isPresent()) {
            throw new RuntimeException("Ya existe un usuario con el nombre ingresado.");
        }

        this.clienteRepository.save(new Cliente(usuario));
    }

ClienteController:

    @PostMapping("/")
    public ResponseEntity<String> crearCliente(@RequestParam String nombreUsuario) {
        try {
            clienteService.crearCliente(nombreUsuario);
        } catch (RuntimeException e) {
            return ResponseEntity.internalServerError().body(e.getMessage());
        }
        return ResponseEntity.ok().body("Cliente creado con éxito.");
    }