Skip to content

step 2 DI 컨테이너 구현하기(힌트) #52

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

Open
wants to merge 1 commit into
base: newvelop
Choose a base branch
from

Conversation

newvelop
Copy link

step1에서 recursive하게 inject하는 부분이 구현되어있어서 step1의 피드백을 적용해서 step2 pr을 열었는데요,

혹시 추가적으로 피드백 주실부분있으면 주시면감사하겠습니다~

@kang-hyungu
Copy link
Contributor

step1에 inject 기능이 미리 구현되어 있진 않을텐데... java-jdbc에 구현된 코드를 적용하셨다는 말씀이실까요?
현재 pr에서는 생성자가 하나만 있는 상태를 가정하고 구현하신걸로 보이네요
TODO 주석에 작성하신 것처럼 인터페이스나 필드 주입 등 다른 케이스도 커버할 수 있도록 수정해보시고 테스트 코드로 의도한대로 동작하는지 검증해보시는 걸 추천드립니다.

Copy link
Contributor

@kang-hyungu kang-hyungu left a comment

Choose a reason for hiding this comment

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

안녕하세요 현오님
리뷰 내용은 pr 코멘트로 남겨두었습니다.
수정하시고 리뷰 재요청주세요~

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