Skip to content

7주차 미션_담담#22

Open
sungs25 wants to merge 10 commits into
mainfrom
feature/new-task
Open

7주차 미션_담담#22
sungs25 wants to merge 10 commits into
mainfrom
feature/new-task

Conversation

@sungs25

@sungs25 sungs25 commented May 15, 2026

Copy link
Copy Markdown
Collaborator

📝 작업 내용

  • 기존 XML/Fragment 기반 UI를 Jetpack Compose로 전면 마이그레이션
  • Material3 + Type-Safe Navigation 도입
  • 화면 5개 + 프로필 수정 화면을 Composable로 재구현
  • BottomNavigationView를 Row 기반 커스텀 BottomBar Composable로 대체
  • 장바구니 화면의 '주문하기' 버튼 클릭 시 구매하기 탭으로 전환되도록 State Hoisting 패턴으로 구현
  • 사용하지 않게 된 Fragment 6개, layout XML 7개, menu/color 리소스, ViewBinding 설정 제거

📸 스크린샷

2주차 화면 그대로


🙏 리뷰 요구사항 (선택)

  • 화면 Composable의 콜백 파라미터 네이밍이나 State Hoisting 분리 단위가 적절한지 의견 부탁드립니다.

@kimdoyeon1234 kimdoyeon1234 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

수고하셨습니다!
Type-safe Navigation 올바르게 적용하셨고, 파일 분리도 완벽하게 되어있어요!
콜백 네이밍(onOrderClick, onEditClick, onTabClick)도 동사+목적어로 일관성 있게 잘 지키고 있어서 좋았습니다!

지금도 좋지만 수정하면 좋을점 남겨두었습니다!

  1. ShopScreen 내부의 selectedTab 상태가 고립되어 있어요!State Hoisting을 적용해서 상위로 끌어올리면 좋을거 같아요!

  2. State Hoisting
    fun ShopScreen(
    selectedTab: Int,
    onTabSelected: (Int) -> Unit,
    modifier: Modifier = Modifier
    )

전반적으로 이번 주차 코드 중 가장 완성도가 높아요! 다음 주차도 화이팅 !

Comment thread damdam/UMC_Work_7th/app/src/main/java/com/sungs/myapplication/ui/MainScreen.kt Outdated
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