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

Refactor/domain layer #59

Open
wants to merge 3 commits into
base: dev
Choose a base branch
from
Open

Refactor/domain layer #59

wants to merge 3 commits into from

Conversation

eshc123
Copy link
Member

@eshc123 eshc123 commented Aug 30, 2024

주요 내용

  1. Domain 모듈 의존성 걷어내기
  • NetworkResult -> DomainResult 로 이름 변경 후 Domain 모듈로 이동 (네이밍은 다시 생각해보는게 좋을 듯...)
  • Request, Response 모델 Data 모듈로 이동
    • Data 모듈 이외에서 필요한 경우 적절하게 데이터 모델 정의 후 매핑.
  • DataStore -> Data 모듈이 바라보게 수정
  1. Model 모듈 -> 사용되는 각 모듈의 Model 패키지로 분리( or 분리 필요)

결과

리팩토링 전

리팩토링전 dot

리팩토링 후

리팩토링후

  • 도메인 계층은 다른 모듈에 대해 의존성을 갖지 않고 데이터 모듈과 프레젠테이션(feature 모듈들) 모듈들은 도메인 모듈을 바라보도록 변경

참고

  • 빌드 및 앱 실행까지 확인하였으나 많은 부분을 바꿨기 때문에 충분한 테스트 이후 dev에 머지를 해야할 것으로 보입니다!

@eshc123 eshc123 requested a review from bywindow August 30, 2024 14:03
@eshc123 eshc123 self-assigned this Aug 30, 2024
@bywindow
Copy link
Collaborator

bywindow commented Sep 2, 2024

모듈화를 좀 고민해봤는데..! 클린아키텍처를 따를거고 그걸 좀 더 모듈명에도 분명하게 드러내면 어떨까 싶긴 해!
image
아직 미완성이지만 이런 구조는 어떨까?

@eshc123
Copy link
Member Author

eshc123 commented Sep 2, 2024

오, 좋아! UI를 담당한다는 의미와 더불어 클린 아키텍처의 기조를 따른다는 면에서 feature보다 presentation이 더 적합할 것 같아보이네!
Build logic 모듈은 나도 한번 공부하려고 했던 부분이라 이번 기회에 접해보는 것도 좋을 것 같아!

@bywindow
Copy link
Collaborator

bywindow commented Sep 2, 2024

오, 좋아! UI를 담당한다는 의미와 더불어 클린 아키텍처의 기조를 따른다는 면에서 feature보다 presentation이 더 적합할 것 같아보이네! Build logic 모듈은 나도 한번 공부하려고 했던 부분이라 이번 기회에 접해보는 것도 좋을 것 같아!

그럼 이거 일단 현재 모듈화 상태에서 build-logic 모듈 분리하는 것부터 해볼까?

@eshc123
Copy link
Member Author

eshc123 commented Sep 2, 2024

그럼 이거 일단 현재 모듈화 상태에서 build-logic 모듈 분리하는 것부터 해볼까?

현재 모듈화 상태 가 이번 PR 전인 현재 dev 상태인거야? build-logic 자체는 이번 PR이랑 크게 관련이 없으니까 여기 브랜치에서 이어서 작업해도 되고, dev 기준으로 브랜치 하나 만들어서 해도 될 것 같아! 그럼 난 모듈 네이밍을 바꿔볼게!

@bywindow
Copy link
Collaborator

bywindow commented Sep 2, 2024

그럼 이거 일단 현재 모듈화 상태에서 build-logic 모듈 분리하는 것부터 해볼까?

현재 모듈화 상태 가 이번 PR 전인 현재 dev 상태인거야? build-logic 자체는 이번 PR이랑 크게 관련이 없으니까 여기 브랜치에서 이어서 작업해도 되고, dev 기준으로 브랜치 하나 만들어서 해도 될 것 같아! 그럼 난 모듈 네이밍을 바꿔볼게!

오키오키!! 내가 트라이 해본 뒤에 이 MR 머지되면 적용해볼게!

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