[refactor] Spring data JPA - findBy vs getReferenceBy

[HanKwanJin]

🤔 코드 개선 필요성

저희 코드에서는 findBy 만을 사용하고 있습니다. 조회에는 findBy를 쓰되 저장, 삭제, 수정에 사용되는 findBy 메소드들은 getReferenceBy로 수정하면 어떨까합니다. 이유는 다음과 같습니다.

findBy와 getReferenceBy 둘의 대표적인 차이점으로는 2가지가 있습니다.

1. findBy는 매개변수로 전달된 값에 해당하는 Entity를 반환하거나, 해당하는 Entity가 없을 경우 Optional.empty()를 반환합니다. 즉, 탐색 결과가 없더라도 내부에서 예외를 발생시키기 않습니다. 반면에, getReference는 매개변수로 전달된 값에 해당하는 Entity가 없으면 내부에서 EntityNotFoundException 예외를 발생시킵니다. 그러므로 try-catch로 예외를 처리해줘야하는 코드가 추가되긴합니다.

Returns a reference to the entity with the given identifier.
Depending on how the JPA persistence provider is implemented
this is very likely to always return an instance and throw an EntityNotFoundException on first access.
Some of them will reject invalid identifiers immediately.

2. findBy와 getReferenceBy는 Entity를 찾아오는건 동일하지만 동작 방식이 다릅니다. getReferenceBy를 호출하면 참조값만 가져온 후에 조회된 Entity의 내부에 값이 필요할 때 DB를 조회하는 lazy loading으로 동작합니다. 즉, getReferenceBy를 내부의 값을 필요로 하지 않는 저장, 삭제, 수정에 사용하고 내부의 값이 필요한 조회에 경우 findBy를 사용하면 lazy loading으로 인해 조금이라도 성능 개선이 될 것이라고 예상됩니다.

💡 참고자료

[findBy vs getReferenceBy]

• https://creampuffy.tistory.com/162
• https://tuhrig.de/find-vs-get/

[프록시 lazy loading]

• https://velog.io/@neity16/3-JPA-%ED%94%84%EB%A1%9C%EA%B7%B8%EB%9E%98%EB%B0%8D-9-%ED%94%84%EB%A1%9D%EC%8B%9C-%EC%A7%80%EC%97%B0%EB%A1%9C%EB%94%A9LAZY-%EC%A6%89%EC%8B%9C%EB%A1%9C%EB%94%A9EAGER