Manejo de errores con @RestControllerAdvice
Contenido del curso
Salome Arboleda
Alejandro Ramírez
Profesor
Oscar Erazo
David Mauricio Gorraiz Moreno
Roberto Iván Pineda Hernández
Milton Hernan Camacho Quitian
Walter Alexis Sánchez Cerritos
José Galdámez
Milton Hernan Camacho Quitian
Santiago Carmona Pineda
ANDRES ALFONSO MIRA MEJIA
Xavier Armando Larios Alvarenga
Comparto mi solución:
-------------------- Cree la excepción MovieNotFound --------------------
package com.platzi.play.domain.exception;
public class MovieNotFound extends RuntimeException {
public MovieNotFound() {
super("Pelicula no encontrada.");
}
}
-------------------- La agregué al RestExceptionHandler --------------------
@ExceptionHandler(MovieNotFound.class)
public ResponseEntity<Error> handleException(MovieNotFound e) {
return ResponseEntity.status(HttpStatus.NOT_FOUND).body(new Error("movie-not-found", e.getMessage()));
}
-------------------- Así maneje el getById --------------------
public MovieDto getById(long id) {
MovieEntity movieEntity = this.crudMovieEntity.findById(id).orElseThrow(MovieNotFound::new);
return this.movieMapper.toDto(movieEntity);
}
-------------------- Así manejé el update --------------------
public MovieDto update(long id, UpdateMovieDto updateMovieDto) {
MovieEntity movieEntity = this.crudMovieEntity.findById(id).orElseThrow(MovieNotFound::new);
// Acá fue para manejar que no se repitiera el título de la película
if (!movieEntity.getTitulo().equals(updateMovieDto.title())) {
MovieEntity existingMovie = this.crudMovieEntity.findFirstByTitulo(updateMovieDto.title());
if (existingMovie != null) {
throw new MovieAlreadyExistException(updateMovieDto.title());
}
}
this.movieMapper.updateEntityFromDto(updateMovieDto, movieEntity);
return this.movieMapper.toDto(this.crudMovieEntity.save(movieEntity));
}
-------------------- Así maneje el deleteById --------------------
public void deleteById(long id) {
var movieEntity = crudMovieEntity.findById(id).orElseThrow(MovieNotFound::new);
this.crudMovieEntity.delete(movieEntity);
}
Muy bien, Salomé!
Reminder:
Las excepciones del dominio representan errores de negocio, independientemente de como se expone el servicio. Además, no contienen información sobre el protocolo y se pueden reutilizar en cualquier interfaz.
En la capa web, se incluye un manejador global de errores, con el fin de estructurar una respuesta mas diciente de cara al usuario, donde se especifique el tipo de error y un mensaje con el detalle.
Esta separación garantiza que el dominio sea limpio y agnóstico a la web.
Primero cree la clase de que crea la excepcion personalizada:
package com.platzi.play.domain.exception; public class MovieNotExistException extends RuntimeException { public MovieNotExistException(long id) { super("La pelicula con el id " + id + " no existe."); } }
Luego En los metodos para actualizar y borrar se arroja esta excepcion en caso de que no se encuentren las peliculas:
@Override public MovieDto update(long id, UpdateMovieDto updateMovieDto) { MovieEntity movieEntity = this.crudMovieEntity.findById(id).orElse(null); if (movieEntity == null) { throw new MovieNotExistException(id); } this.movieMapper.updateEntityFromDto(updateMovieDto, movieEntity); return this.movieMapper.toDto(this.crudMovieEntity.save(movieEntity)); } @Override public MovieDto delete(long id) { MovieEntity movieEntity = this.crudMovieEntity.findById(id).orElse(null); if (movieEntity == null) { throw new MovieNotExistException(id); } this.crudMovieEntity.delete(movieEntity); return this.movieMapper.toDto(movieEntity); }
Finalmente hago una sobrecarga del metodo handleException pues es el mismo metodo pero con una exception distinta en el parametro que recibe:
package com.platzi.play.web.exception; import com.platzi.play.domain.exception.MovieAlreadyExistException; import com.platzi.play.domain.exception.MovieNotExistException; import org.springframework.http.ResponseEntity; import org.springframework.web.bind.annotation.ExceptionHandler; import org.springframework.web.bind.annotation.RestControllerAdvice; @RestControllerAdvice public class RestExceptionHandler { @ExceptionHandler(MovieAlreadyExistException.class) public ResponseEntity<Error> handleException(MovieAlreadyExistException ex){ Error error = new Error("movie-already-exist", ex.getMessage()); return ResponseEntity.badRequest().body(error); } @ExceptionHandler(MovieNotExistException.class) public ResponseEntity<Error> handleException(MovieNotExistException ex){ Error error = new Error("movie-does-not-exist", ex.getMessage()); return ResponseEntity.badRequest().body(error); } }
Excelente, clase.
Muy buen curso, hasta el momento esta clase de manejo de errores, me hace comprender que la estructura del proyecto actual es compleja para alguien que no tenga experiencia en desarrollo en java y con las herramientas o frameworks que van de la mano.
¿Porqué ? por las diferentes clases que se deben de manejar en las diferentes carpetas donde se debe ir realizando el manejo de estas excepciones.
De la anterior me surgen preguntas como
Ire leyendo mis apuntes y viendo nuevamente las clases anteriores para tener claridad en esta parte y el tema de inyección de independencias para el manejo de esto.
saludos.
Me surgieron preguntas similares al ver la clase, luego de investigar un poco al respecto llegué a las siguientes conclusiones.
Espero te pueda servir para aclarar o darte una idea para que sigas avanzando.
Realice la exception, por si quieren actualizar un registro que no existe, lance una exception que dice la pelicula con tal id no existe:
domain
MovieNotFoundException:
package com.platzi.play.domain.exception;
public class MovieNotFoundException extends RuntimeException {
public MovieNotFoundException(Long id) {
super("La película con id " + id + " no existe");
}
}
metodo
MovieEntityRepository:
@Override
public MovieDto update(long id, UpdateMovieDto updateMovieDto) {
MovieEntity movieEntity = this.crudMovieEntity.findById(id)
.orElseThrow(() -> new MovieNotFoundException(id));
movieEntity.setTitulo(updateMovieDto.title());
movieEntity.setFechaEstreno(updateMovieDto.releaseDate());
movieEntity.setClasificacion(BigDecimal.valueOf(updateMovieDto.rating()));
return this.movieMapper.toDto(this.crudMovieEntity.save(movieEntity));
}
web
RestExceptionHadler:
@ExceptionHandler(MovieNotFoundException.class)
public ResponseEntity<Error> handleMovieNotFound(MovieNotFoundException ex) {
Error error = new Error(
"movie-not-found",
ex.getMessage()
);
return ResponseEntity.status(HttpStatus.NOT_FOUND).body(error);
}
¿No es recomendado utilizar el tradicional try/catch?
Comparto mi propuesta.
Inicialmente creé la clase MovieNotFoundExceptionMovieNotFoundException
package com.platzi.play.domain.exception; public class MovieNotFoundException extends RuntimeException{ public MovieNotFoundException(String message){ super(message); } }
Luego en MovieEntityRepository se lanza la excepción desde el método actualizar y eliminar:
@Override public MovieDto update(long id, UpdateMovieDto updateMovieDto) { MovieEntity movieEntity = this.crudMovieEntity.findById(id).orElse(null); if(movieEntity == null) throw new MovieNotFoundException("Película no encontrada."); this.movieMapper.updateEntityFromDto(updateMovieDto, movieEntity); return this.movieMapper.toDTO(this.crudMovieEntity.save(movieEntity)); } @Override public void delete(long id) { MovieEntity movieEntity = crudMovieEntity.findById(id).orElse(null); if (movieEntity == null) throw new MovieNotFoundException("Película no encontrada."); crudMovieEntity.delete(movieEntity); }
Finalmente, desde el RestExceptionHadler se gestiona la excepción:
@ExceptionHandler(MovieNotFoundException.class) public ResponseEntity<Error> handleException(MovieNotFoundException ex){ Error error = new Error("Movie Not Found", ex.getMessage()); return ResponseEntity.badRequest().body(error); }
No sé si alguno en vez de Postman use Swagger UI (Como en mi caso). El punto es que si alguno lo usa, es posible que tengan un error al lanzar la API y Swagger no responda.
Este error sucedió cuando implementé el RestExceptionHandler. Si les pasa eso, lo más probable es que tengan que cambiar la versión de Swagger y Spring, debido a que hay una incompatibilidad que aún no entiendo.
Yo usé esta configuración en el build.gradle y con eso Swagger ya funcionaba correctamente:
plugins { id 'java' id 'org.springframework.boot' version '3.2.1' id 'io.spring.dependency-management' version '1.1.7'}
// Swagger UIimplementation 'org.springdoc:springdoc-openapi-starter-webmvc-ui:2.3.0'
✅
Se crea el Record "Error" para definir una estructura estandarizada para las respuestas de error de la API, incluyendo un tipo de error y un mensaje descriptivo.
