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

[JDBC 라이브러리 구현하기 - 3,4단계] 애쉬(정설희) 미션 제출합니다. #589

Merged
merged 5 commits into from
Oct 11, 2023

Conversation

xxeol2
Copy link

@xxeol2 xxeol2 commented Oct 9, 2023

안녕하세요 로건! 🎈
미션의 마지막 날이네요..! 연휴동안 이것저것 할 일들이 많아서, 미션 제출이 많이 늦어졌네요.. 죄송합니다 🥲
이번 3,4단계에서는 미션의 요구사항을 지키는데 집중해봤어요!
아래는 지난 미션에 남겨주신 코멘트에 대한 답변입니다..!

템플릿 콜백 패턴을 적용해보니 어떠셨나요 !?
저는 적용하지 않았는데요. 이유는 JdbcTemplate에서는 템플릿 콜백 패턴이 적용되어 있는 계기가 코드 규모가 커졌기 나중에 사용하게 된게 아닐까..??라는 생각이 많이 들었어요.
이전 미션에서는 어댑터 패턴을 사용한 이유가 명확했는데, 지금 미션 규모에서 템플릿 콜백 패턴을 적용하는건 배보다 배꼽이 더 큰게 아닐까라는 생각이 들더라구요. 그런데 애쉬 코드를 보니까 완전 깔끔해져서 제 코드도 바꾸는게 좋다는 생각이 드네요 ㅋㅋ 😀
실제로 적용해본 애쉬의 후기가 궁금해요!

템플릿 콜백 패턴을 활용해보니 확실히 코드의 관심사를 분리할 수 있었어요!
사용하는 측에서는 콜백 메서드를 구현하는데 집중할 수 있다는것이 가장 큰 장점인 것 같아요!
이번 3,4단계에서도 TransactionManager와 TransactionExecutor를 템플릿 콜백 패턴을 활용해 구현해봤어요!

감사합니다 🙇🏻‍♀️

@xxeol2 xxeol2 changed the base branch from main to xxeol2 October 9, 2023 11:22
Copy link
Member

@70825 70825 left a comment

Choose a reason for hiding this comment

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

안녕하세요 애쉬 !

코드 완전 깔끔하게 작성하셔서 이해하기 편했습니다 🥳

궁금한 부분 코멘트 남겼는데 확인해주세요~

@xxeol2
Copy link
Author

xxeol2 commented Oct 11, 2023

안녕하세요 로건!!
리뷰 반영 완료했습니다 👍 🙇🏻‍♀️

Copy link
Member

@70825 70825 left a comment

Choose a reason for hiding this comment

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

안녕하세요 애쉬~
미션하느라 고생 많으셨어요~!!
다음 미션도 화이팅입니다!!! 🎉🥳🎉🥳

@70825 70825 merged commit b694b1e into woowacourse:xxeol2 Oct 11, 2023
1 check failed
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