[#146] 뷰모델의 State 구조체가 Equatable을 채택하도록 개선한다

opficdev · web-flow · commit 9056ee53c62c · 2026-03-03T23:05:19.000+09:00
diff --git a/DevLog/Domain/Entity/TodoQuery.swift b/DevLog/Domain/Entity/TodoQuery.swift
@@ -7,7 +7,7 @@
 
 import Foundation
 
-struct TodoQuery {
+struct TodoQuery: Equatable {
     enum SortTarget: Equatable, Hashable {
         case createdAt
         case updatedAt
diff --git a/DevLog/Presentation/ViewModel/AccountViewModel.swift b/DevLog/Presentation/ViewModel/AccountViewModel.swift
@@ -9,7 +9,7 @@ import Foundation
 
 @Observable
 final class AccountViewModel: Store {
-    struct State {
+    struct State: Equatable {
         var currentProvider: AuthProvider?
         var connectedProviders: [AuthProvider] = []
         var disconnectedProviders: [AuthProvider] = []
@@ -87,7 +87,7 @@ final class AccountViewModel: Store {
                 .filter { !allProviders.contains($0) }
         }
 
-        self.state = state
+        if self.state != state { self.state = state }
         return effects
     }
 
diff --git a/DevLog/Presentation/ViewModel/LoginViewModel.swift b/DevLog/Presentation/ViewModel/LoginViewModel.swift
@@ -12,7 +12,7 @@ import GoogleSignIn
 
 @Observable
 final class LoginViewModel: Store {
-    struct State {
+    struct State: Equatable {
         var signIn: Bool?
         var isLoading = false
         var showAlert: Bool = false
@@ -76,7 +76,7 @@ final class LoginViewModel: Store {
             state.signIn = result
         }
         
-        self.state = state
+        if self.state != state { self.state = state }
         return effects
     }
 
diff --git a/DevLog/Presentation/ViewModel/ProfileViewModel.swift b/DevLog/Presentation/ViewModel/ProfileViewModel.swift
@@ -9,7 +9,7 @@ import Foundation
 
 @Observable
 final class ProfileViewModel: Store {
-    struct State {
+    struct State: Equatable {
         var name: String = ""
         var email: String = ""
         var statusMessage: String = ""
@@ -185,7 +185,7 @@ final class ProfileViewModel: Store {
         case .updateStatusTextFieldFocus(let focused):
             state.showDoneButton = focused
         }
-        self.state = state
+        if self.state != state { self.state = state }
         return effects
     }
     // swiftlint:enable cyclomatic_complexity
diff --git a/DevLog/Presentation/ViewModel/PushNotificationListViewModel.swift b/DevLog/Presentation/ViewModel/PushNotificationListViewModel.swift
@@ -9,7 +9,7 @@ import Foundation
 
 @Observable
 final class PushNotificationListViewModel: Store {
-    struct State {
+    struct State: Equatable {
         var notifications: [PushNotificationItem] = []
         var showAlert: Bool = false
         var showToast: Bool = false
@@ -19,7 +19,6 @@ final class PushNotificationListViewModel: Store {
         var isLoading: Bool = false
         var hasMore: Bool = false
         var nextCursor: PushNotificationCursor?
-        var pendingTask: (PushNotificationItem, Int)?
         var query: PushNotificationQuery
         var selectedTodoID: TodoIDItem?
     }
@@ -57,6 +56,7 @@ final class PushNotificationListViewModel: Store {
     private let toggleReadUseCase: TogglePushNotificationReadUseCase
     private let fetchQueryUseCase: FetchPushNotificationQueryUseCase
     private let updateQueryUseCase: UpdatePushNotificationQueryUseCase
+    private var pendingTask: (PushNotificationItem, Int)?
 
     init(
         fetchUseCase: FetchPushNotificationsUseCase,
@@ -99,7 +99,7 @@ final class PushNotificationListViewModel: Store {
             effects = reduceByRun(action, state: &state)
         }
 
-        self.state = state
+        if self.state != state { self.state = state }
         return effects
     }
 
@@ -158,12 +158,12 @@ private extension PushNotificationListViewModel {
         switch action {
         case .deleteNotification(let item):
             var effects: [SideEffect] = []
-            if let (pendingItem, _) = state.pendingTask {
+            if let (pendingItem, _) = pendingTask {
                 effects = [.delete(pendingItem)]
             }
 
             if let index = state.notifications.firstIndex(where: { $0.id == item.id }) {
-                state.pendingTask = (item, index)
+                pendingTask = (item, index)
                 state.notifications.remove(at: index)
                 setToast(&state, isPresented: true)
             }
@@ -175,9 +175,9 @@ private extension PushNotificationListViewModel {
                 return [.toggleRead(item.todoID)]
             }
         case .undoDelete:
-            guard let (item, index) = state.pendingTask else { return [] }
+            guard let (item, index) = pendingTask else { return [] }
             state.notifications.insert(item, at: index)
-            state.pendingTask = nil
+            pendingTask = nil
         case .setAlert(let isPresented):
             setAlert(&state, isPresented: isPresented)
         case .toggleSortOption:
@@ -218,11 +218,11 @@ private extension PushNotificationListViewModel {
             state.nextCursor = nil
             return [.fetchNotifications(state.query, cursor: nil)]
         case .loadNextPage:
-            guard state.hasMore, !state.isLoading, state.pendingTask == nil else { return [] }
+            guard state.hasMore, !state.isLoading, pendingTask == nil else { return [] }
             return [.fetchNotifications(state.query, cursor: state.nextCursor)]
         case .confirmDelete:
-            guard let (item, _) = state.pendingTask else { return [] }
-            state.pendingTask = nil
+            guard let (item, _) = pendingTask else { return [] }
+            pendingTask = nil
             return [.delete(item)]
         case .setToast(let isPresented):
             setToast(&state, isPresented: isPresented)
@@ -245,7 +245,7 @@ private extension PushNotificationListViewModel {
             state.nextCursor = nil
         case .appendNotifications(let notifications, let nextCursor):
             let filteredNotifications: [PushNotificationItem]
-            if let (pendingItem, _) = state.pendingTask {
+            if let (pendingItem, _) = pendingTask {
                 filteredNotifications = notifications.filter { $0.id != pendingItem.id }
             } else {
                 filteredNotifications = notifications
diff --git a/DevLog/Presentation/ViewModel/PushNotificationSettingsViewModel.swift b/DevLog/Presentation/ViewModel/PushNotificationSettingsViewModel.swift
@@ -9,7 +9,7 @@ import Foundation
 
 @Observable
 final class PushNotificationSettingsViewModel: Store {
-    struct State {
+    struct State: Equatable {
         var pushNotificationEnable: Bool = false
         var viewPushNotificationTime: Date = .init()
         var sheetPushNotificationTime: Date = .init()
@@ -99,7 +99,7 @@ final class PushNotificationSettingsViewModel: Store {
             state.showTimePicker = false
             state.sheetPushNotificationTime = state.viewPushNotificationTime
         }
-        self.state = state
+        if self.state != state { self.state = state }
         return effects
     }
 
diff --git a/DevLog/Presentation/ViewModel/RootViewModel.swift b/DevLog/Presentation/ViewModel/RootViewModel.swift
@@ -10,7 +10,7 @@ import Combine
 
 @Observable
 final class RootViewModel: Store {
-    struct State {
+    struct State: Equatable {
         var showAlert: Bool = false
         var alertTitle: String = ""
         var alertMessage: String = ""
@@ -89,7 +89,7 @@ final class RootViewModel: Store {
             state.signIn = result
         }
         
-        self.state = state
+        if self.state != state { self.state = state }
         return effects
     }
     
diff --git a/DevLog/Presentation/ViewModel/SearchViewModel.swift b/DevLog/Presentation/ViewModel/SearchViewModel.swift
@@ -10,7 +10,7 @@ import OrderedCollections
 
 @Observable
 final class SearchViewModel: Store {
-    struct State {
+    struct State: Equatable {
         var isLoading: Bool = false
         var isSearching: Bool = false
         var searchQuery: String = ""
@@ -124,7 +124,7 @@ final class SearchViewModel: Store {
             state.showAllWebPages = shouldShowAll
         }
 
-        self.state = state
+        if self.state != state { self.state = state }
         return effects
     }
 
diff --git a/DevLog/Presentation/ViewModel/SettingViewModel.swift b/DevLog/Presentation/ViewModel/SettingViewModel.swift
@@ -10,7 +10,7 @@ import Combine
 
 @Observable
 final class SettingViewModel: Store {
-    struct State {
+    struct State: Equatable {
         var theme: SystemTheme = .automatic
         var dirSize: Int64 = 0
         var isLoading = false
@@ -95,7 +95,7 @@ final class SettingViewModel: Store {
             }
         }
 
-        self.state = state
+        if self.state != state { self.state = state }
         return effects
     }
 
diff --git a/DevLog/Presentation/ViewModel/TodoDetailViewModel.swift b/DevLog/Presentation/ViewModel/TodoDetailViewModel.swift
@@ -9,7 +9,7 @@ import Foundation
 
 @Observable
 final class TodoDetailViewModel: Store {
-    struct State {
+    struct State: Equatable {
         var todo: Todo?
         var isLoading: Bool  = false
         var showAlert: Bool  = false
@@ -70,7 +70,7 @@ final class TodoDetailViewModel: Store {
             effects = [.upsertTodo(todo)]
         }
 
-        self.state = state
+        if self.state != state { self.state = state }
         return effects
     }
 
diff --git a/DevLog/Presentation/ViewModel/TodoEditorViewModel.swift b/DevLog/Presentation/ViewModel/TodoEditorViewModel.swift
@@ -10,14 +10,14 @@ import OrderedCollections
 
 @Observable
 final class TodoEditorViewModel: Store {
-    struct State {
+    struct State: Equatable {
         var title: String = ""
         var content: String = ""
         var dueDate: Date?
         var tags: OrderedSet<String> = []
         var tagText: String = ""
         var focusOnEditor: Bool = false
-        var hasDueDate: Bool { return dueDate != nil }
+        var hasDueDate: Bool { dueDate != nil }
         var tabViewTag: Tag = .editor
         var isValidToSave: Bool {
             !title.trimmingCharacters(in: .whitespacesAndNewlines).isEmpty &&
@@ -111,7 +111,7 @@ final class TodoEditorViewModel: Store {
             }
         }
 
-        self.state = state
+        if self.state != state { self.state = state }
         return []
     }
 }
diff --git a/DevLog/Presentation/ViewModel/TodoListViewModel.swift b/DevLog/Presentation/ViewModel/TodoListViewModel.swift
diff --git a/DevLog/Presentation/ViewModel/TodoManageViewModel.swift b/DevLog/Presentation/ViewModel/TodoManageViewModel.swift

Original file line number	Diff line number	Diff line change
`@@ -9,7 +9,7 @@ import Foundation`
`9`	`9`
`10`	`10`	`@Observable`
`11`	`11`	`final class AccountViewModel: Store {`
`12`		`- struct State {`
	`12`	`+ struct State: Equatable {`
`13`	`13`	`var currentProvider: AuthProvider?`
`14`	`14`	`var connectedProviders: [AuthProvider] = []`
`15`	`15`	`var disconnectedProviders: [AuthProvider] = []`
`@@ -87,7 +87,7 @@ final class AccountViewModel: Store {`
`87`	`87`	`.filter { !allProviders.contains($0) }`
`88`	`88`	`}`
`89`	`89`
`90`		`- self.state = state`
	`90`	`+ if self.state != state { self.state = state }`
`91`	`91`	`return effects`
`92`	`92`	`}`
`93`	`93`
Original file line number	Diff line number	Diff line change
`@@ -12,7 +12,7 @@ import GoogleSignIn`
`12`	`12`
`13`	`13`	`@Observable`
`14`	`14`	`final class LoginViewModel: Store {`
`15`		`- struct State {`
	`15`	`+ struct State: Equatable {`
`16`	`16`	`var signIn: Bool?`
`17`	`17`	`var isLoading = false`
`18`	`18`	`var showAlert: Bool = false`
`@@ -76,7 +76,7 @@ final class LoginViewModel: Store {`
`76`	`76`	`state.signIn = result`
`77`	`77`	`}`
`78`	`78`
`79`		`- self.state = state`
	`79`	`+ if self.state != state { self.state = state }`
`80`	`80`	`return effects`
`81`	`81`	`}`
`82`	`82`
Original file line number	Diff line number	Diff line change
`@@ -10,7 +10,7 @@ import Combine`
`10`	`10`
`11`	`11`	`@Observable`
`12`	`12`	`final class RootViewModel: Store {`
`13`		`- struct State {`
	`13`	`+ struct State: Equatable {`
`14`	`14`	`var showAlert: Bool = false`
`15`	`15`	`var alertTitle: String = ""`
`16`	`16`	`var alertMessage: String = ""`
`@@ -89,7 +89,7 @@ final class RootViewModel: Store {`
`89`	`89`	`state.signIn = result`
`90`	`90`	`}`
`91`	`91`
`92`		`- self.state = state`
	`92`	`+ if self.state != state { self.state = state }`
`93`	`93`	`return effects`
`94`	`94`	`}`
`95`	`95`
Original file line number	Diff line number	Diff line change
`@@ -10,7 +10,7 @@ import OrderedCollections`
`10`	`10`
`11`	`11`	`@Observable`
`12`	`12`	`final class SearchViewModel: Store {`
`13`		`- struct State {`
	`13`	`+ struct State: Equatable {`
`14`	`14`	`var isLoading: Bool = false`
`15`	`15`	`var isSearching: Bool = false`
`16`	`16`	`var searchQuery: String = ""`
`@@ -124,7 +124,7 @@ final class SearchViewModel: Store {`
`124`	`124`	`state.showAllWebPages = shouldShowAll`
`125`	`125`	`}`
`126`	`126`
`127`		`- self.state = state`
	`127`	`+ if self.state != state { self.state = state }`
`128`	`128`	`return effects`
`129`	`129`	`}`
`130`	`130`