droidknights / DroidKnightsApp

국내 최대 규모 안드로이드 컨퍼런스 드로이드나이츠 앱
https://www.droidknights.dev
255 stars 69 forks source link

(ENG)[#252] core 이하 모듈의 의존성 개선 #254

Closed SeongUgJung closed 1 year ago

SeongUgJung commented 1 year ago

Issue

Overview (Required)

Before After
architecture-module-graph architecture-module-graph

Next TODO

github-actions[bot] commented 1 year ago

Test Results

18 tests   18 :heavy_check_mark:  6s :stopwatch: 11 suites    0 :zzz: 11 files      0 :x:

Results for commit 422cf2d2.

:recycle: This comment has been updated with latest results.

taehwandev commented 1 year ago

👍

taehwandev commented 1 year ago

요렇게 접근하면 장점은 보통 많이 이야기하는 솔리드의 인터페이스 분리원칙에 따르게 됩니다. 모듈 접근의 디펜던시도 줄어들게됩니다.

해당 브랜치를 추가로 만들어두면, 기존과 다르게 이렇게까지 가능함을 공유해볼 수 있을것 같아요.

SeongUgJung commented 1 year ago

의존성 개선 v2 에서 언급할 예정이었는데 domain, datasource, data 과 model 에 모든 구현체와 인터페이스가 집합되어 있는데

sponsor 모델 또는 data 만을 변경해도 contributor 와 같은 연관성이 없는 부분까지 영향을 받기때문에 이 부분 또한 종단으로 분리할 예정입니다.

해당 내용은 feature 모듈의 flatten 작업까지 완료되면 새롭게 이슈로 제안하고 MVVM_Split_API_Module 브랜치에서 진행하도록 하겠습니다.

taehwandev commented 1 year ago

그렇게 해주실거라서 별도로 분리해두고 이렇게까지 가능함을 구분해보고 싶어서 메인으로 머지 하지 않는것이 더 좋을것 같아서요! 두개가 공존하는 제안도 좋을것 같습니다. 메인 README에도 업데이트 해야겠네요.

SeongUgJung commented 1 year ago

모든 내용 적용하였습니다.

droidknights:MVVM_Split_API_Module 로의 머지를 고려해주시기 바랍니다.