Fix Bug Exception on Scroll, Fix Bug replace all some views stuck, Fix bug null layout param

Author: ramanaptr

app/src/main/java/com/ramanaptr/sample/MainActivity.kt

@@ -33,7 +33,7 @@ class MainActivity : AppCompatActivity() {
     /**
      * turn "isSingleLayout" into "false" when you want to try multi layout stuff
      * */
-    private var isSingleLayout = false
+    private var isSingleLayout = true
 
     override fun onCreate(savedInstanceState: Bundle?) {
         super.onCreate(savedInstanceState)
@@ -70,7 +70,7 @@ class MainActivity : AppCompatActivity() {
         binding.srl.setOnRefreshListener {
             binding.srl.isRefreshing = false
             currentPage = 0
-            rvSample.removeAll()
+            rvSample.reset()
             when {
                 isSingleLayout -> {
                     exampleDataForSingleLayout(15)
@@ -140,7 +140,7 @@ class MainActivity : AppCompatActivity() {
         }
 
         // this is just dummy data
-        exampleDataForSingleLayout(5)
+        exampleDataForSingleLayout(20)
     }
 
     private fun exampleEzRecycleMultipleLayout() {
@@ -219,7 +219,7 @@ class MainActivity : AppCompatActivity() {
                 // Example handling data in single layout
                 val dataList = arrayListOf<SampleData>().apply {
 
-                     // if (currentPage >= 2) return@apply // limit the page for test stop populate the data
+//                    if (currentPage >= 2) return@apply // limit the page for test stop populate the data
                     for (i in 1..size) {
                         add(SampleData("Key $i", "Value $i"))
                     }
@@ -259,7 +259,7 @@ class MainActivity : AppCompatActivity() {
                 // populate the data list
                 val dataList = arrayListOf<SampleData>().apply {
 
-//                     if (currentPage >= 2) return@apply // limit the page for test stop populate the data
+//                    if (currentPage >= 2) return@apply // limit the page for test stop populate the data
                     for (i in 1..size) {
 
                         // example handling data using ezViewType in layout 2

ez-recyleview/src/main/java/com/ramanaptr/widget/EzRecyclerView.java

@@ -43,7 +43,9 @@ public class EzRecyclerView<Data extends EzBaseData> extends RecyclerView {
     private int startShimmerSize = 0;
     private int endShimmerSize = 0;
     private int offset = 0;
+    private int offsetTemp = 0;
     private int limit = 0;
+    private int limitTemp = 0;
     private int currentPage = 0;
     private Data data;
 
@@ -206,38 +208,38 @@ public void startShimmer(int shimmerSize, Data data) {
     }
 
     public void hideShimmer() {
-        if (endShimmerSize > 0) {
-            baseAdapter.removeRange(startShimmerSize, (endShimmerSize + startShimmerSize));
-            endShimmerSize = 0;
-            endShimmerSize = tempShimmerSize;
-            flagEzRecyclerViewFirstLoad();
-            flagShimmerStop();
-
-            // Remove all views when data list is zero
-            if (baseAdapter.getDataList().size() <= 0) {
-                removeAllViews();
+        post(() -> {
+            if (endShimmerSize > 0) {
+                baseAdapter.removeRange(startShimmerSize, (endShimmerSize + startShimmerSize));
+                endShimmerSize = 0;
+                endShimmerSize = tempShimmerSize;
+                flagEzRecyclerViewFirstLoad();
+                flagShimmerStop();
+
+                // Remove all views when data list is zero
+                if (baseAdapter.getDataList().size() <= 0) {
+                    removeAllViews();
+                }
             }
-        }
+        });
     }
 
     private void settingAnimator() {
     }
 
     public void add(@NonNull Data data) {
         flagStopLoading();
-        baseAdapter.add(data);
+        post(() -> baseAdapter.add(data));
     }
 
     public void addAll(@NonNull List<Data> dataList) {
-        post(() -> {
-            if (dataList.size() <= 0) {
-                flagOnStartLoading();
-                currentPage = 0;
-                return;
-            }
-            flagStopLoading();
-            baseAdapter.addAll(dataList);
-        });
+        if (dataList.size() <= 0) {
+            flagOnStartLoading();
+            currentPage = 0;
+            return;
+        }
+        flagStopLoading();
+        post(() -> baseAdapter.addAll(dataList));
     }
 
     public void replace(@NonNull Data data) {
@@ -249,7 +251,6 @@ public void replaceAll(@NonNull List<Data> dataList) {
         if (dataList.size() <= 0) {
             flagOnStartLoading();
             currentPage = 0;
-            return;
         }
         flagStopLoading();
         removeAllViews();
@@ -278,14 +279,15 @@ public void refresh() {
         baseAdapter.refresh();
     }
 
-    public void resetAllViewsAndShimmer() {
-        offset = 0;
-        limit = 0;
-        endShimmerSize = tempShimmerSize;
-        flagOnStartLoading();
+    /**
+     * Use #reset() every re-start
+     * Not recommend for first init
+     * */
+    public void reset() {
+        this.limit = limitTemp;
+        this.offset = offsetTemp;
         flagEzRecyclerViewFirstLoad();
         removeAll();
-        removeAllViews();
     }
 
     public void setEzPaginationListener(EzPaginationListener ezPaginationListener) {
@@ -294,13 +296,17 @@ public void setEzPaginationListener(EzPaginationListener ezPaginationListener) {
         this.offset = 0;
         this.limit = 0;
         this.currentPage = 0;
+        this.offsetTemp = 0;
+        this.limitTemp = 0;
     }
 
     public void setEzPaginationListener(int initialLimit, int initialOffset, EzPaginationListener ezPaginationListener) {
         addOnScrollListener(onScrollListener);
         this.ezPaginationListener = ezPaginationListener;
         this.offset = initialOffset;
         this.limit = initialLimit;
+        this.offsetTemp = initialOffset;
+        this.limitTemp = initialLimit;
     }
 
     private final OnScrollListener onScrollListener = new OnScrollListener() {
@@ -471,7 +477,6 @@ public void setEzMultipleLayout(EzMultipleLayout ezMultipleLayout) {
             this.ezMultipleLayout = ezMultipleLayout;
         }
 
-
         public void add(@NonNull Data data) {
             this.dataList.add(data);
             notifyDataSetChanged();
@@ -593,6 +598,7 @@ public void onBindViewHolder(@NonNull BaseViewHolder holder, int position) {
             holder.setIsRecyclable(false);
             final Data data = dataList.get(position);
             data.setPosition(position); // to know position of view
+            if (data.isCustomShimmerLayout()) return;
             listener.setDataOnViewHolder(holder.itemView, data);
             // startShimmerItem(holder, data);
         }