Android Architecture 기반 구조 리팩터링

[ExpeditionMoon]

작업 목적

Android Architecture 기준으로 프로젝트 구조를 리팩터링했습니다.

이번 작업은 Robert C. Martin의 Clean Architecture 적용 브랜치와 비교하기 위한 아키텍처 학습 및 구조 비교 목적의 작업입니다.

작업 브랜치

• architecture/android

주요 변경 사항

1. UiState와 Effect 책임 분리

화면에 지속적으로 표시되는 상태와 일회성 이벤트를 분리했습니다.

• UiState: loading, input, selected item, list data 등 화면 상태
• Effect: snackbar, toast, 화면 이동, 카메라 이동, 저장 완료 등 일회성 이벤트

이를 통해 recomposition 상황에서 일회성 이벤트가 반복 처리되는 문제를 줄이고, ViewModel의 책임을 명확히 했습니다.

2. ViewModel coroutine 처리 정리

consult 작성 흐름을 중심으로 ViewModel coroutine 처리 방식을 정리했습니다.

• 상담 생성/수정 흐름을 viewModelScope와 suspend 함수 기반으로 정리
• ViewModel 내부에서 별도 CoroutineScope를 직접 생성하지 않도록 개선
• 상담 생성 후 알림 전송 흐름을 상담 생성 성공 처리 안에서 순차적으로 수행
• 저장/조회/알림 전송 흐름을 읽기 쉽게 정리

3. ViewModel 의존성 기준 정리

Android Architecture 기준으로 단순 조회/검색 작업은 Repository를 직접 사용하도록 정리했습니다.

• 단순 Repository 호출만 감싸는 UseCase 제거
• 복잡한 비즈니스 규칙, 검증, 여러 Repository 조합이 필요한 UseCase는 유지
• Android Architecture와 Clean Architecture의 차이를 비교할 수 있도록 의존성 기준 정리

4. data/domain 모듈 책임 정리

data 모듈에서는 Repository, DataSource, Mapper 책임을 점검했습니다.

• Repository 결과 처리 기준 정리
• Dispatcher 사용 기준 정리
• DataResourceResult 중심의 결과 반환 기준 통일
• DTO 구조 변경은 이번 범위에서 제외

domain 모듈에서는 단순 UseCase를 정리하고 Repository 반환 타입 기준을 점검했습니다.

5. component-ui 공통 UI 책임 정리

component-ui가 domain 모델에 직접 의존하지 않도록 정리했습니다.

• PharmacyUiModel
• PharmacistUiModel

을 추가하여 공통 UI는 화면 표시용 모델만 알도록 변경했습니다.

또한 PharmacySelector의 검색어 상태를 hoisting하여 공통 UI가 상태를 직접 소유하지 않도록 수정했습니다.

6. navigation callback 기준 정리

feature screen에서 NavController를 직접 사용하지 않도록 정리했습니다.

• Screen은 navigation callback만 받도록 변경
• 실제 navigate, popBackStack, navigateUp 처리는 NavGraphBuilder 또는 app shell에서 담당
• ViewModel은 navigation을 직접 실행하지 않고 Effect로 이동 의도만 전달

7. app 모듈 최종 정리

사용되지 않는 TopBar route helper를 제거했습니다.

• TitleBarCustom.kt 삭제

최종 검증

아래 명령어로 전체 debug 빌드가 성공하는 것을 확인했습니다.

./gradlew clean assembleDebug

결과:

BUILD SUCCESSFUL

비교 예정

이 PR은 Android Architecture 적용 결과입니다.

이후 별도 브랜치에서 Robert C. Martin의 Clean Architecture 기준으로 같은 프로젝트를 리팩터링하고, 두 구조의 차이를 비교할 예정입니다.

• Android Architecture 브랜치: architecture/android
• Clean Architecture 브랜치: architecture/clean

관련 이슈

Refs #115