Add LogHelper with centralized log tags

Author: MihaiCristianCondrea

app/src/main/kotlin/com/d4rk/cleaner/app/clean/analyze/ui/AnalyzeScreen.kt

@@ -36,9 +36,10 @@ import com.d4rk.cleaner.app.clean.scanner.domain.data.model.ui.FileEntry
 import com.d4rk.cleaner.app.clean.scanner.domain.data.model.ui.UiScannerModel
 import com.d4rk.cleaner.app.clean.scanner.ui.ScannerViewModel
 import com.d4rk.cleaner.app.clean.scanner.ui.components.TwoRowButtons
+import com.d4rk.cleaner.core.utils.helpers.LogHelper
 import kotlinx.coroutines.CoroutineScope
 
-private const val TAG = "AnalyzeScreen"
+private const val TAG = LogHelper.ANALYZE_SCREEN
 
 @OptIn(ExperimentalAnimationApi::class)
 @Composable
@@ -69,17 +70,23 @@ fun AnalyzeScreen(
             when (data.analyzeState.state) {
 
                 CleaningState.Analyzing -> {
+                    Log.d(TAG, "Analyzing files")
                     LoadingScreen()
                 }
 
                 CleaningState.Cleaning -> {
+                    Log.d(
+                        TAG,
+                        "Cleaning: cleaned=${data.analyzeState.cleanedFilesCount} total=${data.analyzeState.totalFilesToClean}"
+                    )
                     CleaningAnimationScreen(
                         cleaned = data.analyzeState.cleanedFilesCount,
                         total = data.analyzeState.totalFilesToClean,
                     )
                 }
 
                 CleaningState.ReadyToClean -> {
+                    Log.d(TAG, "ReadyToClean: groups=${groupedFiles.size}")
                     if (groupedFiles.isNotEmpty()) {
                         TabsContent(
                             groupedFiles = groupedFiles,
@@ -89,22 +96,27 @@ fun AnalyzeScreen(
                             data = data,
                         )
                     } else {
+                        Log.d(TAG, "No files found")
                         NoFilesFoundScreen(viewModel = viewModel)
                     }
                 }
 
                 CleaningState.Result -> {
+                    Log.d(TAG, "Showing results")
                     NoFilesFoundScreen(viewModel = viewModel)
                 }
 
                 CleaningState.Error -> {
+                    Log.e(TAG, "Error state encountered")
                     ErrorScreen(onRetry = {
                         viewModel.resetAfterError()
                         viewModel.onEvent(ScannerEvent.AnalyzeFiles)
                     })
                 }
 
-                CleaningState.Idle -> {}
+                CleaningState.Idle -> {
+                    Log.d(TAG, "Idle state")
+                }
             }
         }
         if (groupedFiles.isNotEmpty() && data.analyzeState.state == CleaningState.ReadyToClean) {

app/src/main/kotlin/com/d4rk/cleaner/app/clean/contacts/domain/usecases/GetDuplicateContactsUseCase.kt

@@ -6,10 +6,11 @@ import com.d4rk.cleaner.app.clean.contacts.data.ContactsRepository
 import com.d4rk.cleaner.app.clean.contacts.domain.data.model.RawContactInfo
 import com.d4rk.cleaner.core.domain.model.network.Errors
 import com.d4rk.cleaner.core.utils.extensions.toError
+import com.d4rk.cleaner.core.utils.helpers.LogHelper
 import kotlinx.coroutines.flow.Flow
 import kotlinx.coroutines.flow.flow
 
-private const val TAG = "GetDuplicateContactsUseCase"
+private const val TAG = LogHelper.GET_DUPLICATE_CONTACTS_USE_CASE
 
 class GetDuplicateContactsUseCase(private val repository: ContactsRepository) {
     operator fun invoke(): Flow<DataState<List<List<RawContactInfo>>, Errors>> = flow {

app/src/main/kotlin/com/d4rk/cleaner/app/clean/contacts/ui/ContactsCleanerViewModel.kt

@@ -17,6 +17,7 @@ import com.d4rk.cleaner.app.clean.contacts.domain.usecases.DeleteOlderContactsUs
 import com.d4rk.cleaner.app.clean.contacts.domain.usecases.GetDuplicateContactsUseCase
 import com.d4rk.cleaner.app.clean.contacts.domain.usecases.MergeContactsUseCase
 import com.d4rk.cleaner.core.utils.extensions.asUiText
+import com.d4rk.cleaner.core.utils.helpers.LogHelper
 import kotlinx.coroutines.flow.collectLatest
 import kotlinx.coroutines.flow.update
 
@@ -31,7 +32,7 @@ class ContactsCleanerViewModel(
 ) {
 
     companion object {
-        private const val TAG = "ContactsCleanerViewModel"
+        private const val TAG = LogHelper.CONTACTS_CLEANER_VIEW_MODEL
     }
 
     init {

app/src/main/kotlin/com/d4rk/cleaner/app/clean/dashboard/ui/ScannerDashboardScreen.kt

@@ -54,6 +54,7 @@ import com.d4rk.cleaner.app.clean.dashboard.ui.components.DashboardActionCard
 import com.d4rk.cleaner.app.clean.whatsapp.summary.ui.WhatsAppCleanerActivity
 import com.d4rk.cleaner.app.images.picker.ui.ImagePickerActivity
 import com.d4rk.cleaner.core.data.datastore.DataStore
+import com.d4rk.cleaner.core.utils.helpers.LogHelper
 import com.google.android.gms.common.ConnectionResult
 import com.google.android.gms.common.GoogleApiAvailability
 import org.koin.compose.koinInject
@@ -81,7 +82,7 @@ private sealed interface HomeItem {
     data class Ad(val slot: AdSlot) : HomeItem
 }
 
-private const val TAG = "ScannerDashboardScreen"
+private const val TAG = LogHelper.SCANNER_DASHBOARD_SCREEN
 
 @Composable
 fun ScannerDashboardScreen(

app/src/main/kotlin/com/d4rk/cleaner/app/clean/largefiles/ui/LargeFilesScreen.kt

@@ -28,10 +28,11 @@ import com.d4rk.cleaner.app.clean.analyze.ui.components.FilesByDateSection
 import com.d4rk.cleaner.app.clean.largefiles.domain.actions.LargeFilesEvent
 import com.d4rk.cleaner.app.clean.largefiles.domain.data.model.ui.UiLargeFilesModel
 import com.d4rk.cleaner.app.clean.scanner.domain.data.model.ui.CleaningState
+import com.d4rk.cleaner.core.utils.helpers.LogHelper
 import org.koin.compose.viewmodel.koinViewModel
 import java.io.File
 
-private const val TAG = "LargeFilesScreen"
+private const val TAG = LogHelper.LARGE_FILES_SCREEN
 @OptIn(ExperimentalMaterial3Api::class)
 @Composable
 fun LargeFilesScreen(activity: LargeFilesActivity) {

app/src/main/kotlin/com/d4rk/cleaner/app/clean/scanner/work/FileCleanupWorker.kt

@@ -20,6 +20,7 @@ import com.d4rk.cleaner.app.clean.scanner.domain.usecases.DeleteFilesUseCase
 import com.d4rk.cleaner.core.domain.model.network.Errors
 import com.d4rk.cleaner.core.utils.helpers.CleaningEventBus
 import com.d4rk.cleaner.core.utils.helpers.isProtectedAndroidDir
+import com.d4rk.cleaner.core.utils.helpers.LogHelper
 import com.google.android.material.color.MaterialColors
 import org.koin.core.component.KoinComponent
 import org.koin.core.component.inject
@@ -286,6 +287,6 @@ class FileCleanupWorker(
         const val MAX_PATHS_PER_WORKER = 100
         private const val NOTIFICATION_ID = 2001
         private const val NOTIFICATION_CHANNEL = "file_cleanup"
-        private const val TAG = "FileCleanupWorker"
+        private const val TAG = LogHelper.FILE_CLEANUP_WORKER
     }
 }

app/src/main/kotlin/com/d4rk/cleaner/core/utils/helpers/LogHelper.kt

@@ -0,0 +1,10 @@
+package com.d4rk.cleaner.core.utils.helpers
+
+object LogHelper {
+    const val CONTACTS_CLEANER_VIEW_MODEL = "ContactsCleanerViewModel"
+    const val GET_DUPLICATE_CONTACTS_USE_CASE = "GetDuplicateContactsUseCase"
+    const val ANALYZE_SCREEN = "AnalyzeScreen"
+    const val SCANNER_DASHBOARD_SCREEN = "ScannerDashboardScreen"
+    const val LARGE_FILES_SCREEN = "LargeFilesScreen"
+    const val FILE_CLEANUP_WORKER = "FileCleanupWorker"
+}