update loadmore

Author: quanda-0562

app/src/main/java/com/example/moviedb/ui/base/loadmorerefresh/BaseLoadMoreRefreshViewModel.kt

@@ -2,7 +2,6 @@ package com.example.moviedb.ui.base.loadmorerefresh
 
 import com.example.moviedb.data.constant.Constants
 import com.example.moviedb.ui.base.BaseViewModel
-import com.example.moviedb.ui.widgets.EndlessRecyclerOnScrollListener
 import kotlinx.coroutines.flow.MutableStateFlow
 
 /**
@@ -15,20 +14,14 @@ abstract class BaseLoadMoreRefreshViewModel<Item>() : BaseViewModel() {
 
     // load more flag
     private val isLoadMore = MutableStateFlow(false)
+    private var loadMoreTimeMillis = 0L
 
     // current page
     private val currentPage = MutableStateFlow(getPreFirstPage())
 
     // last page flag
     private val isLastPage = MutableStateFlow(false)
 
-    // scroll listener for recycler view
-    val onScrollListener = object : EndlessRecyclerOnScrollListener(getLoadMoreThreshold()) {
-        override fun onLoadMore() {
-            doLoadMore()
-        }
-    }
-
     // item list
     val itemList = MutableStateFlow(arrayListOf<Item>())
 
@@ -70,20 +63,29 @@ abstract class BaseLoadMoreRefreshViewModel<Item>() : BaseViewModel() {
     /**
      * load first page
      */
-    protected fun refreshData() {
+    private fun refreshData() {
         loadData(getFirstPage())
     }
 
+    fun onBind(position: Int) {
+//        Timber.v("Check load more on $position")
+        if (itemList.value.size - position < getLoadMoreThreshold()) {
+            doLoadMore()
+        }
+    }
+
     fun doLoadMore() {
         when {
             isLoading()
                     || isRefreshing.value
                     || isLoadMore.value
-                    || isLastPage.value -> {
+                    || isLastPage.value
+                    || System.currentTimeMillis() - loadMoreTimeMillis < 2000 -> {
             }
 
             else -> {
                 isLoadMore.value = true
+                loadMoreTimeMillis = System.currentTimeMillis()
                 loadMore()
             }
         }
@@ -92,7 +94,7 @@ abstract class BaseLoadMoreRefreshViewModel<Item>() : BaseViewModel() {
     /**
      * load next page
      */
-    protected fun loadMore() {
+    private fun loadMore() {
         loadData(currentPage.value.plus(1))
     }
 
@@ -116,8 +118,7 @@ abstract class BaseLoadMoreRefreshViewModel<Item>() : BaseViewModel() {
     /**
      * reset load more
      */
-    fun resetLoadMore() {
-        onScrollListener.resetOnLoadMore()
+    private fun resetLoadMore() {
         isLastPage.value = false
     }
 
@@ -144,26 +145,21 @@ abstract class BaseLoadMoreRefreshViewModel<Item>() : BaseViewModel() {
         isLastPage.value = (items?.size ?: 0) < getNumberItemPerPage()
 
         // reset load
-//        hideLoading()
         isRefreshing.value = false
         isLoadMore.value = false
-
+        showSuccess()
         // check empty list
         checkEmptyList()
-        showSuccess()
     }
 
     /**
      * handle load fail
      */
     override suspend fun onError(throwable: Throwable) {
         super.onError(throwable)
-        onScrollListener.isLoading = false
-
         // reset load
         isRefreshing.value = false
         isLoadMore.value = false
-
         // check empty list
         checkEmptyList()
     }

app/src/main/java/com/example/moviedb/ui/screen/popularmovie/PopularMovieAdapter.kt

@@ -5,10 +5,12 @@ import com.example.moviedb.R
 import com.example.moviedb.data.model.Movie
 import com.example.moviedb.databinding.ItemMovieBinding
 import com.example.moviedb.ui.base.BaseListAdapter
+import com.example.moviedb.ui.base.BaseViewHolder
 import com.example.moviedb.utils.setSingleClick
 
 class PopularMovieAdapter(
-    val itemClickListener: (Movie) -> Unit = {}
+    val itemClickListener: (Movie) -> Unit = {},
+    val onBindPosition: (Int) -> Unit = {}
 ) : BaseListAdapter<Movie, ItemMovieBinding>(object : DiffUtil.ItemCallback<Movie>() {
     override fun areItemsTheSame(oldItem: Movie, newItem: Movie): Boolean {
         return oldItem.id == newItem.id
@@ -33,4 +35,8 @@ class PopularMovieAdapter(
         }
     }
 
+    override fun onBindViewHolder(holder: BaseViewHolder<ItemMovieBinding>, position: Int) {
+        super.onBindViewHolder(holder, position)
+        onBindPosition(position)
+    }
 }

app/src/main/java/com/example/moviedb/ui/screen/popularmovie/PopularMovieFragment.kt

@@ -22,7 +22,8 @@ class PopularMovieFragment :
     override val viewModel: PopularMovieViewModel by viewModels()
     override val listAdapter: BaseListAdapter<Movie, out ViewDataBinding> by lazy {
         PopularMovieAdapter(
-            itemClickListener = { toMovieDetail(it) }
+            itemClickListener = { toMovieDetail(it) },
+            onBindPosition = { viewModel.onBind(it) }
         )
     }
     override val swipeRefreshLayout: SwipeRefreshLayout

app/src/main/java/com/example/moviedb/ui/widgets/EndlessRecyclerOnScrollListener.kt

@@ -10,7 +10,6 @@ import android.widget.TextView
 import androidx.annotation.DrawableRes
 import androidx.core.widget.ContentLoadingProgressBar
 import androidx.databinding.BindingAdapter
-import androidx.recyclerview.widget.RecyclerView
 import androidx.swiperefreshlayout.widget.SwipeRefreshLayout
 import com.bumptech.glide.Glide
 import com.bumptech.glide.load.engine.DiskCacheStrategy
@@ -31,11 +30,6 @@ fun SwipeRefreshLayout.customRefreshing(refreshing: Boolean?) {
     isRefreshing = refreshing == true
 }
 
-@BindingAdapter("onScrollListener")
-fun RecyclerView.customScrollListener(listener: RecyclerView.OnScrollListener?) {
-    if (listener != null) addOnScrollListener(listener)
-}
-
 @BindingAdapter("glideSrc")
 fun ImageView.setGlideSrc(@DrawableRes src: Int?) {
     Glide.with(context).load(src).into(this)

app/src/main/java/com/example/moviedb/utils/BindingUtils.kt

@@ -30,8 +30,7 @@
             <androidx.recyclerview.widget.RecyclerView
                 android:id="@+id/recycler_view"
                 android:layout_width="match_parent"
-                android:layout_height="match_parent"
-                app:onScrollListener="@{ viewModel.onScrollListener }" />
+                android:layout_height="match_parent" />
 
         </androidx.swiperefreshlayout.widget.SwipeRefreshLayout>