[#486] TodayViewModel 생성 및 생명주기를 개선한다 (#496)

* fix: Today 데이터 fetch 시점을 탭 선택 상태 기준으로 변경

* fix: 초기 탭 선택 상태의 데이터 fetch 누락 방지

* refactor: Home 데이터 fetch 액션명 정리

Author: opficdev

Application/DevLogPresentation/Sources/Home/Home/HomeViewCoordinator.swift

@@ -45,8 +45,8 @@ final class HomeViewCoordinator {
         )
     }
 
-    func loadInitialData() {
-        viewModel.send(.loadInitialData)
+    func fetchData() {
+        viewModel.send(.fetchData)
     }
 
     func makeTodoManageViewModel() -> TodoManageViewModel {

Application/DevLogPresentation/Sources/Home/Home/HomeViewModel.swift

@@ -38,7 +38,7 @@ final class HomeViewModel: Store {
     }
 
     enum Action {
-        case loadInitialData
+        case fetchData
         case networkStatusChanged(Bool)
         case setPresentation(Presentation, Bool)
         case setAlert(isPresented: Bool, type: AlertType? = nil)
@@ -145,7 +145,7 @@ final class HomeViewModel: Store {
         switch action {
         case .networkStatusChanged(let isConnected):
             state.isNetworkConnected = isConnected
-        case .loadInitialData, .setPresentation, .setAlert, .setToast, .refreshWebPages,
+        case .fetchData, .setPresentation, .setAlert, .setToast, .refreshWebPages,
                 .tapTodoCategory, .orderTodoCategory, .addTodo, .updateWebPageURLInput,
                 .addWebPage, .deleteWebPage, .undoDeleteWebPage:
             effects = reduceByView(action, state: &state)
@@ -274,7 +274,7 @@ private extension HomeViewModel {
     // swiftlint:disable cyclomatic_complexity
     func reduceByView(_ action: Action, state: inout State) -> [SideEffect] {
         switch action {
-        case .loadInitialData:
+        case .fetchData:
             return [.fetchTodoCategoryPreferences, .fetchRecentTodos, .fetchWebPages]
         case .refreshWebPages:
             return [.fetchWebPages]

Application/DevLogPresentation/Sources/Main/MainView.swift

@@ -41,9 +41,11 @@ struct MainView: View {
         .onAppear {
             coordinator.mainViewModel.send(.onAppear)
         }
-        .onChange(of: selectedTab) { _, newValue in
+        .onChange(of: selectedTab, initial: true) { _, newValue in
             if newValue == .home {
-                homeViewCoordinator.loadInitialData()
+                homeViewCoordinator.fetchData()
+            } else if newValue == .today {
+                todayViewCoordinator.fetchData()
             }
         }
         .alert(

Application/DevLogPresentation/Sources/Today/TodayView.swift

@@ -29,7 +29,6 @@ struct TodayView: View {
         .toolbar { toolbarContent }
         .background(NavigationBarConfigurator())
         .refreshable { coordinator.viewModel.send(.refresh) }
-        .onAppear { coordinator.viewModel.send(.onAppear) }
         .alert(
             coordinator.viewModel.state.alertTitle,
             isPresented: Binding(

Application/DevLogPresentation/Sources/Today/TodayViewCoordinator.swift

@@ -24,4 +24,8 @@ final class TodayViewCoordinator {
             updateTodayDisplayOptionsUseCase: container.resolve(UpdateTodayDisplayOptionsUseCase.self)
         )
     }
+
+    func fetchData() {
+        viewModel.send(.fetchData)
+    }
 }

Application/DevLogPresentation/Sources/Today/TodayViewModel.swift

@@ -62,7 +62,7 @@ final class TodayViewModel: Store {
         case resetDisplayOptions
         case completeTodo(TodayTodoItem)
         case togglePinned(TodayTodoItem)
-        case onAppear
+        case fetchData
         case fetchTodos([TodayTodoItem])
         case setLoading(Bool)
         case updateTodo(TodayTodoItem)
@@ -178,7 +178,7 @@ final class TodayViewModel: Store {
         case .refresh, .setAlert, .setSectionScope, .setDueDateVisibility, .setFocusVisibility,
                 .resetDisplayOptions, .completeTodo, .togglePinned:
             effects = reduceByUser(action, state: &state)
-        case .onAppear:
+        case .fetchData:
             effects = reduceByView(action, state: &state)
         case .fetchTodos, .setLoading, .updateTodo, .removeTodo:
             effects = reduceByRun(action, state: &state)
@@ -305,7 +305,7 @@ private extension TodayViewModel {
 
     func reduceByView(_ action: Action, state: inout State) -> [SideEffect] {
         switch action {
-        case .onAppear:
+        case .fetchData:
             return [.fetchTodos]
         default:
             break