Skip to content
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

뮤직 세션 레이어 분리 #173

Merged
merged 5 commits into from
Dec 8, 2022
Merged

Conversation

radiantchoi
Copy link
Collaborator

12/7 #90 #142 #156

작업한 내용

MusicSession 객체를 클린 아키텍처를 지향하여 분리하였습니다.

  • Repository에서 재생 요청을 보내게 됩니다.
  • 음악 플레이어는 뷰와 무관한 객체고, 직접 기기의 미디어 재생 수단과 연결되기 때문에 이렇게 하는 것이 가능합니다.

덤 : 서버와 일기 리스트 뷰를 연결했습니다.

  • 음악 재생을 테스트하기 위해 구조를 뜯다가 수행했습니다.

덤 : 서버와 일기 상세 뷰를 연결했습니다.

  • 일기 목록 URL에 쿼리만 붙여주면 되더군요..?

고민한 점 및 어려웠던 점

MusicContentsView의 재생/일시정지 상태에 따라 UI를 업데이트해야 할 듯 합니다.

  • 이를 위해 뷰 모델에 재생중이냐 아니냐를 나타내는 프로퍼티는 있어야 할 것 같습니다.

한곡반복은 어떻게 해야 할지 고민했습니다.

  • 일기를 보는 동안은 무한 반복이 돼야 하는데, 그것은 아직 구현하지 못했습니다.

Copy link
Collaborator

@rudah7 rudah7 left a comment

Choose a reason for hiding this comment

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

직접 실행해보진 못했지만, 코드상으로 봤을 땐 문제 없어보입니다!

수고하셨습니다 :)

Copy link
Member

@LEEYOONJONG LEEYOONJONG left a comment

Choose a reason for hiding this comment

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

세션들 적절하게 분리해주셨네요. 역시 실행해보고 싶습니다

@radiantchoi radiantchoi merged commit b24456b into develop Dec 8, 2022
@radiantchoi radiantchoi deleted the feature/clean-music-player branch December 8, 2022 04:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants