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.

Sign up for GitHub

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

Jump to bottom

클래스 다이어그램 #2

Open
bohyunkang opened this issue Sep 18, 2022 · 1 comment
Open

클래스 다이어그램 #2

bohyunkang opened this issue Sep 18, 2022 · 1 comment
Assignees
@bohyunkang
Labels
documentation Improvements or additions to documentation

Comments

@bohyunkang
Copy link
Collaborator

bohyunkang commented Sep 18, 2022 •
edited
Loading

KOTD : Kpop of the day 🎶🎤

클래스 다이어그램 작성 (9/18)

image

클래스 다이어그램 추가 작성 (9/19)

KOTD
@bohyunkang bohyunkang added the documentation Improvements or additions to documentation label Sep 19, 2022
@bohyunkang bohyunkang self-assigned this Sep 19, 2022
@wholemann
Copy link

DataService는 굉장히 목적이 모호한 이름입니다. 존재이유가 전혀 드러나지 않고 있죠.

Service가 뭔지 이해를 하셔야 해요. 비즈니스 로직의 진입점입니다.

그러면 PlaylistService라던지 뭔가 의도가 명확히 드러나야죠.

그러면 PlayLists가 별도의 객체로 있을 필요도 없어요. 왜 KpopOfTheday가 PlayLists에 의존하나요?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bohyunkang bohyunkang

Labels
documentation Improvements or additions to documentation
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@wholemann @bohyunkang