경북대 Android_류지원_1주차_과제

[akuby21]

1단계 코드

a1d06c7 까지의 커밋이 1단계 완성 코드입니다!

코드 작성하며 어려웠던 점

• 메서드 분리의 기준을 잡는 것이 어려웠습니다.
• 모델(연락처)의 액티비티간에 공유 방법을 선택하는 것이 어려웠습니다. Intent를 이용하는 방법 중에서도 객체 자체를 보낼 수 있고, id 값을 보내는 방법 등 다양한 방법이 있기에 오히려 더 고민이 되었습니다.
• 성능 향상을 위해 UI에 들어가는 이미지를 모두 drawable로 구현하였는데 익숙하지 않아 시간이 다소 걸렸습니다.

중점적으로 리뷰해주셨으면 하는 부분

• Intent를 활용한 데이터 전달 방식
  • 저는 객체 자체를 주고받기 보다는 repository 내의 해당 객체의 인덱스를 주고 받는 식으로 하였습니다.
• dimen, string 등 resource 활용
  • style, dimen, string등 다양한 resource를 활용하여 이번 과제를 진행하였습니다. 이러한 값들의 활용 방식이 올바른지 궁금합니다. (ex) 이런 곳은 하드코딩이 오히려 낫다.. 등등)
• 메서드가 올바르게 분리 되었는지 궁금합니다
  • 하나의 메서드가 하나의 역할만 하도록 나름 분리를 하였는데, 너무 많이 한 것인지 아니면 더 분리 하는 것이 좋은지 궁금합니다.
• 라이브 클론 코딩 강의에서 미리 앞서가지 말라는 강사님의 조언이 있어서 MVVM 패턴, Room, LiveData(or DataStore) 등 사용 해봄직한 기술은 사용하지 않았음을 밝힙니다!

실행 화면

연락처 생성 화면

연락처 목록 화면

연락처 상세 화면

(성별이 MALE, FEMALE로 나오는 현상은 이후 한국어로 나오도록 수정하였습니다(#9682493)

[akuby21]

감사합니다! 수정 사항 반영하여 다시 커밋 올렸습니다.
말씀해주신 점들(캡슐화, 뷰 계층)에 대해 생각해보고 리팩토링을 진행하며 더 공부가 되었던 것 같습니다!
바쁘실텐데도 세심한 리뷰 다시 한 번 감사드립니다 :)