优化

app/src/main/java/io/legado/app/ui/book/manga/ReadMangaActivity.kt

@@ -119,7 +119,7 @@ class ReadMangaActivity : VMBaseActivity<ActivityMangaBinding, ReadMangaViewMode
     private var justInitData: Boolean = false
     private var syncDialog: AlertDialog? = null
     private val mScrollTimer by lazy {
-        ScrollTimer(lifecycleScope, this).apply {
+        ScrollTimer(this, binding.mRecyclerManga).apply {
             setSpeed(AppConfig.mangaAutoPageSpeed)
         }
     }

app/src/main/java/io/legado/app/ui/book/manga/recyclerview/ScrollTimer.kt

@@ -1,46 +1,44 @@
 package io.legado.app.ui.book.manga.recyclerview
 
-import kotlinx.coroutines.CoroutineScope
+import androidx.recyclerview.widget.RecyclerView
-import kotlinx.coroutines.Job
+import androidx.recyclerview.widget.RecyclerView.SCROLL_STATE_IDLE
-import kotlinx.coroutines.delay
-import kotlinx.coroutines.isActive
-import kotlinx.coroutines.launch
 
 class ScrollTimer(
-    private val coroutineScope: CoroutineScope,
+    private val callback: ScrollCallback,
-    private val callback: ScrollCallback
+    private val recyclerView: RecyclerView
-) {
+) : RecyclerView.OnScrollListener() {
-    private var job: Job? = null
-    private var delayMs: Long = 20L
     private var distance = 1
     var isEnabled: Boolean = false
         set(value) {
             if (field != value) {
                 field = value
-                restartJob()
+                if (value) {
+                    recyclerView.addOnScrollListener(this)
+                    startScroll()
+                } else {
+                    recyclerView.removeOnScrollListener(this)
+                }
             }
         }
 
+    override fun onScrollStateChanged(
+        recyclerView: RecyclerView,
+        newState: Int
+    ) {
+        if (newState == SCROLL_STATE_IDLE) {
+            startScroll()
+        }
+    }
+
     fun setSpeed(distance: Int) {
         this.distance = distance
-        restartJob()
     }
 
-    private fun restartJob() {
+    private fun startScroll() {
-        job?.cancel()
+        callback.scrollBy(distance)
-        if (!isEnabled || delayMs == 0L) {
-            job = null
-            return
-        }
-        job = coroutineScope.launch {
-            while (isActive) {
-                callback.scrollBy(distance)
-                delay(delayMs)
-            }
-        }
     }
 
-    interface ScrollCallback{
+    interface ScrollCallback {
-        fun scrollBy(distance:Int)
+        fun scrollBy(distance: Int)
     }
 }

app/src/main/java/io/legado/app/ui/book/read/MangaMenu.kt

@@ -233,7 +233,7 @@ class MangaMenu @JvmOverloads constructor(
     interface CallBack {
         fun openBookInfoActivity()
         fun upSystemUiVisibility(menuIsVisible: Boolean)
-        fun skipToPage(pos: Int)
+        fun skipToPage(index: Int)
     }
 
 }