[SpringBoot/JPA] deleteById에 잘못된 값을 넣어도 오류가 발생하지 않음

📌환경

IntelliJ Ultimate

Java 17

 

Spring boot 3.2.3

Gradle - Groovy

Dependencies: 

Spring Web

Thymeleaf

Spring Data JPA

lombok

MariaDB 10.11

Spring Dev tool

 

📌문제

`Reply`를 삭제 처리하기 위하여 /URI/{rno}에 `DELETE` 방식으로 삭제를 요청함.

`Reply`의 아이디인 `rno`를 데이터베이스에 존재하는 아이디로 설정할 경우 정상적으로 동작. 그러나 데이터베이스에 없는 번호로 설정하고 요청하면 Exception 없이 code 200을 반환 받았음.

기대했던 Exception : `EmptyResultDataAccessException` 500

결과: 200

 

 

 

Hibernate: 
    select
        r1_0.rno,
        r1_0.board_bno,
        r1_0.moddate,
        r1_0.regdate,
        r1_0.reply_text,
        r1_0.replyer 
    from
        reply r1_0 
    where
        r1_0.rno=?

 

 

 

console에서 select문만 실행되고 아무런 오류 없이 끝나는 것을 확인했다. 심지어 `NoSuchElementException`도 발생하지 않는다... 그래서 `deleteById()` 메서드를 까보기로 했다.

 

`CrudRepository`의 구현체인 `SimpleJpaRepository`의 `deleteById()` 메서드는 다음과 같다.

 

@Transactional
@Override
public void deleteById(ID id) {

    Assert.notNull(id, ID_MUST_NOT_BE_NULL);

    findById(id).ifPresent(this::delete);
}

 

 

내부에서 `findById()`를 실행한 후 존재하면 `delete`를 실행하는 것을 볼 수 있다. 그런데 `findById(id)`가 null일 경우 아무런 Exception을 발생시키지 않는다!

 

2023년 1월에 작성된 블로그에서 봤던 `deleteById`와 달라보였다. 출처: https://velog.io/@baekgom/deleteById-delete

 

https://velog.io/@baekgom/deleteById-delete

 

이미지 내에서는 `findById` 실행 시 `orElseThrow( () -> new EmptyResultDataAccessException)`를 발생시키고 있으나 직접 까본 `SimpleJpaRepository`에는 해당 내용이 없었다. 

 

 

그럼 이제 `delete()`를 보자.

 

@Override
@Transactional
@SuppressWarnings("unchecked")
public void delete(T entity) {

    Assert.notNull(entity, "Entity must not be null");

    if (entityInformation.isNew(entity)) {
        return;
    }

    Class<?> type = ProxyUtils.getUserClass(entity);

    T existing = (T) entityManager.find(type, entityInformation.getId(entity));

    // if the entity to be deleted doesn't exist, delete is a NOOP
    if (existing == null) {
        return;
    }

    entityManager.remove(entityManager.contains(entity) ? entity : entityManager.merge(entity));
}

 

 

내부에서 `entityManager`가 `find()`로 값을 찾은 뒤 존재할 경우 `remove()`를 호출하여 삭제하고 있다. 이 부분은 위의 블로그에서 확인한 `delete()` 메서드와 동일했다. 

변경된 이유는 알 수 없으나 `deleteById()`의 문제로 보고 삭제 로직을 변경하기로 했다.

 

 

 

📌해결

`Service`에서 삭제 메서드 구현 시 `deleteById()`를 사용함에 따라 발생하는 문제로 보고 `findById()`로 객체를 찾아 해당 객체를 매개변수로 `delete()`를 호출하기로 했다.

 

@Override
public void remove(Long rno) {

//        replyRepository.deleteById(rno);

    Optional<Reply> result = replyRepository.findById(rno);
    Reply reply = result.orElseThrow();
    replyRepository.delete(reply);

}

 

 

 

 

`NoSuchElementException`이 정상적으로 동작하는 것을 확인할 수 있다.

 

 

 

📌참고

자바 웹 개발 워크북

https://velog.io/@baekgom/deleteById-delete

deleteById, delete

https://docs.spring.io/spring-data/jpa/reference/repositories/core-domain-events.html#page-title

Publishing Events from Aggregate Roots :: Spring Data JPA