Borrando registros

Continuando con las buenas practicas (try-catch, http-notfound)

    @DeleteMapping("/{idPizza}")
    public ResponseEntity<?> delete(@PathVariable int idPizza){
        try {
            if (this.pizzaService.exists(idPizza)) {
                this.pizzaService.delete(idPizza);
                return ResponseEntity.ok().build();
            }
            return ResponseEntity.status(HttpStatus.NOT_FOUND).body("La Pizza no Existe!");
        } catch (Exception e) {
            log.error(e.getMessage());
            return ResponseEntity.internalServerError().build();
        }
    }

Para no tener doble return:

@DeleteMapping("/{id}")
public ResponseEntity<Void> delete(@PathVariable int id) {
    ResponseEntity<Void> responseEntity = ResponseEntity.notFound().build();
    if (this.pizzaService.exists(id)) {
        this.pizzaService.delete(id);
        responseEntity = ResponseEntity.ok().build();
    }
    return responseEntity;
}

✨ Detalles ✨

No seria mejor que en el endpoint de get pizzas lanzar un error 404 Not Found sino se encuentra la pizza, en lugar de retornar algo ok y vacio

Yo realice el proceso de eliminar asi (se que faltan excepciones y eso)

Service:

public boolean delete(int idPizza){
        if (exists(idPizza)){
            this.pizzaRepository.deleteById(idPizza);
            return true;
        }
        return false;
    }

Controller:

@DeleteMapping("/{idPizza}")
    public ResponseEntity<String> delete(@PathVariable(name = "idPizza") int idPizza) {
        if (this.pizzaService.delete(idPizza)){
            return ResponseEntity.ok("Pizza with ID: " + idPizza + " Deleted");
        }
        return ResponseEntity.notFound().build();
    }