🔗 :: (#379) 마이페이지 상세 ReactorKit #380

leejh08 · 2025-12-25T13:12:47Z

개요

마이페이지 상세 ReactorKit

작업사항

마이페이지 상세 ReactorKit

UI

Summary by CodeRabbit

릴리스 노트

New Features

버그 리포트: 리액터 기반 제출 흐름 도입 — 이미지 첨부 전송, 제출 완료 토스트 및 자동 뒤로가기 반영. 주요 분류 선택이 상태로 연동되어 UI 동기화 개선.
관심 분야: 선택 화면에서 자동 루트 복귀 기능 추가.

Refactor

알림 설정·버그 리포트·관심 분야 전체를 리액터(상태 중심) 아키텍처로 전환해 UI 바인딩과 내비게이션 일관성 향상.

Other

개발 유형에 대해 로컬 문구로부터 생성 가능한 초기화 기능 추가.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

coderabbitai · 2025-12-25T13:12:58Z

Walkthrough

MyPage의 BugReport, InterestField(및 Check), NotificationSetting 모듈을 MVVM(ViewModel)에서 ReactorKit(Reactor) 기반으로 전면 마이그레이션했습니다. 기존 ViewModel 파일 삭제, 대응 Reactor 추가, ViewController/Flow/DI에서 viewModel 참조를 reactor로 교체했습니다.

Changes

응집도 / 파일(들)	변경 내용 요약
BugReport `Projects/Presentation/Sources/BugReport/BugReportReactor.swift`, `Projects/Presentation/Sources/BugReport/BugReportViewController.swift`, `Projects/Presentation/Sources/BugReport/BugReportViewModel.swift`, `Projects/Flow/Sources/MyPage/BugReport/BugReportFlow.swift`	`BugReportReactor` 추가(액션/뮤테이션/상태), ViewController를 `BaseReactorViewController<BugReportReactor>`로 전환, 기존 `BugReportViewModel` 삭제, Flow에서 `rootViewController.reactor` 사용으로 변경
InterestField `Projects/Presentation/Sources/InterestField/InterestFieldReactor.swift`, `Projects/Presentation/Sources/InterestField/InterestFieldViewController.swift`, `Projects/Presentation/Sources/InterestField/InterestFieldViewModel.swift`, `Projects/Flow/Sources/MyPage/InterestField/InterestFieldFlow.swift`	`InterestFieldReactor` 추가, ViewController를 Reactor 기반으로 변경, 기존 ViewModel 삭제, 선택/상태 로직을 reactor.state로 이전, Flow에서 `.popHomeFieldIsRequired` 처리 및 `popToMyPage()` 도입
InterestFieldCheck `Projects/Presentation/Sources/InterestField/InterestFieldCheckReactor.swift`, `Projects/Presentation/Sources/InterestField/InterestFieldCheckViewController.swift`, `Projects/Presentation/Sources/InterestField/InterestFieldCheckViewModel.swift`, `Projects/Flow/Sources/MyPage/InterestField/InterestFieldCheckFlow.swift`	`InterestFieldCheckReactor` 추가(학생정보 조회, 자동 네비게이션 타이머), ViewController를 Reactor 기반으로 변경, 기존 ViewModel 삭제, Flow에서 루트 팝 처리 추가
NotificationSetting `Projects/Presentation/Sources/NotificationSetting/NotificationSettingReactor.swift`, `Projects/Presentation/Sources/NotificationSetting/NotificationSettingViewController.swift`, `Projects/Presentation/Sources/NotificationSetting/NotificationSettingViewModel.swift`, `Projects/Flow/Sources/MyPage/Notification/NotificationSettingFlow.swift`	`NotificationSettingReactor` 추가, ViewController를 Reactor 기반으로 변경, 기존 ViewModel 삭제, Flow에서 next-step 스테퍼를 reactor로 전환
DI 어셈블리 `Projects/Presentation/Sources/DI/Assembly/SettingPresentationAssembly.swift`	Presentation DI 등록/해결을 ViewModel → Reactor로 교체 (Notification/BugReport/InterestField/InterestFieldCheck)
도메인 유틸 `Projects/Domain/Sources/Enums/DevelopmentType.swift`	개발 영역 enum에 `init?(localizedString:)` failable 이니셜라이저 추가

Sequence Diagram(s)

sequenceDiagram
    autonumber
    participant User as 사용자 (UI)
    participant VC as ViewController
    participant Reactor as Reactor
    participant UseCase as UseCase
    participant Flow as Flow(Stepper)

    rect rgb(245,250,255)
    Note right of VC: bindAction / bindState
    end

    User->>VC: UI 이벤트 (예: 제출/토글/선택)
    VC->>Reactor: action (예: bugReportButtonDidTap / toggleNotification / selectButtonDidTap)
    Reactor->>UseCase: execute / 요청 (네트워크/도메인 작업)
    UseCase-->>Reactor: 결과(성공/실패)
    alt 성공
      Reactor->>Reactor: emit Mutation -> state 업데이트
      Reactor->>Flow: steps.emit(네비게이션 스텝)  %% 강조된 새 상호작용
    else 실패
      Reactor->>Reactor: emit 실패 관련 Mutation -> state 업데이트
    end
    Reactor->>VC: state 변경 (bindState로 UI 반영)

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~45분

Possibly related issues

마이페이지 내부 기능 ReactorKit Refactoring #379 — MyPage의 MVVM → ReactorKit 리팩토링 전반을 다루고 있어 본 변경과 목적이 일치합니다.

Possibly related PRs

🔗 :: (#295) 관심분야 기능 개발 #310 — InterestField 관련 네비게이션/프레젠테이션 변경과 코드 수준 연관성 높음
🔗 :: (#375) MyPage ReactorKit Refactoring #376 — MyPage 영역의 Reactor 마이그레이션(Flow/스테퍼 교체)과 직접 연관됨

(최대 3개로 제한)

Suggested reviewers

circle0802
cyj513

Poem

🐰 버튼을 탭하니 Reactor가 깡충,
State가 춤추고 UI가 반짝,
ViewModel은 쉬러 가고 우리는 전진,
변경의 당근을 냠냠 먹으며 축하하네 🥕✨

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	You can run `@coderabbitai generate docstrings` to improve docstring coverage.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	PR 제목이 마이페이지 상세 화면을 ReactorKit으로 리팩터링한다는 주요 변경사항을 명확하게 반영하고 있습니다.

✨ Finishing touches

📝 Generate docstrings

🧪 Generate unit tests (beta)

Create PR with unit tests
Post copyable unit tests in a comment
Commit unit tests in branch feature/(#379)-Mypage_Detail_Refactoring

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

github-actions · 2025-12-25T13:13:21Z

All File Checked ✅

✅	Auto Pull Request SwiftLint Success!!!!

coderabbitai

Actionable comments posted: 4

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)

Projects/Presentation/Sources/NotificationSetting/NotificationSettingViewController.swift (1)
164-172: 중복된 UI 업데이트 로직으로 인한 잠재적 충돌

configureViewController()에서 allNotificationSwitchView.clickSwitchButton을 구독하여 모든 스위치 뷰를 직접 업데이트하고 있습니다. 그러나 bindState()(Lines 128-161)에서도 reactor state 기반으로 각 스위치 뷰를 업데이트하고 있어, 두 로직이 동시에 동작하면서 UI 상태 충돌이 발생할 수 있습니다.

Reactor 패턴에서는 모든 UI 업데이트가 state를 통해 이루어져야 하므로, 이 직접 업데이트 로직은 제거하는 것이 좋습니다.
🔎 중복 로직 제거 제안
     public override func configureViewController() {
-        allNotificationSwitchView.clickSwitchButton
-            .asObservable()
-            .bind(onNext: { [weak self] isOn in
-                self?.switchViewArray.forEach {
-                    $0.setup(isOn: isOn)
-                }
-            }).disposed(by: disposeBag)
     }

🧹 Nitpick comments (10)

Projects/Flow/Sources/MyPage/InterestField/InterestFieldFlow.swift (1)
28-34: InterestFieldCheckStep 케이스 처리가 명시적이지 않습니다.

default 케이스를 사용하면 나중에 InterestFieldCheckStep에 새로운 케이스가 추가될 때 컴파일러 경고 없이 .none으로 처리됩니다. 모든 케이스를 명시적으로 처리하면 향후 유지보수에 도움이 됩니다.
🔎 명시적 케이스 처리 제안
 } else if let step = step as? InterestFieldCheckStep {
     switch step {
     case .popHomeFieldIsRequired:
         return popToMyPage()
-    default:
+    case .interestFieldIsRequired, .interestFieldCheckIsRequired:
         return .none
     }
 }
Projects/Flow/Sources/MyPage/InterestField/InterestFieldCheckFlow.swift (1)

58-61: popToMyPage() 함수가 InterestFieldFlow.swift와 중복됩니다.

동일한 구현이 두 Flow 파일에 존재합니다. 공통 확장이나 프로토콜로 추출하면 코드 중복을 줄일 수 있습니다. 다만, Flow 간 독립성을 위해 의도적으로 분리한 것일 수 있습니다.

Projects/Presentation/Sources/InterestField/InterestFieldCheckViewController.swift (1)

12-15: 주석 처리된 코드를 정리해 주세요.

backButton 관련 코드가 주석 처리되어 있습니다. 더 이상 필요하지 않다면 제거하는 것이 코드 가독성을 높입니다.

Also applies to: 20-21, 29-33
Projects/Presentation/Sources/InterestField/InterestFieldViewController.swift (1)
105-116: 셀 구성 시 reactor.currentState 접근에 주의가 필요합니다.

rx.items 클로저 내에서 reactor.currentState.selectedInterests에 접근하고 있습니다. 이 클로저는 availableInterests가 emit될 때 호출되지만, selectedInterests가 변경되어도 셀이 자동으로 업데이트되지 않습니다.

Lines 143-149의 reloadData() 호출로 이 문제를 해결하고 있지만, combineLatest를 사용하면 더 반응적인 방식으로 처리할 수 있습니다.
🔎 combineLatest 사용 제안
Observable.combineLatest(
    reactor.state.map { $0.availableInterests },
    reactor.state.map { $0.selectedInterests }
)
.bind(to: majorCollectionView.rx.items(...)) { index, (codeEntity, selectedInterests), cell in
    // ...
}
Projects/Presentation/Sources/BugReport/BugReportViewController.swift (1)

11-12: 이미지 상태를 Reactor로 이동하는 것을 고려해 주세요.

imageStringList와 imageList가 ViewController의 인스턴스 변수로 관리되고 있습니다. ReactorKit 패턴에서는 상태를 Reactor의 State에서 관리하는 것이 일관성 있는 단방향 데이터 흐름을 유지하는 데 도움이 됩니다.
Projects/Presentation/Sources/InterestField/InterestFieldCheckReactor.swift (1)
35-38: fetchStudentInfo 액션에서 에러 처리 누락

fetchStudentInfoUseCase.execute()가 실패할 경우 에러가 전파되어 스트림이 종료될 수 있습니다. catchAndReturn 또는 catchErrorJustComplete를 사용하여 에러를 적절히 처리하는 것을 권장합니다.
🔎 에러 처리 추가 제안
         case .fetchStudentInfo:
             return fetchStudentInfoUseCase.execute()
                 .asObservable()
                 .map { .setStudentName($0.studentName) }
+                .catch { _ in .empty() }
Projects/Presentation/Sources/DI/Assembly/SettingPresentationAssembly.swift (1)
81-88: 등록 순서 개선 제안

BugReportViewController가 BugReportReactor보다 먼저 등록되어 있습니다. 가독성을 위해 의존성(Reactor)을 먼저 등록하고 이를 사용하는 ViewController를 이후에 등록하는 것이 좋습니다.
🔎 등록 순서 변경 제안
         // Bug Report
+        container.register(BugReportReactor.self) { resolver in
+            BugReportReactor(
+                reportBugUseCase: resolver.resolve(ReportBugUseCase.self)!
+            )
+        }
         container.register(BugReportViewController.self) { resolver in
             BugReportViewController(resolver.resolve(BugReportReactor.self)!)
         }
-        container.register(BugReportReactor.self) { resolver in
-            BugReportReactor(
-                reportBugUseCase: resolver.resolve(ReportBugUseCase.self)!
-            )
-        }
Projects/Presentation/Sources/InterestField/InterestFieldReactor.swift (2)
52-65: fetchInterestFields에서 에러 처리 누락

fetchCodeListUseCase와 fetchStudentInfoUseCase 호출 시 에러 처리가 없습니다. 네트워크 실패 시 스트림이 종료되어 이후 액션이 처리되지 않을 수 있습니다.
🔎 에러 처리 추가 제안
         case .fetchInterestFields:
             let interests = fetchCodeListUseCase.execute(
                 keyword: nil,
                 type: .job,
                 parentCode: nil
             )
             .asObservable()
             .map { Mutation.setAvailableInterests($0) }
+            .catch { _ in .empty() }

             let studentInfo = fetchStudentInfoUseCase.execute()
                 .asObservable()
                 .map { Mutation.setStudentName($0.studentName) }
+                .catch { _ in .empty() }

             return .merge(interests, studentInfo)
70-78: selectButtonDidTap 액션에서 에러 처리 누락

changeInterestsUseCase.execute()가 실패할 경우 사용자에게 피드백 없이 실패할 수 있습니다. 에러 상태를 State에 추가하거나, 최소한 에러를 로깅하는 것을 권장합니다.
🔎 에러 처리 추가 제안
         case .selectButtonDidTap:
             let codeIDs = currentState.selectedInterests.map { $0.code }
             return changeInterestsUseCase.execute(codeIDs: codeIDs)
                 .asObservable()
                 .do(onCompleted: { [weak self] in
                     self?.steps.accept(InterestFieldStep.interestFieldCheckIsRequired)
                 })
                 .flatMap { _ in Observable<Mutation>.empty() }
+                .catch { _ in .empty() }
Projects/Presentation/Sources/NotificationSetting/NotificationSettingReactor.swift (1)
61-86: mutate 함수에서 에러 처리 누락

fetchSubscribeStateUseCase, subscribeNotificationUseCase, subscribeAllNotificationUseCase 호출 시 에러 처리가 없습니다. 네트워크 실패 시 적절한 사용자 피드백이 필요할 수 있습니다.
🔎 에러 처리 추가 예시
         case .fetchNotificationSettings:
             return fetchSubscribeStateUseCase.execute()
                 .asObservable()
                 .map { entities -> [NotificationType: Bool] in
                     var states: [NotificationType: Bool] = [:]
                     entities.forEach { states[$0.topic] = $0.isSubscribed }
                     return states
                 }
                 .map { .setSubscribeStates($0) }
+                .catch { _ in .empty() }

📜 Review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between b8d06ab and d509d60.

📒 Files selected for processing (17)

Projects/Flow/Sources/MyPage/BugReport/BugReportFlow.swift
Projects/Flow/Sources/MyPage/InterestField/InterestFieldCheckFlow.swift
Projects/Flow/Sources/MyPage/InterestField/InterestFieldFlow.swift
Projects/Flow/Sources/MyPage/Notification/NotificationSettingFlow.swift
Projects/Presentation/Sources/BugReport/BugReportReactor.swift
Projects/Presentation/Sources/BugReport/BugReportViewController.swift
Projects/Presentation/Sources/BugReport/BugReportViewModel.swift
Projects/Presentation/Sources/DI/Assembly/SettingPresentationAssembly.swift
Projects/Presentation/Sources/InterestField/InterestFieldCheckReactor.swift
Projects/Presentation/Sources/InterestField/InterestFieldCheckViewController.swift
Projects/Presentation/Sources/InterestField/InterestFieldCheckViewModel.swift
Projects/Presentation/Sources/InterestField/InterestFieldReactor.swift
Projects/Presentation/Sources/InterestField/InterestFieldViewController.swift
Projects/Presentation/Sources/InterestField/InterestFieldViewModel.swift
Projects/Presentation/Sources/NotificationSetting/NotificationSettingReactor.swift
Projects/Presentation/Sources/NotificationSetting/NotificationSettingViewController.swift
Projects/Presentation/Sources/NotificationSetting/NotificationSettingViewModel.swift

💤 Files with no reviewable changes (4)

Projects/Presentation/Sources/NotificationSetting/NotificationSettingViewModel.swift
Projects/Presentation/Sources/BugReport/BugReportViewModel.swift
Projects/Presentation/Sources/InterestField/InterestFieldViewModel.swift
Projects/Presentation/Sources/InterestField/InterestFieldCheckViewModel.swift

🧰 Additional context used

🧬 Code graph analysis (7)

Projects/Presentation/Sources/NotificationSetting/NotificationSettingViewController.swift (1)

Projects/Presentation/Sources/NotificationSetting/Component/NotificationSectionView.swift (1)

setup (57-62)

Projects/Presentation/Sources/InterestField/InterestFieldCheckReactor.swift (3)

Projects/Presentation/Sources/InterestField/Components/InterestCheckView.swift (1)

setStudentName (57-65)

Projects/Presentation/Sources/InterestField/InterestFieldReactor.swift (2)

mutate (50-79)

reduce (81-100)

Projects/Presentation/Sources/NotificationSetting/NotificationSettingReactor.swift (2)

mutate (61-87)

reduce (89-108)

Projects/Presentation/Sources/InterestField/InterestFieldReactor.swift (3)

Projects/Presentation/Sources/InterestField/Components/InterestCheckView.swift (1)

setStudentName (57-65)

Projects/Presentation/Sources/InterestField/InterestFieldCheckReactor.swift (2)

mutate (33-50)

reduce (52-61)

Projects/Presentation/Sources/NotificationSetting/NotificationSettingReactor.swift (2)

mutate (61-87)

reduce (89-108)

Projects/Flow/Sources/MyPage/InterestField/InterestFieldCheckFlow.swift (1)

Projects/Flow/Sources/MyPage/InterestField/InterestFieldFlow.swift (1)

popToMyPage (59-62)

Projects/Presentation/Sources/NotificationSetting/NotificationSettingReactor.swift (1)

Projects/App/Sources/AppDelegate.swift (1)

messaging (91-104)

Projects/Presentation/Sources/InterestField/InterestFieldCheckViewController.swift (2)

Projects/Presentation/Sources/InterestField/InterestFieldViewController.swift (2)

bindAction (83-102)

bindState (104-150)

Projects/Presentation/Sources/NotificationSetting/NotificationSettingViewController.swift (2)

bindAction (90-125)

bindState (127-162)

Projects/Presentation/Sources/BugReport/BugReportViewController.swift (3)

Projects/Presentation/Sources/MyPage/MyPageViewController.swift (2)

bindAction (92-145)

bindState (147-165)

Projects/Presentation/Sources/BugReport/MajorBottomSheet/MajorBottomSheetViewController.swift (1)

bind (52-58)

Projects/Modules/DesignSystem/Sources/Extensions/UIViewController/UIViewController+showJobisToast.swift (1)

showJobisToast (5-33)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)

GitHub Check: JOBIS_DSM_iOS | JOBIS v2 CI | Test - iOS
GitHub Check: JOBIS_DSM_iOS | JOBIS v2 CI | Build - iOS

🔇 Additional comments (16)

Projects/Flow/Sources/MyPage/BugReport/BugReportFlow.swift (1)

36-36: ReactorKit 패턴으로의 마이그레이션이 잘 되었습니다.

viewModel에서 reactor로의 전환이 일관되게 적용되었으며, steps relay를 통한 네비게이션 스텝 처리 방식이 ReactorKit의 Stepper 프로토콜과 잘 맞습니다.

Also applies to: 45-45

Projects/Presentation/Sources/BugReport/BugReportReactor.swift (1)

37-43: State 프로퍼티의 접근 제어자를 확인해 주세요.

State struct의 프로퍼티들이 internal 접근 수준입니다. 외부 모듈에서 reactor.currentState에 접근해야 하는 경우 public으로 선언해야 할 수 있습니다.

Projects/Flow/Sources/MyPage/Notification/NotificationSettingFlow.swift (1)

33-33: LGTM!

ReactorKit 패턴으로의 마이그레이션이 올바르게 적용되었습니다.

Projects/Flow/Sources/MyPage/InterestField/InterestFieldFlow.swift (1)

45-45: Reactor 기반 stepper로의 전환이 잘 되었습니다.

viewModel에서 reactor로의 마이그레이션이 일관되게 적용되었습니다.

Also applies to: 55-55

Projects/Flow/Sources/MyPage/InterestField/InterestFieldCheckFlow.swift (1)

40-40: LGTM!

Reactor 기반 stepper로의 전환이 일관되게 적용되었습니다.

Also applies to: 54-54

Projects/Presentation/Sources/InterestField/InterestFieldCheckViewController.swift (1)

41-61: ReactorKit 바인딩 패턴이 잘 적용되었습니다.

viewWillAppear에서 데이터를 fetch하고 viewDidAppear에서 자동 네비게이션을 시작하는 패턴이 명확합니다. State 바인딩에서 filter와 distinctUntilChanged를 적절히 사용하고 있습니다.

Projects/Presentation/Sources/InterestField/InterestFieldViewController.swift (1)

159-162: Safe subscript 확장이 좋습니다.

배열 범위 초과 접근을 방지하는 안전한 패턴입니다. 다만 이 확장이 다른 곳에서도 사용된다면 Core 모듈 등 공통 위치로 이동하는 것을 고려해 보세요.

Projects/Presentation/Sources/BugReport/BugReportViewController.swift (1)

169-176: 이미지 리스트 업데이트와 버튼 탭 액션의 순서 의존성이 있습니다.

do(onNext:)에서 updateImageList를 보낸 후 bugReportButtonDidTap을 보내는데, 두 액션이 순차적으로 처리된다는 보장이 있어야 합니다. Reactor의 mutate가 동기적이므로 현재는 동작하지만, 이미지 리스트를 Reactor State로 이동하면 이 의존성이 명확해집니다.

Projects/Presentation/Sources/InterestField/InterestFieldCheckReactor.swift (1)

40-48: 자동 네비게이션 타이머 구현이 적절합니다.

2초 타이머 후 steps.accept를 통해 네비게이션을 트리거하는 구현이 ReactorKit 패턴을 잘 따르고 있습니다. [weak self]를 사용하여 메모리 누수도 방지하고 있습니다.

Projects/Presentation/Sources/DI/Assembly/SettingPresentationAssembly.swift (2)

48-56: NotificationSettingReactor DI 등록이 올바르게 변경되었습니다.

ViewModel에서 Reactor로의 마이그레이션이 적절히 반영되었습니다.

109-130: InterestField 관련 Reactor 등록이 올바르게 구성되었습니다.

InterestFieldReactor와 InterestFieldCheckReactor의 의존성 주입이 적절히 설정되어 있습니다.

Projects/Presentation/Sources/NotificationSetting/NotificationSettingViewController.swift (2)

90-125: bindAction 구현이 ReactorKit 패턴을 잘 따르고 있습니다.

각 스위치 버튼에 skip(1)을 사용하여 초기값 방출을 방지하고, 적절한 액션으로 매핑하고 있습니다.

127-162: bindState 구현이 적절합니다.

distinctUntilChanged()를 사용하여 불필요한 UI 업데이트를 방지하고 있으며, 각 상태를 해당 UI 컴포넌트에 올바르게 바인딩하고 있습니다.

Projects/Presentation/Sources/InterestField/InterestFieldReactor.swift (1)

81-100: reduce 함수 구현이 적절합니다.

각 Mutation에 대한 상태 업데이트가 명확하고, toggleSelectedInterest의 toggle 로직이 올바르게 구현되었습니다.

Projects/Presentation/Sources/NotificationSetting/NotificationSettingReactor.swift (2)

39-57: State 구조가 잘 설계되었습니다.

subscribeStates 딕셔너리와 computed properties를 조합하여 각 알림 타입의 상태를 깔끔하게 관리하고 있습니다. isAllNotificationEnabled가 모든 개별 상태의 AND 조건으로 정의되어 있어 논리적으로 올바릅니다.

89-108: reduce 함수 구현이 적절합니다.

각 Mutation에 대한 상태 업데이트가 명확하고 불변성을 유지하며 새로운 State를 반환하고 있습니다.

Projects/Presentation/Sources/BugReport/BugReportReactor.swift

Projects/Presentation/Sources/BugReport/BugReportViewController.swift

Projects/Presentation/Sources/NotificationSetting/NotificationSettingReactor.swift

coderabbitai

Actionable comments posted: 1

♻️ Duplicate comments (3)

Projects/Presentation/Sources/BugReport/BugReportReactor.swift (2)
75-84: 에러 처리가 누락되어 사용자 피드백이 없습니다.

네트워크 오류나 서버 에러가 발생해도 사용자에게 아무런 피드백이 제공되지 않습니다. 실패 시 에러 상태를 추가하거나 별도의 mutation을 통해 사용자에게 알려야 합니다.
🔎 에러 처리 추가 제안

State에 에러 프로퍼티 추가:
 public struct State {
     var title: String = ""
     var content: String = ""
     var imageList: [String] = []
     var majorType: String = "전체"
     var isBugReportButtonEnabled: Bool = false
     var isBugReportCompleted: Bool = false
+    var errorMessage: String?
 }
Mutation에 에러 케이스 추가:
 public enum Mutation {
     case setTitle(String)
     case setContent(String)
     case setImageList([String])
     case setMajorType(String)
     case setBugReportButtonIsEnabled(Bool)
     case setBugReportCompleted
+    case setError(String)
 }
mutate에서 에러 처리:
 case .bugReportButtonDidTap:
     return reportBugUseCase.execute(req: .init(
         title: currentState.title,
         content: currentState.content,
         developmentArea: DevelopmentType(rawValue: currentState.majorType.uppercased()) ?? .all,
         attachmentUrls: currentState.imageList
     ))
     .asObservable()
     .map { _ in Mutation.setBugReportCompleted }
+    .catch { error in
+        return .just(.setError(error.localizedDescription))
+    }
ViewController에서 에러 상태 구독하여 토스트 표시:
reactor.state.compactMap { $0.errorMessage }
    .bind(onNext: { [weak self] message in
        self?.showJobisToast(text: message, inset: 70)
    })
    .disposed(by: disposeBag)
79-79: majorType이 한글 문자열이므로 DevelopmentType 변환이 실패합니다.

majorType의 기본값은 "전체"(한글)이고 사용자가 선택하는 값도 한글입니다. DevelopmentType의 rawValue는 "ALL", "SERVER", "WEB" 등 영문이므로, uppercased()를 적용해도 한글은 변환되지 않아 DevelopmentType(rawValue:)이 항상 nil을 반환합니다. 결과적으로 사용자가 선택한 개발 분야가 무시되고 항상 .all로 전송됩니다.
🔎 수정 방안

방안 1: 한글-영문 매핑 함수 추가
private func mapMajorTypeToEnglish(_ majorType: String) -> String {
    switch majorType {
    case "전체": return "ALL"
    case "서버": return "SERVER"
    case "웹": return "WEB"
    case "안드로이드": return "ANDROID"
    case "iOS": return "IOS"
    case "디자인": return "DESIGN"
    case "크로스플랫폼": return "CROSS_PLATFORM"
    default: return "ALL"
    }
}

// mutate에서 사용:
developmentArea: DevelopmentType(rawValue: mapMajorTypeToEnglish(currentState.majorType)) ?? .all
방안 2: State에서 majorType을 영문으로 저장

State를 영문으로 저장하고, UI 표시용 한글 변환은 ViewController에서 처리:
 public struct State {
     var title: String = ""
     var content: String = ""
     var imageList: [String] = []
-    var majorType: String = "전체"
+    var majorType: String = "ALL"
     var isBugReportButtonEnabled: Bool = false
     var isBugReportCompleted: Bool = false
 }
그리고 ViewController에서 한글로 변환:
reactor.state.map { $0.majorType }
    .distinctUntilChanged()
    .map { englishType in
        switch englishType {
        case "ALL": return "전체"
        case "SERVER": return "서버"
        // ... 나머지 매핑
        default: return "전체"
        }
    }
    .bind(to: bugReportMajorView.majorLabel.rx.text)
    .disposed(by: disposeBag)
Projects/Presentation/Sources/BugReport/BugReportViewController.swift (1)
207-214: 완료 처리 로직에 take(1)이 누락되었습니다.

이전 리뷰에서 지적된 reactor.action 구독 문제는 해결되었습니다. 이제 API 호출 완료 후에 올바르게 처리됩니다.

하지만 현재 구독에 take(1) 연산자가 없어, 이론적으로 isBugReportCompleted가 여러 번 true로 변경되면 토스트와 pop이 여러 번 실행될 수 있습니다. 또한 화면이 dismiss된 후에도 구독이 유지될 수 있습니다.
🔎 제안하는 수정
 reactor.state.map { $0.isBugReportCompleted }
     .distinctUntilChanged()
     .filter { $0 }
+    .take(1)
     .bind(onNext: { [weak self] _ in
         self?.showJobisToast(text: "버그제보가 완료되었습니다.", inset: 70)
         self?.navigationController?.popViewController(animated: true)
     })
     .disposed(by: disposeBag)

📜 Review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between d509d60 and 2f664be.

📒 Files selected for processing (2)

Projects/Presentation/Sources/BugReport/BugReportReactor.swift
Projects/Presentation/Sources/BugReport/BugReportViewController.swift

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)

GitHub Check: JOBIS_DSM_iOS | JOBIS v2 CI | Build - iOS

🔇 Additional comments (7)

Projects/Presentation/Sources/BugReport/BugReportViewController.swift (2)

10-10: ReactorKit 마이그레이션이 올바르게 적용되었습니다.

클래스 시그니처가 BaseReactorViewController<BugReportReactor>로 올바르게 변경되어 ReactorKit 아키텍처에 부합합니다.

179-189: State 바인딩이 올바르게 구현되었습니다.

버튼 활성화 상태와 전공 레이블이 reactor state와 정확하게 바인딩되어 있으며, distinctUntilChanged()를 통해 불필요한 UI 업데이트를 방지하고 있습니다.

Projects/Presentation/Sources/BugReport/BugReportReactor.swift (5)

8-18: ReactorKit 구조가 올바르게 구현되었습니다.

BaseReactor와 Stepper를 준수하고, 의존성 주입을 통해 use case를 받아 초기화하는 구조가 ReactorKit 패턴에 부합합니다.

38-45: State 정의가 적절합니다.

필요한 상태들이 모두 정의되어 있으며, ReactorKit의 단방향 데이터 플로우를 잘 따르고 있습니다. isBugReportCompleted 플래그를 통해 완료 상태를 명확하게 관리하고 있습니다.

