Skip to content

[REFACTOR] 3-Layer 기반 클린아키텍처로 모듈화 #121

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
Merged
0Hooni merged 94 commits into from
Apr 21, 2025
Merged

[REFACTOR] 3-Layer 기반 클린아키텍처로 모듈화 #121

0Hooni merged 94 commits into from
Apr 21, 2025

Conversation

@0Hooni
Copy link
Member

@0Hooni 0Hooni commented Apr 21, 2025 •
edited
Loading

📌 이슈

✅ 작업 사항

  • 3-layer 클린아키텍처 기반 모듈화(Data, Domain, Presentation)
  • Presentation에서 사용되던 DTO(Data layer) 제거
  • 3-layer가 DomainInterface를 의존하여 동작하도록 수정
  • 기존의 구현체를 인터페이스화
  • 의존성 주입을 위한 DIContainer 생성
  • Reactor에 구현체 제거 및 의존성 주입

🚀 테스트 방식

빌드 타겟을 앱으로 설정한 뒤 실기기에서 빌드해보면 됩니다.

👀 ETC (추후 개발해야 할 것, 참고자료 등) ->

  • 로그인 서비스 의존성 흐름 개선
  • 카카오 로그인 에러 해결
  • @rpath 문제 해결 및 일부 프레임워크를 다이나믹 프레임워크로 수정

0Hooni added 30 commits April 14, 2025 20:51
@0Hooni
refactor/#112: Convert to Folder로 개선
d7acea3 
- 프로젝트가 추적하지 않는 파일들 삭제
@0Hooni
refactor/#112: KeyPath → Secrets로 변경
fd2d10c 
- 동일한 객체 네이밍이 존해서 변경함
@0Hooni
refactor/#112: Extension 위치 이동
51dcfee
@0Hooni
refactor/#112: 예상 모듈화에 맞게 폴더 구조 변경
226e19c
@0Hooni
refactor/#112: AdminRepository 인터페이스 분리
1db8dab
@0Hooni
refactor/#112: Repository 인터페이스 분리
6ed7c3d
@0Hooni
refactor/#112: Entity 폴더명 임시 수정
0cc4bf9
@0Hooni
refactor/#112: UseCase 인터페이스 분리
cc48b56
@0Hooni
refactor/#112: final, private let 으로 통일
448218a
@0Hooni
refactor/#112: DTO 폴더링
3448cac
@0Hooni
refactor/#112: Presigned 폴더링
405e9fe
@0Hooni
refactor/#112: 클린아키텍처 베이스 폴더링 구조로 변경
0e1fad0
@0Hooni
refactor/#112: 불필요한 파일 제거 및 Layer에 맞게 폴더링
e4230f2
@0Hooni
feat/#113: DIContainer 구현
f76fb70
@0Hooni
feat/#113: 간단한 의존성 주입을 위한 PropertyWrapper 구현
d62546d
@0Hooni
refactor/#113: KeyChainService 의존성 관리를 위해 DIContainer 적용
b99d07c
@0Hooni
refactor/#113: Provider 의존성 대체
aec9f97
@0Hooni
refactor/#113: MapUseCase 의존성 대체
54de56d
@0Hooni
style/#113: DIContainer lint 에러 수정
c25bc34
@0Hooni
refactor/#113: AdminUseCase 의존성 주입
de9e70b
@0Hooni
refactor/#113: MapUseCase 의존성 주입
c34ff2e
@0Hooni
refactor/#113: UserAPIUseCase 의존성 주입
e67aec7
@0Hooni
refactor/#113: PopUpAPIUseCase 의존성 주입
8c0e3eb
@0Hooni
refactor/#113: CommentAPIUseCase 의존성 주입
e489b14
@0Hooni
refactor/#113: HomeAPIUseCase 의존성 제거
d13c2b2
@0Hooni
refactor/#113: AuthAPIUseCase 의존성 주입
955d848
@0Hooni
refactor/#113: PopUpAPIUseCase 의존성 주입
514cad7
@0Hooni
refactor/#113: SignUpAPIUseCase 의존성 주입
588b5fe
@0Hooni
refactor/#113: MapDirectionRepository 의존성 주입
5a75bcf 
- repository의 메서드를 바로 끌어다쓰는 경우로 보임
- 제거되어야 되는 부분
- 반드시 UseCase를 태우도록 변경 필요
@0Hooni
remove/#113: 불필요한 의존성 인스턴스 생성 제거
235a227 
- UseCase를 불필요하게 ViewController에서 사용할 경우를 방지
- 불필요한 인스턴스를 제거하여 메모리 확보
@0Hooni 0Hooni added 🔄 refactor 프로덕션 코드 리팩토링, 파일 삭제, 네이밍 수정 및 폴더링 🎨 style 코드 스타일 수정, 컨벤션 적용 🐛 fix 버그 수정, 잔잔바리 수정, 병합 시 충돌 해결 🚀 feat 새로운 기능을 추가 🔧 chore 빌드 설정, 프로젝트 설정 등 “로직에 영향 없는” 변경 🗑️ remove 파일 삭제 labels Apr 21, 2025
@0Hooni 0Hooni self-assigned this Apr 21, 2025
@0Hooni 0Hooni linked an issue Apr 21, 2025 that may be closed by this pull request
물리적 분리를 위한 모듈화 #112
Closed
dongglehada
dongglehada approved these changes Apr 21, 2025
View reviewed changes
Copy link
Member

@dongglehada dongglehada left a comment

Choose a reason for hiding this comment

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

LGTM

@0Hooni 0Hooni merged commit 6786655 into develop Apr 21, 2025
This was linked to issues Apr 21, 2025
DIContainer 도입 및 Presentation의 의존성 주입 방식 변경 #113
Closed
Domain Layer에서 DTO 제거 (Home, PopUp, SignUp, User API관련) #116
Closed
Domain Layer에서 DTO 제거 (Admin, Auth, Comment, Map, MapDirection) #119
Closed
@0Hooni 0Hooni deleted the refactor/#112-Modulization branch April 21, 2025 14:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@zzangzzangguy zzangzzangguy zzangzzangguy approved these changes

+1 more reviewer

@dongglehada dongglehada dongglehada approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@0Hooni 0Hooni

@dongglehada dongglehada

@zzangzzangguy zzangzzangguy

Labels

🔧 chore 빌드 설정, 프로젝트 설정 등 “로직에 영향 없는” 변경 🚀 feat 새로운 기능을 추가 🐛 fix 버그 수정, 잔잔바리 수정, 병합 시 충돌 해결 🔄 refactor 프로덕션 코드 리팩토링, 파일 삭제, 네이밍 수정 및 폴더링 🗑️ remove 파일 삭제 🎨 style 코드 스타일 수정, 컨벤션 적용

Projects

None yet

Milestone

No milestone

4 participants

@0Hooni @dongglehada @zzangzzangguy