티스토리 뷰

728x90

코드를 리팩토링하면서 대부분이 코드 스멜을 스멀스멀 풍겼지만

그 중에 최악의 코드 스멜은 for loop를 돌며 findById를 매번 해오는 코드였다.

 

코드를 리팩토링해버려서 대충 다시 작성해보자면 다음과 같은 코드였다.

for (Long myId : myIds) {
    myRepository.findByMyId(myId);
}

 

for문을 돌면서 값을 하나씩 받아 매번 DB I/O를 발생시키는 코드였다..😞

 

 

해당 코드는 여러 검색과 질문을 통해 다음과 같은 코드로 변경시킬 수 있었다.

 

List<Entity> findByForeignIdIn(List<Long> foreignIds);

 

JPA의 IN clause를 사용하는 아주 간단한 해결 방법이다.

IN은 범위 조건문으로 BETWEEN, LIKE처럼 사용되지만 모든 값을 일일히 입력해야 하는 문제가 있다.

하지만 WAS에서 FK만 따로 추출해서 받아오면 되므로 큰 문제가 되지 않았다!

 

실성능차이

IN 조건절이 명백히 더 좋아보이는 것은 맞았지만 실제로 얼마나 성능 차이가 날지 궁금했고 바로 실험을 진행해보았다.

 

실험 결과는 다음과 같다

 

100건 기준

//0.30981초
for (Long myId : myIds) {
    myRepository.findByMyId(myId);
}

//0.132818초
List<Entity> findByForeignIdIn(List<Long> foreignIds);

 

1000건 기준

//2.4329초
for (Long myId : myIds) {
    myRepository.findByMyId(myId);
}

//0.483534초
List<Entity> findByForeignIdIn(List<Long> foreignIds);

 

100건 기준으로는 2배 정도의 차이가 났지만

1000건에서는 5배의 차이가 발생했다!

 

커넥션풀 등의 다른 문제까지 생각한다면 이 차이는 더욱 극명하게 벌어질거라는 생각이 든다.

 

좋은 코드의 중요성을 다시 한 번 느낀다!

댓글
공지사항
최근에 올라온 글
최근에 달린 댓글
Total
Today
Yesterday
링크
«   2024/05   »
1 2 3 4
5 6 7 8 9 10 11
12 13 14 15 16 17 18
19 20 21 22 23 24 25
26 27 28 29 30 31
글 보관함