51-63: 입력 검증 로직이 올바르게 구현되었습니다.

제목과 내용이 모두 입력되었을 때만 버튼을 활성화하는 로직이 적절하며, concat을 사용하여 여러 mutation을 순차적으로 반환하는 방식도 ReactorKit의 Best Practice를 따르고 있습니다.

65-73: Navigation과 상태 업데이트가 올바르게 분리되어 있습니다.

majorViewDidTap에서 steps를 통해 navigation을 처리하는 것은 RxFlow 패턴의 올바른 사용법이며, 나머지 액션들도 적절하게 mutation을 반환하고 있습니다.

87-109: Reduce 로직이 올바르게 구현되었습니다.

모든 mutation이 state에 정확하게 반영되고 있으며, ReactorKit의 단방향 데이터 플로우를 잘 따르고 있습니다.

Projects/Presentation/Sources/BugReport/BugReportViewController.swift

coderabbitai

Actionable comments posted: 0

♻️ Duplicate comments (1)

Projects/Presentation/Sources/BugReport/BugReportReactor.swift (1)
75-84: 에러 처리가 여전히 누락되어 있습니다.

reportBugUseCase.execute()가 실패할 경우 사용자에게 피드백이 제공되지 않습니다. 이전 리뷰에서 지적된 문제가 해결되지 않았습니다.
🔎 에러 처리 추가 제안
 case .bugReportButtonDidTap:
     return reportBugUseCase.execute(req: .init(
         title: currentState.title,
         content: currentState.content,
         developmentArea: DevelopmentType(localizedString: currentState.majorType) ?? .all,
         attachmentUrls: currentState.imageList
     ))
     .asObservable()
     .map { _ in Mutation.setBugReportCompleted }
