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

[week2] clean architecture #18

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

[week2] clean architecture #18

wants to merge 1 commit into from

Conversation

vo0a
Copy link
Contributor

@vo0a vo0a commented Feb 25, 2024

https://github.com/geulsseundang/season1/blob/feature/dromkim/week2/clean%20architecture.md

1주차에 작성했던 clean architecture 에 이어서 뒷 내용을 작성해봤습니다.
1주차에는 클린아키텍처가 무엇인지에 대한 정의를 담았다면, 이번에는 구현하는 내용을 작성했습니다.
3주차 에는 테스트 방법과, 경계간 매핑에 대해 작성해보겠습니다.

@vo0a vo0a self-assigned this Feb 25, 2024
Copy link
Member

@yunyezl yunyezl left a comment

Choose a reason for hiding this comment

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

한 주제에 대해 꾸준히 깊게 작성하시는 모습 좋아요~!

Comment on lines +33 to +37
### 1.2. 입력 유효성 검증

유스케이스가 하는 일 중 1번을 설명하며 `유효성 검증은 다른 곳에서 처리`한다고 언급했는데, 여전히 이 작업은 애플리케이션 계층의 책임에 해당합니다. 애플리케이션 코어의 바깥쪽으로부터 유효하지 않은 입력값을 받게 되고, 모델의 상태를 해칠 수 있기 때문입니다.

하지만 유스케이스 클래스에서는 하지 않고, 입력 모델을 사용하여 처리합니다.
Copy link
Member

Choose a reason for hiding this comment

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

저는 유즈케이스에서 여러 처리들을 많이 했던 것 같아서(대부분 비즈니스 로직이긴 하지만)
이 부분 어떻게 처리되는 건지 궁금한데, 자바에 대한 지식이 부족해서 이해가 어렵네요 ㅠㅠ

Comment on lines +53 to +56
| 구분점 | 도메인 모델의 현재 상태에 접근 X | 도메인 모델의 현재 상태에 접근 |
| 구현 방법 | 선언적으로 구현 - ex) @NotNull | 맥락이 필요 |
| 의미 | 구문상의(syntactical) 유효성 검증 | 의미적인(semantical) 유효성 검증 |
| 구현 위치 | 생성자 | 도메인 엔티티 |
Copy link
Member

Choose a reason for hiding this comment

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

요 부분을 읽어보니, Null이 되선 안되는 값이 Null로 들어올 때와 같은 상황인가보네요. (제가 잘 모르는 걸 수도 있지만.. ) 클라단에서는 발생하기 어려운 상황인 것 같아서 이해를 잘 못했던 것 같아요 ㅎㅎ

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