Endpoint PUT con MapStruct en Spring Boot
Contenido del curso
Salome Arboleda
David Mauricio Gorraiz Moreno
Linda Montoya
ANDRES ALFONSO MIRA MEJIA
Xavier Armando Larios Alvarenga
JHONATAN AGUILERA GARCIA
Frederick Lozano
Ingenieria Resuelve
Milton Hernan Camacho Quitian
Mi solución:
--------------------------- MovieController ---------------------------
@DeleteMapping("/{id}")
public ResponseEntity<String> deleteById(@PathVariable long id) {
movieService.deleteById(id);
return ResponseEntity.ok("Movie with ID " + id + " was deleted successfully.");
}
--------------------------- MovieRepository ---------------------------
void deleteById(long id);
--------------------------- MovieEntityRepository ---------------------------
@Override
public void deleteById(long id) {
crudMovieEntity.findById(id)
.ifPresent(crudMovieEntity::delete);
}
--------------------------- MovieService ---------------------------
public void deleteById(long id) {
this.movieRepository.deleteById(id);
}
Primero agregue el metodo para borrar en MovieRepository:
package com.platzi.play.domain.repository; import com.platzi.play.domain.dto.MovieDto; import com.platzi.play.domain.dto.UpdateMovieDto; import java.util.List; public interface MovieRepository { List<MovieDto> getAll(); MovieDto getById(Long id); MovieDto save(MovieDto movieDto); MovieDto update(long id, UpdateMovieDto updateMovieDto); MovieDto delete(long id); } ```Luego agregue su funcionamiento en la clase MovieEntityRepository: ```java @Override public MovieDto delete(long id) { MovieEntity movieEntity = this.crudMovieEntity.findById(id).orElse(null); if (movieEntity == null) return null; this.crudMovieEntity.delete(movieEntity); return this.movieMapper.toDto(movieEntity); } ```Luego en MovieService cree el metodo que se usara en el controller posteriormente: ```java public MovieDto delete(long id){ return this.movieRepository.delete(id); } ```Finalmente implemente este metodo del service en el MovieController: ```java @DeleteMapping("/{id}") public ResponseEntity<MovieDto> delete(@PathVariable long id){ return ResponseEntity.ok(this.movieService.delete(id)); }
Mi solución fue la siguiente:
Cree un método en la interfaz MovieRepository
Hice la implementación usan Streams
Implemente el método delete en la clase MovieService
Implemente el método en el Controlador llamando al método del servicio
✅
Comparto mi solución, pero hice un borrado lógico, para no perder información de la entidad:
//Solicitud HTTP en Postman { "status": "D" // o "I" para INACTIVE } //Creación de Record para manejar el estado package com.platzi.play.domain.dto; public record LogicDeleteDTO(String status) {} //MovieRepository MovieDTO deleteLogic(long id, LogicDeleteDTO logicDeleteDTO); //MovieMapper @Mapping(target = "estado", source = "status") void updateEntityStatus(LogicDeleteDTO logicDeleteDTO, @MappingTarget MovieEntity movieEntity); //MovieEntityRepository public MovieDTO deleteLogic(long id, LogicDeleteDTO logicDeleteDTO) { MovieEntity movieEntity = crudMovieEntity.findById(id).orElse(null); if (movieEntity == null) return null; this.movieMapper.updateEntityStatus(logicDeleteDTO, movieEntity); return this.movieMapper.toDto(this.crudMovieEntity.save(movieEntity));} //MovieController @PutMapping("change-status/{id}") public ResponseEntity<MovieDTO> deleteMovieLogic(@PathVariable long id, @RequestBody LogicDeleteDTO logicDeleteDTO) { return ResponseEntity.ok(this.movieService.deleteLogic(id, logicDeleteDTO));}
Me gustaría saber si estoy utilizando la lógica adecuada o incluso el StatusMapper de forma correcta.
Solución al reto Controller
@DeleteMapping("/{id}") public ResponseEntity<String> deleteMovie(@PathVariable long id) { this.movieService.deleteMovie(id); return ResponseEntity.ok("Movie has been deleted"); }
Repository
void deleteMovie(long id);
Service
public void deleteMovie(long id) { this.movieRepository.deleteMovie(id); }
MovieEntityRepository
@Overridepublic void deleteMovie(long id) { if(this.crudMovieEntity.findById(id).isPresent()) { this.crudMovieEntity.deleteById(id); }}
Hola todos,
No entiendo por que hacer 2 select a la base de datos ?
creo que solo deberia haber uno, validar si retorna null
algo asi:
@PutMapping("/{id}")public ResponseEntity<MovieDto> update(@PathVariable long id, @RequestBody UpdateMovieDto updateMovieDto) { MovieDto movieDto = this.movieService.update(id, updateMovieDto); if (movieDto == null) { return ResponseEntity.notFound().build(); } return ResponseEntity.ok(movieDto);}
PD: "puede que lo que diga esta mal"
porque en el metodo update primero va el target y luego el source??
Comparto mi solución para eliminar una película.
En la interfaz MovieRepository se agregó el método delete:
void delete(long id);
Luego, en la clase MovieEntityRepository se realizó la implementación:
@Override public void delete(long id) { MovieEntity movieEntity = crudMovieEntity.findById(id).orElse(null); if (movieEntity == null) throw new RuntimeException("Pelicula no encontrada"); crudMovieEntity.delete(movieEntity); }
Se crea el método delete en MovieService:
public void delete(long id){ this.movieRepository.delete(id); }
Finalmente, en MovieController se crea el método delete el cual responde a una petición HTTP de tipo Delete:
@DeleteMapping("/{id}") public ResponseEntity<Void> delete(@PathVariable long id) { this.movieService.delete(id); return ResponseEntity.noContent().build(); }