+    .catch { error in
+        // TODO: 에러를 State에 저장하거나 별도 처리
+        print("Bug report failed: \(error)")
+        return .empty()
+    }

🧹 Nitpick comments (1)

Projects/Domain/Sources/Enums/DevelopmentType.swift (1)
29-38: 대소문자 처리 및 유지보수성 검토 필요

영문 문자열 매칭이 대소문자를 구분하고 있습니다. 예를 들어 "all" 또는 "ALL"을 입력하면 nil이 반환됩니다. 입력 소스에 따라 이것이 의도된 동작인지 확인이 필요합니다.

또한, localizedString() 메서드와 이 이니셜라이저 간의 매핑을 동기화해야 하므로, 향후 유지보수 시 불일치가 발생할 수 있습니다.
🔎 대소문자 무시 매칭을 위한 개선안
 public init?(localizedString: String) {
-    switch localizedString {
+    switch localizedString.lowercased() {
-    case "전체", "All": self = .all
-    case "서버", "Server": self = .server
-    case "웹", "Web": self = .web
-    case "안드로이드", "Android": self = .android
-    case "iOS": self = .ios
+    case "전체", "all": self = .all
+    case "서버", "server": self = .server
+    case "웹", "web": self = .web
+    case "안드로이드", "android": self = .android
+    case "ios": self = .ios
     default: return nil
     }
 }

📜 Review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 2f664be and 64036a1.

📒 Files selected for processing (2)

Projects/Domain/Sources/Enums/DevelopmentType.swift
Projects/Presentation/Sources/BugReport/BugReportReactor.swift

🧰 Additional context used

🧬 Code graph analysis (1)

Projects/Presentation/Sources/BugReport/BugReportReactor.swift (3)

Projects/Modules/DesignSystem/Sources/Alert/AlertBuilder.swift (1)

setTitle (22-25)

Projects/Presentation/Sources/CompanyDetail/Components/CompanyDetailLabel.swift (1)

setContent (22-24)

Projects/Domain/Sources/Enums/DevelopmentType.swift (1)

localizedString (10-27)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)

