티스토리 뷰
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
링크
TAG
- IDE
- 배포
- RequestBody
- 도커
- 루나빔
- vim
- 레디스
- RequestPart
- ModelAttribute
- JavaScript
- Dap
- 아키텍처
- lunarvim
- neovim
- RequestParam
일 | 월 | 화 | 수 | 목 | 금 | 토 |
---|---|---|---|---|---|---|
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 |
글 보관함