refactor: reduce 메서드 분리 및 얼럿 로직 추가

Author: opficdev

DevLog/Presentation/ViewModel/HomeViewModel.swift

@@ -9,45 +9,35 @@ import Foundation
 
 final class HomeViewModel: Store {
     struct State {
-        // UI
         var todoKindPreferences = TodoKind.allCases.map { TodoKindPreference(kind: $0, isVisible: true) }
         var pinnedTodos: [Todo] = []
         var showTodoKindPicker: Bool = false
         var showTodoEditor: Bool = false
         var showSearchView: Bool = false
         var selectedTodoKind: TodoKind?
-
-        // User Input
         var searchText: String = ""
         var isSearching: Bool = false
         var reorderTodo: Bool = false
-
-        // Side Effect UI
         var isLoading: Bool = false
-        var toastMessage: String = ""
-        var showToast: Bool = false
+        var showAlert: Bool = false
+        var alertTitle: String = ""
+        var alertMessage: String = ""
     }
 
     enum Action {
-        // Life Cycle
-        case onAppear
-
-        // User
         case tapTodoKind(TodoKind)
-        case upsertTodo(Todo)
         case orderTodoKindPreferences([TodoKindPreference])
-
-        // Binding
-        case updateSearching(Bool)
-        case updateSearchText(String)
         case setReorderTodo(Bool)
         case setShowTodoEditor(Bool)
         case setShowTodoKindPicker(Bool)
         case setShowSearchView(Bool)
-        case setShowToast(Bool)
-
-        // Call from run
-        case didFetchPinnedTodos([Todo])
+        case setAlert(Bool)
+        case onAppear
+        case updateSearching(Bool)
+        case updateSearchText(String)
+        case upsertTodo(Todo)
+        case fetchPinnedTodos([Todo])
+        case setLoading(Bool)
     }
 
     enum SideEffect {
@@ -69,53 +59,115 @@ final class HomeViewModel: Store {
 
     func reduce(with action: Action) -> [SideEffect] {
         var state = self.state
+        var effects: [SideEffect] = []
+
         switch action {
-        case .onAppear:
-            return [.fetchPinnedTodos]
-        case .tapTodoKind(let kind):
-            state.selectedTodoKind = kind
-            state.showTodoKindPicker = false
-            state.showTodoEditor = true
-        case .updateSearching(let value):
-            state.isSearching = value
-        case .updateSearchText(let value):
-            state.searchText = value
-        case .setReorderTodo(let value):
-            state.reorderTodo = value
-        case .setShowTodoEditor(let value):
-            state.showTodoEditor = value
-            if !value {
-                state.selectedTodoKind = nil
-            }
-        case .setShowTodoKindPicker(let value):
-            state.showTodoKindPicker = value
-        case .setShowSearchView(let value):
-            state.showSearchView = value
-        case .setShowToast(let value):
-            state.showToast = value
-        case .upsertTodo(let value):
-            return [.upsertTodo(value)]
-        case .orderTodoKindPreferences(let value):
-            state.todoKindPreferences = value
-        case .didFetchPinnedTodos(let todos):
-            state.pinnedTodos = todos
+        case .tapTodoKind, .orderTodoKindPreferences, .setReorderTodo,
+                .setShowTodoEditor, .setShowTodoKindPicker, .setShowSearchView, .setAlert:
+            effects = reduceByUser(action, state: &state)
+
+        case .onAppear, .updateSearching, .updateSearchText, .upsertTodo:
+            effects = reduceByView(action, state: &state)
+
+        case .fetchPinnedTodos, .setLoading:
+            effects = reduceByRun(action, state: &state)
         }
 
         self.state = state
-        return []
+        return effects
     }
 
     func run(_ effect: SideEffect) {
         switch effect {
         case .upsertTodo(let todo):
             Task {
-                try await upsertTodoUseCase.execute(todo)
+                do {
+                    defer { send(.setLoading(false)) }
+                    send(.setLoading(true))
+                    try await upsertTodoUseCase.execute(todo)
+                } catch {
+                    send(.setAlert(true))
+                }
             }
         case .fetchPinnedTodos:
             Task {
-                let todos = try await fetchPinnedTodosUseCase.execute()
-                send(.didFetchPinnedTodos(todos))
+                do {
+                    defer { send(.setLoading(false)) }
+                    send(.setLoading(true))
+                    let todos = try await fetchPinnedTodosUseCase.execute()
+                    send(.fetchPinnedTodos(todos))
+                } catch {
+                    send(.setAlert(true))
+                }
             }
         }
     }
 }
+
+// MARK: - Reduce Methods
+private extension HomeViewModel {
+    func reduceByUser(_ action: Action, state: inout State) -> [SideEffect] {
+        switch action {
+        case .tapTodoKind(let kind):
+            state.selectedTodoKind = kind
+            state.showTodoKindPicker = false
+            state.showTodoEditor = true
+        case .orderTodoKindPreferences(let preferences):
+            state.todoKindPreferences = preferences
+        case .setReorderTodo(let isPresented):
+            state.reorderTodo = isPresented
+        case .setShowTodoEditor(let isPresented):
+            state.showTodoEditor = isPresented
+            if !isPresented { state.selectedTodoKind = nil }
+        case .setShowTodoKindPicker(let isPresented):
+            state.showTodoKindPicker = isPresented
+        case .setShowSearchView(let isPresented):
+            state.showSearchView = isPresented
+        case .setAlert(let isPresented):
+            setAlert(&state, isPresented: isPresented)
+        default:
+            break
+        }
+        return []
+    }
+
+    func reduceByView(_ action: Action, state: inout State) -> [SideEffect] {
+        switch action {
+        case .onAppear:
+            return [.fetchPinnedTodos]
+        case .updateSearching(let isSearching):
+            state.isSearching = isSearching
+        case .updateSearchText(let text):
+            state.searchText = text
+        case .upsertTodo(let todo):
+            return [.upsertTodo(todo)]
+        default:
+            break
+        }
+        return []
+    }
+
+    func reduceByRun(_ action: Action, state: inout State) -> [SideEffect] {
+        switch action {
+        case .fetchPinnedTodos(let todos):
+            state.pinnedTodos = todos
+        case .setLoading(let isLoading):
+            state.isLoading = isLoading
+        default:
+            break
+        }
+        return []
+    }
+}
+
+// MARK: - Helper Methods
+private extension HomeViewModel {
+    func setAlert(
+        _ state: inout State,
+        isPresented: Bool
+    ) {
+        state.alertTitle = "오류"
+        state.alertMessage = "문제가 발생했습니다. 잠시 후 다시 시도해주세요."
+        state.showAlert = isPresented
+    }
+}