GitHub Check: JOBIS_DSM_iOS | JOBIS v2 CI | Test - iOS
GitHub Check: JOBIS_DSM_iOS | JOBIS v2 CI | Build - iOS

🔇 Additional comments (8)

Projects/Presentation/Sources/BugReport/BugReportReactor.swift (8)

1-18: 클래스 구조와 초기화가 올바릅니다.

ReactorKit 패턴을 올바르게 따르고 있으며, 의존성 주입 및 RxFlow 통합이 적절합니다.

20-27: 액션 정의가 명확합니다.

모든 사용자 인터랙션을 적절히 표현하고 있습니다.

29-45: Mutation과 State 정의가 적절합니다.

상태 변화를 명확하게 표현하고 있으며, 초기값이 합리적입니다.

51-63: 제목과 내용 업데이트 로직이 올바릅니다.

버튼 활성화 조건을 정확히 검증하고 있습니다.

65-73: 이미지 리스트, 분야 타입 업데이트 및 네비게이션 처리가 올바릅니다.

Stepper 패턴을 적절히 활용하고 있습니다.

87-109: 리듀서 구현이 올바릅니다.

각 Mutation을 적절히 State에 반영하고 있습니다.

75-84: 버그 제보 성공 시의 네비게이션 처리와 알림은 이미 구현되어 있습니다.

