🚀 4단계 - GitHub(인기 저장소)

[SeongHoonC]

안녕하세요 지훈님! 4단계 구현 완료했습니다..!

선택 요구사항을 하기 전까지는 고민이 없었는데요.
인기 저장소가 50 이상이라는 조건이 이 화면에만 해당되지 않을 것 같아서 domain 으로 분리하였습니다.

• Screen(View)이 Domain 을 모르게 분리된 UiModel 을 따로 만드는게 좋을지.
• Screen 에서 Domain Model 을 그대로 사용할지.

저는 뷰 로직이 복잡해지는게 아니면 도메인 모델을 그대로 사용하는 편인데요.
혹시 선호하시는 방향이 있다면 궁금합니다.

마지막 단계 리뷰 잘부탁드립니다!!

[lee-ji-hoon]

안녕하세요 성훈님

4단계를 요구사항들을 너무 잘해오셔서 미션에 대한 피드백을 드릴게 애매하지만 1~3단계를 매우 잘 해오셨기에 특별한 수정이 없이 4단계의 구현이 마무리가 될 수 있던 것이라고 생각을 해요.

그래도 다른 부분에 대해서 고민해 볼 만한 포인트와 성훈님이 질문을 남겨주신 부분에 대해서 답변을 달아뒀습니다, 해당 부분만 조금 고민을 해보시고 성훈님이 다시 리뷰요청을 주시면 그때는 미션을 마무리 하도록 하겠습니다!

[lee-ji-hoon]

안녕하세요 성훈님 벌써 마지막 과제까지 끝이 났네요.

Github 미션 진행 간 너무 잘해주셔서, 미션 외적으로 고민하실만한 거리를 한 두 개씩 드렸는데 바로바로 잘 캐치해 주셔서 좋았습니다.

앞으로의 개발을 하시면서 지금 배웠던 테스트 위주의 관점들이 좋은 영향이 되셨으면 좋겠네요.

고생하셨습니다!