Skip to content

Conversation

@codedbyminjae
Copy link

📌 몇 주차 워크북인가요?

  • Week03

✨ 이번 주에 작업한 내용

  • BottomNavTestActivity 구현 및 activity_bottom_nav_test.xml 연결
  • flo 메인 레이아웃(activity_main.xml) 구성 및 수정
  • Fragment (Home, Diary, Calendar, Friend, Mypage) 생성 및 전환 로직 구현
  • ViewBinding 적용 및 Fragment 생명주기 관리
  • flo 관련 XML 리소스(flo_nav_graph, flo_bottom_menu 등) 수정/추가
  • AndroidManifest.xml 수정 (BottomNavTestActivity를 Launcher로 설정)

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

  • Fragment 전환 로직이 올바르게 구성되었는지 확인 요청
  • Manifest에서 Launcher Activity 변경 방식이 적절한지 피드백 요청
  • flo 스타일에 맞는 리소스/네이밍 구조가 일관성 있는지 검토 부탁드립니다 . (패키지를 통해서 구분하는게 괜찮을지, 이제 xml과 프래그먼트가 많아져서 관리를 어떻게 하는지 효율적일지 궁금해서 입니다.)

✅ 체크리스트

  • week03/ 폴더 안에 과제 정리 완료
  • PR 생성 시 base = 조직 내 본인 브랜치, compare = 내 Fork main 브랜치로 설정 확인
  • PR 제목에 [Week03] 제이/김민재 미션 제출 규칙 맞게 작성

@eldeoddt
Copy link

3주차 피드백 완료 ✅
수고하셨습니다 🙂 👍

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

Fragment 전환 로직 및 리소스 관리 구조

  • Fragment 전환: BottomNavTestActivity의 setOnItemSelectedListener 구현
  • Manifest Launcher 변경: BottomNavTestActivity를 진입점 설정 적절
    실무에선 SplashActivity를 LAUNCHER로 설정 후 초기화 작업 처리
    Splash → MainActivity 흐름이 일반적임 (로그인 체크, 데이터 로딩 등)
  • 리소스 현재 구조로도 충분함 👍

<BottomNavTestActivity>
👍 setOnItemSelectedListener 구현 깔끔
👍 초기 Fragment(HomeFragment) 설정 정상
💙 transaction 변수 재사용으로 코드 간결화 좋음
💙 각 Fragment 생성 시 when 분기 명확

<MainActivity (flo)>
👍 미니플레이어 클릭 → SongActivity Intent 전달 구조
👍 initBottomNavigation() 함수 분리로 가독성 향상
💙 song 데이터 클래스 활용 좋음
💙 Fragment 교체 로직 when 분기 일관성 있음

<AlbumFragment>
👍 ViewBinding 적용 및 뒤로가기 버튼 구현
👍 각 곡 클릭 시 Toast 메시지 표시
💙 실제 데이터 연동 시 ViewModel + LiveData 구조 권장

<Fragment 구조 (Home, Diary, Calendar, Friend, Mypage)>
👍 각 Fragment 분리 완료
👍 일부는 간단한 TextView, 일부는 inflate 방식 혼용
💙 FriendFragment, DiaryFragment, CalendarFragment는 TextView로만 구성 → 추후 실제 레이아웃 적용 필요
💙 HomeFragment는 onCreateView에서 inflate 처리 정상

<SongActivity>
👍 Intent로 title, singer 전달받아 UI 반영
👍 setPlayerStatus() 함수로 재생/일시정지 상태 관리
💙 finish() 호출로 Activity 종료 처리 완료
💙 재생 상태를 ViewModel로 관리하면 화면 회전 시에도 유지 가능

<리소스 관리 구조>
👍 패키지 분리 (flo, bottomnav) + 파일명 prefix (flo_, fragment_)
👍 naming convention 통일 (Fragment, Activity, Layout 모두 일관성 있음)
💙 현재 구조 그대로 유지하면 됨
💙 drawable, string 리소스도 prefix 통일 권장

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