From 8fa9dfc809174d4b88e5dca53d80ecedeb530ec1 Mon Sep 17 00:00:00 2001
From: acite <1498045907@qq.com>
Date: Mon, 25 Aug 2025 17:22:09 +0800
Subject: [PATCH] [optimize] Optimize API call logic, do not create crashes

---
 .../acitelight/aether/service/MediaManager.kt | 34 ++++++++++++++-----
 .../aether/service/RecentManager.kt           | 13 ++++---
 .../com/acitelight/aether/view/VideoPlayer.kt |  6 ++++
 .../aether/viewModel/VideoPlayerViewModel.kt  |  2 +-
 4 files changed, 41 insertions(+), 14 deletions(-)

diff --git a/app/src/main/java/com/acitelight/aether/service/MediaManager.kt b/app/src/main/java/com/acitelight/aether/service/MediaManager.kt
index e73a426..2178a42 100644
--- a/app/src/main/java/com/acitelight/aether/service/MediaManager.kt
+++ b/app/src/main/java/com/acitelight/aether/service/MediaManager.kt
@@ -13,22 +13,38 @@ object MediaManager
 
     suspend fun listVideoKlasses(): List<String>
     {
-        val j = ApiClient.api.getVideoClasses(token)
-        return j.toList()
+        try
+        {
+            val j = ApiClient.api.getVideoClasses(token)
+            return j.toList()
+        }catch(e: Exception)
+        {
+            return listOf()
+        }
     }
 
     suspend fun listVideos(klass: String): List<Video>
     {
-        val j = ApiClient.api.queryVideoClasses(klass, token)
-        return j.map{
-            queryVideo(klass, it)
-        }.toList()
+        try {
+            val j = ApiClient.api.queryVideoClasses(klass, token)
+            return j.map{
+                queryVideo(klass, it)!!
+            }.toList()
+        }catch (e: Exception)
+        {
+            return listOf()
+        }
     }
 
-    suspend fun queryVideo(klass: String, id: String): Video
+    suspend fun queryVideo(klass: String, id: String): Video?
     {
-        val j = ApiClient.api.queryVideo(klass, id, token)
-        return Video(klass = klass, id = id, token=token, j)
+        try {
+            val j = ApiClient.api.queryVideo(klass, id, token)
+            return Video(klass = klass, id = id, token=token, j)
+        }catch (e: Exception)
+        {
+            return null
+        }
     }
 
     suspend fun listComics() : List<String>
diff --git a/app/src/main/java/com/acitelight/aether/service/RecentManager.kt b/app/src/main/java/com/acitelight/aether/service/RecentManager.kt
index 0a11269..64b23fc 100644
--- a/app/src/main/java/com/acitelight/aether/service/RecentManager.kt
+++ b/app/src/main/java/com/acitelight/aether/service/RecentManager.kt
@@ -50,9 +50,11 @@ object RecentManager
         try{
             val r = Json.decodeFromString<List<VideoQueryIndex>>(content)
 
-            _recent.value = r.map{
+            val vn = r.map{
                 MediaManager.queryVideo(it.klass, it.id)
-            }
+            }.filter { it != null }
+
+            _recent.value = vn.map { it!! }
             return r
         }catch (e: Exception)
         {
@@ -82,9 +84,12 @@ object RecentManager
 
             if(o.size >= 21)
                 o.removeAt(o.size - 1)
-            _recent.value = o.map{
+
+            val vn = o.map{
                 MediaManager.queryVideo(it.klass, it.id)
-            }
+            }.filter { it != null }
+            _recent.value = vn.map { it!! }
+
             writeFile(context, "recent.json", Json.encodeToString(o))
         }
     }
diff --git a/app/src/main/java/com/acitelight/aether/view/VideoPlayer.kt b/app/src/main/java/com/acitelight/aether/view/VideoPlayer.kt
index 3f2f895..37a5c2b 100644
--- a/app/src/main/java/com/acitelight/aether/view/VideoPlayer.kt
+++ b/app/src/main/java/com/acitelight/aether/view/VideoPlayer.kt
@@ -914,6 +914,7 @@ fun VideoPlayerLandscape(videoPlayerViewModel: VideoPlayerViewModel)
 @Composable
 fun MiniVideoCard(modifier: Modifier, video: Video, onClick: () -> Unit)
 {
+    var isImageLoaded by remember { mutableStateOf(false) }
     Card(
         modifier = modifier.height(80.dp).fillMaxWidth(),
         colors = CardColors(
@@ -932,6 +933,11 @@ fun MiniVideoCard(modifier: Modifier, video: Video, onClick: () -> Unit)
                     .data(video.getCover())
                     .memoryCacheKey("${video.klass}/${video.id}/cover")
                     .diskCacheKey("${video.klass}/${video.id}/cover")
+                    .listener(
+                        onStart = { },
+                        onSuccess = { _, _ -> isImageLoaded = true },
+                        onError = { _, _ ->  }
+                    )
                     .build(),
                 contentDescription = null,
                 modifier = Modifier
diff --git a/app/src/main/java/com/acitelight/aether/viewModel/VideoPlayerViewModel.kt b/app/src/main/java/com/acitelight/aether/viewModel/VideoPlayerViewModel.kt
index 7567c86..7cf0768 100644
--- a/app/src/main/java/com/acitelight/aether/viewModel/VideoPlayerViewModel.kt
+++ b/app/src/main/java/com/acitelight/aether/viewModel/VideoPlayerViewModel.kt
@@ -55,7 +55,7 @@ class VideoPlayerViewModel() : ViewModel()
 
         remember {
             viewModelScope.launch {
-                video = MediaManager.queryVideo(v.split("/")[0], v.split("/")[1])
+                video = MediaManager.queryVideo(v.split("/")[0], v.split("/")[1])!!
                 RecentManager.Push(context, VideoQueryIndex(v.split("/")[0], v.split("/")[1]))
                 _player = ExoPlayer.Builder(context).build().apply {
                     val url = video?.getVideo() ?: ""