[JDBC 라이브러리 구현하기 - 2단계] 제이미(임정수) 미션 제출합니다. #432
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
안녕하세요 그레이!
빠르게 리뷰해 주셨는데 프로젝트부터 하다 보니 지금이네요... 😅
늦게나마 2단계 리뷰 요청드립니다.
또한, 지난 1단계에서 좋은 리뷰들 감사합니다!
대부분 변명이긴 하지만, 해당 pr에 답변 달아두었습니다. (1단계 pr 바로가기)
2단계를 보지 않고 1단계를 진행했는데, 알고 보니 2단계였던 내용들이 많네요.
그래서 이번 2단계에서는 지난번 그레이가 남겨주신 피드백과 중복 코드를 제거하는 방식에 대해 고민해 보았습니다.
qeuryForObject()
에서 결과가 한 개가 아닌 경우에 대한 예외처리를 진행했으며,try-with-resources 중복을 제거하는 방법에 대해 많은 분들이 콜백 패턴을 적용해 보았습니다.
그런데, 해당 패턴을 처음 학습해보기도 하고 익숙하지 않아 제가 제대로 적용한 것인지 잘 모르겠네요..!
그 외에도 좀 더 고민해 볼 만한 부분 혹은 수정했으면 하는 부분들이 있다면 말씀해 주시면 감사하겠습니다.
이번 리뷰도 잘 부탁드립니다 😊