BugReportViewController의 207-212줄에서 isBugReportCompleted 상태를 관찰하여 "버그제보가 완료되었습니다." 토스트 메시지를 표시하고 navigationController?.popViewController(animated: true)로 화면을 닫는 로직이 존재합니다.

Likely an incorrect or invalid review comment.

75-84: DevelopmentType(localizedString:) 이니셜라이저는 이미 존재하므로 이 코드는 정상입니다.

Projects/Domain/Sources/Enums/DevelopmentType.swift (line 29-38)에서 public init?(localizedString: String) 이니셜라이저가 정의되어 있으므로, line 79의 코드는 문제없이 작동합니다.

Likely an incorrect or invalid review comment.

leejh08 added 5 commits December 25, 2025 21:23

✨ :: (#379) 버그 제보 Reactor

1b4058e

✨ :: (#379) 알림 설정 Reactor

9d63896

✨ :: (#379) 관심분야 Reactor

bf28425

🗂️ :: (#379) viewWillappear -> fetch

3066acd

✨ :: (#379) 관심분야 선택 후 자동 pop

d509d60

leejh08 requested a review from circle0802 December 25, 2025 13:12

leejh08 self-assigned this Dec 25, 2025

leejh08 added the ♻️refactor 코드 리팩토링 할 경우 label Dec 25, 2025

leejh08 linked an issue Dec 25, 2025 that may be closed by this pull request

마이페이지 내부 기능 ReactorKit Refactoring #379

Closed

coderabbitai bot reviewed Dec 25, 2025

View reviewed changes

✨ :: (#379) api 호출 후 뷰 이동 플로우

2f664be

coderabbitai bot reviewed Dec 25, 2025

View reviewed changes

Projects/Presentation/Sources/BugReport/BugReportViewController.swift Show resolved Hide resolved

🗂️ :: (#379) 매핑 에러 수정

64036a1

coderabbitai bot reviewed Dec 25, 2025

View reviewed changes

leejh08 merged commit 8702a5d into develop Dec 25, 2025
2 of 5 checks passed

leejh08 deleted the feature/(#379)-Mypage_Detail_Refactoring branch December 25, 2025 23:37

coderabbitai bot mentioned this pull request Dec 29, 2025

🔗 :: (#381) ReactorKit Refactoring #383

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

🔗 :: (#379) 마이페이지 상세 ReactorKit #380

🔗 :: (#379) 마이페이지 상세 ReactorKit #380

Uh oh!

leejh08 commented Dec 25, 2025 •

edited by coderabbitai bot

Loading

Uh oh!

coderabbitai bot commented Dec 25, 2025 •

edited

Loading

Uh oh!

github-actions bot commented Dec 25, 2025

Uh oh!

coderabbitai bot left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

coderabbitai bot left a comment

Uh oh!

Uh oh!

coderabbitai bot left a comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

🔗 :: (#379) 마이페이지 상세 ReactorKit #380

🔗 :: (#379) 마이페이지 상세 ReactorKit #380

Uh oh!

Conversation

leejh08 commented Dec 25, 2025 • edited by coderabbitai bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

개요

작업사항

UI

Summary by CodeRabbit

릴리스 노트

Uh oh!

coderabbitai bot commented Dec 25, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Changes

Sequence Diagram(s)

Estimated code review effort

Possibly related issues

Possibly related PRs

Suggested reviewers

Poem

Pre-merge checks and finishing touches

Uh oh!

github-actions bot commented Dec 25, 2025

All File Checked ✅

Uh oh!

coderabbitai bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

coderabbitai bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

coderabbitai bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

leejh08 commented Dec 25, 2025 •

edited by coderabbitai bot

Loading

coderabbitai bot commented Dec 25, 2025 •

edited

Loading