Skip to content

Conversation

@shail1027
Copy link
Member

📌 몇 주차 워크북인가요?

  • 예: Week01

✨ 이번 주에 작업한 내용

  • 구현/수정한 기능 요약
    • 예: 회원가입 API 구현
    • 예: UI 레이아웃 정리

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

  • 예: 비밀번호 유효성 검사 로직이 적절한지 확인 부탁드립니다.
  • 예: Controller 단의 코드 구조 피드백 원합니다.

✅ 체크리스트

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

@shail1027 shail1027 changed the title [Week03 [Week03] 제타/이예{빈 미션제출 Oct 5, 2025
@shail1027 shail1027 changed the title [Week03] 제타/이예{빈 미션제출 [Week03] 제타/이예빈 미션제출 Oct 5, 2025
@eldeoddt
Copy link

eldeoddt commented Dec 1, 2025

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

<MainActivity>
👍 ViewPager2 초기화 및 orientation 설정 완료
👍 미니플레이어 클릭 시 SongActivity Intent 전달
💙 미니플레이어 클릭 리스너 3개 중복 (homeMiniPlayer, Title, Singer) → 하나로 통합 권장
부모 뷰(homeMiniPlayer)에만 클릭 리스너 달면 자식 뷰 클릭도 처리됨

<SongActivity>
👍 Intent로 title, singer 받아서 UI 반영 완료
👍 finish() 호출로 뒤로가기 정상 동작

<AlbumFragment>
👍 AlbumVPAdapter 연결 완료
👍 ViewPager2(albumContentTp) adapter 설정
💙 TabLayout 연결

<AlbumVPAdapter>
👍 3개 Fragment(Song/Detail/Video) 구조 명확
👍 getItemCount(), createFragment() 구현 정상

<LockerFragment>
👍 LockerVPAdapter 연결 완료
👍 ViewPager2 adapter 설정 정상
💙 TabLayout 연결

<PannelVPAdapter>
👍 동적으로 Fragment 추가하는 구조 (addFragment) 구현 좋음
👍 notifyItemInserted() 호출로 데이터 변경 알림 처리

<HomeFragment>
👍 PannelVPAdapter 사용하여 ViewPager2 구성
💙 MainActivityBinding 사용 중 → HomeFragment 전용 layout이 필요하면 별도 binding 생성 권장
💙 현재 MainActivity의 레이아웃을 재사용하는 구조 → Fragment용 레이아웃 분리 시 확장성 증가

<Fragment 구조 (Song, Detail, Video, SavedSong, MusicFile)>
👍 각 Fragment ViewBinding 적용 완료
👍 기본 레이아웃 inflate 처리 정상

<리소스 관리 구조>
👍 파일명 naming convention 통일 (song_activity.xml, fragment_*.xml)
👍 colors.xml에 커스텀 색상 정의 good
💙 string.xml에도 반복 텍스트 리소스화 권장
💙 실무 권장: 기능별 패키지 분리 (home, album, locker, song)
home, album, song 등등..

com.example.앱이름/
├── presentation/
│   ├── home/
│   │   ├── view/
│   │   │   ├── HomeFragment.kt
│   │   │   └── HomeActivity.kt
│   │   ├── viewmodel/
│   │   │   └── HomeViewModel.kt
│   │   └── adapter/
│   │       └── HomeAdapter.kt
│   ├── album/
│   │   ├── view/
│   │   ├── viewmodel/
│   │   └── adapter/
│   ├── auth/
│   └── base/
│       ├── BaseActivity.kt
│       └── BaseFragment.kt
│
├── data/
│   ├── repository/
│   │   ├── AlbumRepository.kt
│   │   └── AuthRepository.kt
│   ├── local/
│   │   ├── dao/
│   │   ├── entity/
│   │   └── AppDatabase.kt
│   └── remote/
│       ├── api/
│       └── dto/
│
├── model/
│   ├── Album.kt
│   └── User.kt
│
└── util/

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