Skip to content

[5주차/Leo] 워크북 제출합니다.#16

Merged
yongyong213 merged 2 commits into
UMC-Inha:Leo/mainfrom
yongyong213:main
Nov 4, 2025
Merged

[5주차/Leo] 워크북 제출합니다.#16
yongyong213 merged 2 commits into
UMC-Inha:Leo/mainfrom
yongyong213:main

Conversation

@yongyong213

@yongyong213 yongyong213 commented Oct 28, 2025

Copy link
Copy Markdown

✅ 워크북 체크리스트

  • 모든 핵심 키워드 정리를 마쳤나요?
  • 핵심 키워드에 대해 완벽히 이해하셨나요?
  • 이론 학습 이후 직접 실습을 해보는 시간을 가졌나요?
  • 미션을 수행하셨나요?
  • 미션을 기록하셨나요?

✅ 컨벤션 체크리스트

  • 디렉토리 구조 컨벤션을 잘 지켰나요?
  • pr 제목을 컨벤션에 맞게 작성하였나요?
  • pr에 해당되는 이슈를 연결하였나요?(중요)
  • 적절한 라벨을 설정하였나요?
  • 파트장에게 code review를 요청하기 위해 reviewer를 등록하였나요?
  • 닉네임/main 브랜치의 최신 상태를 반영하고 있는지 확인했나요?(매우 중요!)

📌 주안점

@yongyong213 yongyong213 requested a review from jiyoung02 October 28, 2025 12:16
@yongyong213 yongyong213 linked an issue Oct 29, 2025 that may be closed by this pull request

@jiyoung02 jiyoung02 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM! 수고하셨습니다 😊
음악 재생 기능 구현이 정말 깔끔하게 잘 되어 있습니다!
아래는 함께 개선해보면 완성도를 더 높일 수 있는 부분들이니 참고해주세요

  1. 스플래시 화면 레이아웃
    현재 기본 제공 스플래시 화면을 사용하고 있어, 로고가 깨져 보이는 현상이 있습니다.
    기본 스플래시도 간단한 경우에는 충분하지만, 커스텀에 한계가 있어 의도한 디자인과 다르게 표시될 수 있습니다.
    이를 방지하기 위해 별도의 스플래시 액티비티를 구현해 완전히 커스텀된 레이아웃을 사용하는 것을 추천드립니다. AndroidManifest.xml에서 해당 액티비티를 기본 액티비티로 지정하고, 스플래시 액티비티 내에서 Handler(Looper.getMainLooper())를 사용해 일정 시간이 지난 뒤 메인 액티비티로 전환하면 자연스럽게 구현할 수 있습니다.

  2. RecyclerView 클릭 리스너 구조
    현재는 Fragment가 어댑터의 인터페이스를 직접 상속받아 클릭 이벤트를 처리하는 방식으로 구현되어있는데, Fragment의 책임이 커지고, 어댑터와의 결합도가 높아지는 문제가 있습니다.
    아래 두가지 구조가 결합도를 낮추고, 구조를 단순화 할 수 있어 권장되니 참고해주세요

    • 람다 콜백 방식: Fragment에서 어댑터 생성 시 클릭 이벤트를 람다로 전달

    • 생성자 주입 방식: 어댑터 생성 시 인터페이스 구현체를 함께 주입

추가적으로 미션 제출시에는 미션 태그와 Assignees에 본인 할당하는걸 잊지 말아주세요!

@yongyong213 yongyong213 merged commit 120e1f0 into UMC-Inha:Leo/main Nov 4, 2025
@yongyong213 yongyong213 self-assigned this Nov 4, 2025
@yongyong213 yongyong213 added the mission mission label Nov 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Chapter 5. Thread & Coroutine

2 participants