Chore: Improve BottomNavigationView switch logic

imknown · imknown · commit fc92aa84f017 · 2025-08-05T09:40:09.000+08:00
Signed-off-by: imknown &lt;imknown@qq.com&gt;
diff --git a/app/src/main/java/net/imknown/android/forefrontinfo/ui/MainActivity.kt b/app/src/main/java/net/imknown/android/forefrontinfo/ui/MainActivity.kt
@@ -88,15 +88,15 @@ class MainActivity : AppCompatActivity() {
         setSupportActionBar(binding.toolbar)
 
         binding.bottomNavigationView.setOnItemSelectedListener {
-            supportFragmentManager.switch(it.itemId)
+            supportFragmentManager.switch(it.itemId, false)
 
             true
         }
     }
 
-    private fun FragmentManager.switch(@IdRes selectedId: Int, isFirst: Boolean = false) {
+    private fun FragmentManager.switch(@IdRes selectedId: Int, isFirstTime: Boolean) {
         val lastId = mainViewModel.lastId
-        if (selectedId == lastId && !isFirst) {
+        if (selectedId == lastId && !isFirstTime) {
             return
         }
 

Original file line number	Diff line number	Diff line change
`@@ -88,15 +88,15 @@ class MainActivity : AppCompatActivity() {`
`88`	`88`	`setSupportActionBar(binding.toolbar)`
`89`	`89`
`90`	`90`	`binding.bottomNavigationView.setOnItemSelectedListener {`
`91`		`- supportFragmentManager.switch(it.itemId)`
	`91`	`+ supportFragmentManager.switch(it.itemId, false)`
`92`	`92`
`93`	`93`	`true`
`94`	`94`	`}`
`95`	`95`	`}`
`96`	`96`
`97`		`- private fun FragmentManager.switch(@IdRes selectedId: Int, isFirst: Boolean = false) {`
	`97`	`+ private fun FragmentManager.switch(@IdRes selectedId: Int, isFirstTime: Boolean) {`
`98`	`98`	`val lastId = mainViewModel.lastId`
`99`		`- if (selectedId == lastId && !isFirst) {`
	`99`	`+ if (selectedId == lastId && !isFirstTime) {`
`100`	`100`	`return`
`101`	`101`	`}`
`102`	`102`