Skip to content

Conversation

@codedbyminjae
Copy link

📌 몇 주차 워크북인가요?

  • Week02

✨ 이번 주에 작업한 내용

  • Nike UI 기반 BottomNavigationView 실습
    • activity_bottom_nav_test.xml에서 FragmentContainerView, BottomNavigationView 레이아웃 구성
    • BottomNavTestActivity.kt에서 프래그먼트 전환 로직 구현 (FragmentManager 방식)
    • HomeFragment, DiaryFragment, CalendarFragment, FriendFragment, MypageFragment 생성 및 연결
    • 각 프래그먼트별 기본 레이아웃 작성 및 ViewBinding 적용
    • bottom_nav_menu.xml 메뉴 아이콘/타이틀 구성

🙋 리뷰 요청/확인 받고 싶은 부분

  • FragmentManager를 사용한 전환 방식이 올바르게 구현되었는지 확인 부탁드립니다.
  • UI 구성에서 레이아웃/네이밍 구조가 직관적인지 피드백 주시면 감사하겠습니다.

✅ 체크리스트

  • weekN/ 폴더 안에 과제 정리 완료
  • PR 생성 시 base = 조직 내 본인 브랜치, compare = 내 Fork main 브랜치로 설정했는지 확인
  • PR 제목에 [WeekN] 닉네임/이름 미션 제출 규칙 맞게 작성

@eldeoddt
Copy link

2주차 제이 미션 피드백 완료 ✅
수고하셨습니다 🙂 👍

🙋 리뷰 요청 / 확인 받고 싶은 부분

  • FragmentManager 전환 방식
    프래그먼트 전환 로직 깔끔함
    상태 복원이 필요한 화면인 경우 add() + show()/hide() 방식 전환 사용 고려

  • 레이아웃 / 네이밍 구조
    activity_bottom_nav_test.xml 구조 명확, main_fragment_container, main_bnv 네이밍 일관적. id prefix(bottom_nav_*) 통일 시 확장 시 가독성 향상

<BottomNavTestActivity>

👍

  • ViewBinding 적용으로 코드 간결
  • FragmentManager를 이용한 기본 전환 로직 명확

💙 개선 제안

  • 프래그먼트 객체를 매번 새로 생성 -> supportFragmentManager.findFragmentByTag()로 재사용 구조 고려
  • setOnItemSelectedListenertrue 반환 이유 주석 추가 시 가독성 개선

<HomeFragment - MypageFragment>

👍

  • 구조 일관적, placeholder UI 구성 적절
  • 각 Fragment 최소 단위 기능 분리 good

💙 개선 제안

  • 공통 텍스트만 표시되는 구조이므로, BaseFragment 추출해 중복 코드 감소 가능

<전반>

👍

  • BottomNavigationView 실습 good
  • FragmentContainerView + 메뉴 구성

💙 개선 제안

  • Activity와 Fragment 간 데이터 공유 시 ViewModel 활용 고려
  • 아이콘 및 메뉴 리소스 네이밍 prefix 통일(ic_nav_home, menu_nav_home) 권장

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