create a better logging system

Author: Raival-e

app/src/main/AndroidManifest.xml

@@ -137,6 +137,13 @@
             </intent-filter>
         </activity>
 
+        <activity
+            android:name=".screen.logs.LogsActivity"
+            android:exported="false"
+            android:label="@string/title_activity_logs"
+            android:theme="@style/Theme.FileExplorer"
+            android:windowSoftInputMode="adjustResize" />
+
         <activity
             android:name=".screen.main.MainActivity"
             android:exported="true"

app/src/main/java/com/raival/compose/file/explorer/App.kt

@@ -3,7 +3,6 @@ package com.raival.compose.file.explorer
 import android.app.Application
 import android.content.Context
 import android.os.Process
-import android.util.Log
 import android.widget.Toast
 import androidx.annotation.StringRes
 import coil3.ImageLoader
@@ -19,9 +18,7 @@ import coil3.svg.SvgDecoder
 import coil3.video.VideoFrameDecoder
 import com.raival.compose.file.explorer.coil.apk.ApkFileDecoder
 import com.raival.compose.file.explorer.coil.pdf.PdfFileDecoder
-import com.raival.compose.file.explorer.common.extension.emptyString
-import com.raival.compose.file.explorer.common.extension.printFullStackTrace
-import com.raival.compose.file.explorer.common.extension.toFormattedDate
+import com.raival.compose.file.explorer.common.logger.FileExplorerLogger
 import com.raival.compose.file.explorer.screen.main.MainActivityManager
 import com.raival.compose.file.explorer.screen.main.tab.files.FilesTabManager
 import com.raival.compose.file.explorer.screen.main.tab.files.coil.DocumentFileMapper
@@ -38,6 +35,7 @@ import io.github.rosemoe.sora.langs.textmate.registry.model.ThemeModel
 import io.github.rosemoe.sora.langs.textmate.registry.provider.AssetsFileResolver
 import kotlinx.coroutines.CoroutineScope
 import kotlinx.coroutines.Dispatchers
+import kotlinx.coroutines.SupervisorJob
 import kotlinx.coroutines.launch
 import org.eclipse.tm4e.core.registry.IThemeSource
 import java.io.File
@@ -49,18 +47,21 @@ class App : Application(), coil3.SingletonImageLoader.Factory {
 
         val globalClass
             get() = appContext as App
+
+        val logger
+            get() = globalClass.logger
     }
 
+    private val applicationScope = CoroutineScope(SupervisorJob() + Dispatchers.IO)
+
+    lateinit var logger: FileExplorerLogger
+        private set
+
     val appFiles: LocalFileHolder
         get() = LocalFileHolder(
             File(globalClass.cacheDir, "files").apply { if (!exists()) mkdirs() }
         )
 
-    private val errorLogFile: LocalFileHolder
-        get() = LocalFileHolder(
-            File(globalClass.cacheDir, "logs.txt").apply { if (!exists()) createNewFile() }
-        )
-
     val recycleBinDir: LocalFileHolder
         get() = LocalFileHolder(File(getExternalFilesDir(null), "bin").apply { mkdirs() })
 
@@ -78,19 +79,24 @@ class App : Application(), coil3.SingletonImageLoader.Factory {
     val zipManager: ZipManager by lazy { ZipManager() }
 
     override fun onCreate() {
-        Thread.setDefaultUncaughtExceptionHandler { _: Thread?, throwable: Throwable? ->
-            throwable?.let {
-                Log.e("AppCrash", emptyString, it).also { log(throwable) }
-            }
-            Process.killProcess(Process.myPid())
-            exitProcess(2)
-        }
-
         super.onCreate()
 
+        logger = FileExplorerLogger(this, applicationScope)
+        setupGlobalExceptionHandler()
+
         appContext = this
     }
 
+    private fun setupGlobalExceptionHandler() {
+        val defaultHandler = Thread.getDefaultUncaughtExceptionHandler()
+        Thread.setDefaultUncaughtExceptionHandler { thread, exception ->
+            logger.logError(exception)
+            defaultHandler?.uncaughtException(thread, exception)
+            Process.killProcess(Process.myPid())
+            exitProcess(2)
+        }
+    }
+
     private fun setupTextMate() {
         CoroutineScope(Dispatchers.IO).launch {
             FileProviderRegistry.getInstance().addFileProvider(
@@ -142,33 +148,6 @@ class App : Application(), coil3.SingletonImageLoader.Factory {
 
     fun generateUid() = uid++
 
-    fun log(throwable: Throwable) {
-        log(throwable.printFullStackTrace(), "Error")
-    }
-
-    fun log(msg: String, header: String = emptyString) {
-        if (!errorLogFile.exists()) return
-        if (errorLogFile.isFolder) return
-
-        if (errorLogFile.size > 1024 * 100) {
-            errorLogFile.writeText(emptyString)
-        }
-
-        errorLogFile.appendText(
-            buildString {
-                if (header.isNotEmpty()) {
-                    append(System.lineSeparator().repeat(2))
-                    append("-".repeat(4))
-                    append(" $header: ${System.currentTimeMillis().toFormattedDate()} ")
-                    append("-".repeat(4))
-                    append(System.lineSeparator())
-                }
-                append(msg)
-                append(System.lineSeparator())
-            }
-        )
-    }
-
     override fun newImageLoader(context: PlatformContext): ImageLoader {
         return ImageLoader(context)
             .newBuilder()

app/src/main/java/com/raival/compose/file/explorer/common/extension/Extensions.kt

@@ -255,13 +255,13 @@ fun Uri.exists(context: Context): Boolean {
                 // which means the URI is valid and accessible.
                 return true
             }
-        } catch (e: FileNotFoundException) {
+        } catch (_: FileNotFoundException) {
             // The content provider reported that the file doesn't exist.
             return false
-        } catch (e: SecurityException) {
+        } catch (_: SecurityException) {
             // We don't have permission to read the URI.
             return false
-        } catch (e: Exception) {
+        } catch (_: Exception) {
             // Handle other potential exceptions, such as IllegalArgumentException for a malformed URI.
             return false
         }