[BOOK-437] refactor: 알림 설정 동기화 로직 단순화 및 유틸 함수로 분리

Author: seoyoon513

core/common/src/main/kotlin/com/ninecraft/booket/core/common/utils/NotificationSyncUtils.kt

@@ -0,0 +1,4 @@
+package com.ninecraft.booket.core.common.utils
+
+fun shouldSyncNotification(effectiveEnabled: Boolean, lastSynced: Boolean?): Boolean =
+    lastSynced == null || lastSynced != effectiveEnabled

feature/home/src/main/kotlin/com/ninecraft/booket/feature/home/HomePresenter.kt

@@ -7,6 +7,7 @@ import androidx.compose.runtime.rememberCoroutineScope
 import androidx.compose.runtime.setValue
 import com.ninecraft.booket.core.common.analytics.AnalyticsHelper
 import com.ninecraft.booket.core.common.utils.handleException
+import com.ninecraft.booket.core.common.utils.shouldSyncNotification
 import com.ninecraft.booket.core.data.api.repository.AuthRepository
 import com.ninecraft.booket.core.data.api.repository.BookRepository
 import com.ninecraft.booket.core.data.api.repository.UserRepository
@@ -115,14 +116,15 @@ class HomePresenter @AssistedInject constructor(
                 is HomeUiEvent.OnNotificationPermissionResult -> {
                     scope.launch {
                         val isPermissionGranted = event.granted
-                        val userEnabled = userRepository.getUserNotificationEnabled()
+                        val userSettingEnabled = userRepository.getUserNotificationEnabled()
                         val lastSyncedServerEnabled = userRepository.getLastSyncedNotificationEnabled()
 
-                        val shouldSync = (!isPermissionGranted && lastSyncedServerEnabled != false) ||
-                            (userEnabled && (lastSyncedServerEnabled == null || lastSyncedServerEnabled != isPermissionGranted))
+                        val effectiveNotificationEnabled = userSettingEnabled && isPermissionGranted
+
+                        val shouldSync = shouldSyncNotification(effectiveNotificationEnabled, lastSyncedServerEnabled)
 
                         if (shouldSync) {
-                            syncNotificationSettings(isPermissionGranted)
+                            syncNotificationSettings(effectiveNotificationEnabled)
                         }
                     }
                 }

feature/settings/src/main/kotlin/com/ninecraft/booket/feature/settings/notification/NotificationPresenter.kt

@@ -6,6 +6,7 @@ import androidx.compose.runtime.mutableStateOf
 import androidx.compose.runtime.rememberCoroutineScope
 import androidx.compose.runtime.setValue
 import com.ninecraft.booket.core.common.utils.handleException
+import com.ninecraft.booket.core.common.utils.shouldSyncNotification
 import com.ninecraft.booket.core.data.api.repository.UserRepository
 import com.ninecraft.booket.feature.screens.LoginScreen
 import com.ninecraft.booket.feature.screens.NotificationScreen
@@ -92,14 +93,15 @@ class NotificationPresenter @AssistedInject constructor(
                 is NotificationUiEvent.OnNotificationPermissionResult -> {
                     scope.launch {
                         val isPermissionGranted = event.granted
-                        val userEnabled = userRepository.getUserNotificationEnabled()
+                        val userSettingEnabled = userRepository.getUserNotificationEnabled()
                         val lastSyncedServerEnabled = userRepository.getLastSyncedNotificationEnabled()
 
-                        val shouldSync = (!isPermissionGranted && lastSyncedServerEnabled != false) ||
-                            (userEnabled && (lastSyncedServerEnabled == null || lastSyncedServerEnabled != isPermissionGranted))
+                        val effectiveNotificationEnabled = userSettingEnabled && isPermissionGranted
+
+                        val shouldSync = shouldSyncNotification(effectiveNotificationEnabled, lastSyncedServerEnabled)
 
                         if (shouldSync) {
-                            syncNotificationSettings(isPermissionGranted)
+                            syncNotificationSettings(effectiveNotificationEnabled)
                         }
                     }
                 }