-
Notifications
You must be signed in to change notification settings - Fork 4
[Week03] 엠키/문현우 미션 제출 #14
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: mky03
Are you sure you want to change the base?
Conversation
eldeoddt
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
3주차 피드백 완료 ✅
수고하셨습니다 🙂 👍
🙋 리뷰 요청 / 확인 받고 싶은 부분
AlbumFragment → DetailFragment 데이터 전달 이슈
- 원인: TextView.toString 은 객체 주소 반환 (text.toString 사용 필요)
- -> ViewPager2 Adapter의 createFragment() 호출 시점에 bundle이 전달되지 않음
- 학습용: Adapter 생성자에 데이터 전달 후 createFragment()에서 bundle 설정
- 권장: ViewModel(activityViewModels) 사용 → fragment 생명주기 독립적이고 configuration 변경 시에도 데이터 유지
<AlbumFragment>
👍 TabLayout + ViewPager2 구조 완성
👍 HomeFragment에서 Bundle로 데이터 수신 정상 동작
💙 binding.albumSingerNameTv.toString() → binding.albumSingerNameTv.text.toString() 수정 필요
💙 Adapter에 생성자 파라미터 추가해서 데이터 전달 or ViewModel 사용 권장
<AlbumVPAdapter>
👍 3개 Fragment(Song/Detail/Video) 구조 명확
💙 createFragment() 내부에서 DetailFragment에 Bundle 설정 필요
1 -> DetailFragment().apply {
arguments = Bundle().apply {
putString("singerName", singerName) //생성자로 받은값
putString("albumTitle", albumTitle)
}
}<DetailFragment>
👍 arguments에서 데이터 수신 로직 구현
👍 Log로 디버깅 시도
💙 받은 데이터가 null일 경우 기본값 설정 권장 (?: "default")
<LockerFragment>
👍 TabLayout + ViewPager2 구조 완성
💙 3개 탭 Fragment(Savedsong/Songfile/Savedalbum) 실제 데이터 연동 시 ViewModel 활용 고려
<SongActivity>
👍 Song 데이터 클래스 활용한 Intent 전달 구조 좋음
👍 재생/반복/랜덤 상태 토글 로직 구현
💙 MainActivity로 돌아갈 때 finish() 추가로 액티비티 스택 정리 권장
<MainActivity>
👍 BottomNavigationView Fragment 전환 구조 명확
|
확인했습니다! 감사합니당 |
📌 몇 주차 워크북인가요?
✨ 이번 주에 작업한 내용
🙋 리뷰 요청/확인 받고 싶은 부분
✅ 체크리스트
weekN/폴더 안에 과제 정리 완료[WeekN] 닉네임/이름 미션 제출규칙 맞게 작성