Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[2단계 - JDBC 라이브러리 구현하기] 페드로(류형욱) 미션 제출합니다. #763

Merged
merged 10 commits into from
Oct 11, 2024

Conversation

hw0603
Copy link
Member

@hw0603 hw0603 commented Oct 10, 2024

안녕하세요 우주! 오랜만에 다시 요청드리는 것 같네요😁

이번 단계에서는 RowMapper를 인터페이스로 추출하고, PreparedStatementSetter를 도입하여 DAO단에서는 쿼리 문자열을 생성하고 파라미터의 순서에 맞게 메서드 인자 순서를 맞춰 호출만 하면 동작할 수 있도록 리팩토링 해 봤어요.

사실 학습 테스트의 양이 더 많네요ㅎㅎ 이번 리뷰도 잘 부탁드려요!

@hw0603 hw0603 self-assigned this Oct 10, 2024
Copy link

@tsulocalize tsulocalize left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

안녕하세요, 페드로!
이번 단계는 이전 단계에서 이미 리팩터링하기도 했고, 학습 테스트도 주석 남겨주신 걸 보니 열심히 공부하신 것 같아 Approve하겠습니다.
다음 단계에서 만나요~

@tsulocalize tsulocalize merged commit ed7f467 into woowacourse:hw0603 Oct 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants