I have a newly created Springboot service class to handle CRUD operations using the JPA derived classed.
There are many examples of this on the internet, some include existsById
checks on delete, update (and get) - some don't.
This is what mine looks like:
public class BirdServiceImpl implements BirdService {
private final BirdRepository birdRepository;
@Override
public Bird create(Bird bird) {
log.info("Saving new bird: {}", bird.getName() );
return birdRepository.save(bird);
}
@Override
public Collection<Bird> list() {
log.info("Finding all birds");
return birdRepository.findAll();
}
@Override
public Bird get(Long id) {
return birdRepository.findById(id).get();
}
@Override
public Bird update(Bird bird) {
return null;
}
@Override
public void delete(Long id) {
log.info("About to delete bird : {}", id);
Bird bird = new Bird();
if (birdRepository.existsById(id)) {
birdRepository.deleteById(id);
}
}
}
QUESTION: Should those checks existsById
be in there or should there be some Exception Handling and if so where?
How can one be sure that the delete was successful as it doesn't return anything?
As per my knowledge two exception is thrown by the deleteById
in following cases
id
is null IllegalArgumentException
is thrown.id
is not present in the persistence store EmptyResultDataAccessException
is thrown. There is no need to check using existsById
as it will initiate unnecessary query to the persistence store (Database).
My suggestion would be following:
If nothing special to be done, except just returning particular HTTP Status
in the api response, then do not handle this exception in the Service
layer. Two cases are possible here
1.1 You want to handle every non existing id cases similarly for every services. In that case you want to handle this in the Controller Advice . Return predefined http status and proper error message for the exceptions thrown by the Service layers.
1.2 You might want to handle deletion failed due to non existence differently, in that case declare a dedicate exception class extended from RuntimeException
. Catch the EmptyResultDataAccessException
inside the service and prepare a exception object of your newly created exception then throw it. Then handle that exception inside the Controller Advice
specially.
Following is how to handle exception in Controller Advice
@RestControllerAdvice()
public class GlobalExceptionHandler extends ResponseEntityExceptionHandler {
@ExceptionHandler(EmptyResultDataAccessException.class)
@ResponseStatus(value = HttpStatus.NOT_FOUND)
public ResponseEntity error(EmptyResultDataAccessException ex) {
return ...; // Build your response body here.
}
}
Service
and do what you want to do (ie you may log entry in the activity log that deletion failed for that id). After doing your business task throw appropriate exception (based on point 1.1
or 1.2
whichever suits your case well) which is meant to be handled in the Controller Advice
.You can use existsById and if it returns true you can delete the specific record otherwise you can throw an exception that it didn't exist.
if (birdRepository.existsById(id)) {
birdRepository.deleteById(id);
}else {
throw new EmptyResultDataAccessException(
String.format("No %s entity with id %s exists!", entityInformation.getJavaType(), id), 1)
);
}
Another option can be using Optional Container.
Bird bird = birdRepository.findById(id)
.orElseThrow(() -> new EmptyResultDataAccessException(
String.format("No %s entity with id %s exists!", entityInformation.getJavaType(), id), 1)))
birdRepository.delete(bird);
It depends whether you want to handle Run Time exceptions and serialize custom response and send back to client. If yes, you can use above approach and handle this in Controller Advice.
Another option would be, you can directly handle the exception in Controller Advice without explicitly throwing anything from service layer. Like handling IllegalArgument Exception or EmptyResultDataAccessException in Controller Advice.
The technical post webpages of this site follow the CC BY-SA 4.0 protocol. If you need to reprint, please indicate the site URL or the original address.Any question please contact:yoyou2525@163.com.