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

Step3 패키지 쪼개기 #40

Merged
merged 11 commits into from
Feb 23, 2020
Merged

Step3 패키지 쪼개기 #40

merged 11 commits into from
Feb 23, 2020

Conversation

isisrin
Copy link

@isisrin isisrin commented Feb 22, 2020

패키지 쪼개봤습니다 :)

Copy link

@serverwizard serverwizard left a comment

Choose a reason for hiding this comment

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

3단계 미션인 기능 우선 패키지 구성하기 를 진행하시느라 고생하셨습니다~👍

생각해보면 좋을만한 코멘트를 남겼습니다.

한번 더 생각해보시고 재요청주시면 merge하도록 할게요! 😄


import kitchenpos.model.TableGroup;
import kitchenpos.hall.model.TableGroup;

import java.util.List;
import java.util.Optional;

Choose a reason for hiding this comment

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

아키텍처를 설계할 때 등장하는 영역은 크게 4가지로 볼 수 있어요. 
- 표현(presentation) 
- 응용(application)
- 도메인(domain)
- 인프라스트럭처(infrastructure): 실제 구현을 담당, **구현 기술**과 아주 밀접한 연관이 있는 레이어

위를 참고했을 때, 혜린 님께서 나눠주신 controller 는 presentation layer, bo는 application layer, model은 domain layer, dao는 infrastructure를 나타낸다고 볼 수 있을 것 같은데요.

이런 관점에서 dao에 있는 아래 인터페이스는 특정 구현기술과 밀접한 연관이 없는데도 dao 에 들어 간 것 같아요.

image

해당 인터페이스를 어디에 두면 좋을지 고민해보면 좋을 것 같아요.

만약 이름의 끝에 Dao가 붙어있는데, dao 패키지에 두지 않는 것이 이상하다면 이름을 Repository라고 변경해보는 건 어떨까요?

image

@isisrin
Copy link
Author

isisrin commented Feb 23, 2020

변명같지만.. ㅠㅠ 2주차라서 2주차 내용만 생각하고 분리했습니다.. 하하핳

피드백 감사합니당 :>

Copy link

@serverwizard serverwizard left a comment

Choose a reason for hiding this comment

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

model 패키지와 repository 패키지를, domain 패키지를 만들어 그 곳에 두는 건 어떨까요?

@serverwizard serverwizard reopened this Feb 23, 2020
Copy link

@serverwizard serverwizard left a comment

Choose a reason for hiding this comment

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

3단계 미션인 기능 우선 패키지 구성하기 를 진행하시느라 고생하셨습니다~👍

다음 단계를 위해서 Merge하도록 하겠습니다.

@serverwizard serverwizard merged commit 03b3223 into next-step:isisrin Feb 23, 2020
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