🚀 4단계 - 페이먼츠(카드 수정)

[ethanchaee]

안녕하세요 윤혁님 리뷰 잘부탁드립니다 :)

구현 사항

• 코드리뷰 반영
• 수정기능

---

기존에 클릭 이벤트를 composable 에서 처리했는데, viewModel 과 Activity 에서 처리하게 변경했습니다.
composable 은 UI 를 그리기 위함이지 로직 처리를 하는 곳은 아닌데, UI 를 그리는 것과 상관 없는, 토스트 노출이나 내비게이션 동작은
Activity 에서 처리했습니다.

추가/수정을 판가름하기 위해 최초에 받아온 Card 정보를 처리하는 부분도, rember 를 이용했습니다.
보통 프로젝트에서 DI 를 사용해서 viewModel 에 해당 값을 Injection 해서 사용했었는데, 이번처럼 composable 에 영향을 줄 수 있는 곳에 사용하다 보니 해당 키워드를 더 잘 이해하게 되어 배운점이 많은 과제였습니다 :)

[malibinYun]

4단계 미션 고생 많으셨어요!
여러 코멘트 남겨두었으니 확인부탁드려요 :)
미션을 이만 마무리 짓고 싶으시다면 코멘트나 메시지 남겨주시고, 더 시도하고 싶으신 것이 있으시다면 반영 후 다시 리뷰요청 부탁드려요!

[malibinYun]

카드 수정하기 Screen과 새로 만드는 Screen 컴포넌트는 완전히 동일하니,
기존 만들어 둔 Stateless Screen 컴포넌트를 수정, 추가 액티비티에서 활용하는 형태로 구성해보는 건 어떨까요?
Stateless Screen 컴포넌트를 각각 가지는 Stateful Screen이 두 개 만들어질 수도 있겠어요.
그렇다면 각 stateful Screen을 테스트할 때도 용이할 거라 생각해요!

[malibinYun]

저는 Preview이름을 지을 때, 컴포넌트Preview_그컴포넌트의특징 이런 형태로 짓는 편이에요.

ex) CardListPreview_ManyCards()

Preview렌더링 된 것을 보더라도 어떤 특징을 가지는지 알기 어려울 때, Preview의 함수명에 좀 더 그 특징을 묘사해서 조금이라도 Preview를 쉽게 파악할 수 있게 도와주려고 이렇게 작성하는 편이에요 :)

[malibinYun]

카드 컴포넌트 자체에 특정 기능이 들어가면 재활용이 어렵다고 생각해요.
클릭할 수 있는 리스너를 외부에서 받거나, Modifier를 외부에서 받도록 구성해보는 건 어떨까요?

[malibinYun]

이 컴포넌트들을 파일 분리한다면 테스트도 컴포넌트별로 국소적으로 구성해볼 수 있겠어요!

[malibinYun]

전부 컴포넌트화 하는 것이 쉬운 일이 아니셨을 텐데 모두 반영해주시다니 멋지십니다 👍

[malibinYun]

이부분은 지난 리뷰에서 놓쳤네요!
이부분도, 특정 클릭 이벤트에 종속적이 되니, 클릭 이벤트는 외부에서 주입 받도록 하는 것은 어떨까요?

[malibinYun]

여기 context는 실수로 추가된 걸까요 ?_?

[malibinYun]

어떤 포매팅을 위한 interface - 구현체를 만든뒤, Card 컴포넌트의 파라미터에는 interface를 받고, 디폴트 파라미터로 구현체를 받는 형태로 의존관계를 느슨하게 구성해보는 건 어떨까요?