fix index out of bounds exception when notifyItemChanged

chuross · Mar 29, 2018 · 6dd2070 · 6dd2070
1 parent 40e3db5
commit 6dd2070
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/...rview-adapters/src/main/java/com/github/chuross/recyclerviewadapters/BaseItemAdapter.java b/...rview-adapters/src/main/java/com/github/chuross/recyclerviewadapters/BaseItemAdapter.java
@@ -26,19 +26,21 @@ public long getItemId(int position) {
     @Override
     public void onViewAttachedToWindow(final VH holder) {
         super.onViewAttachedToWindow(holder);
-        final int position = getPosition(holder);
-        if (position < 0) return;
 
         holder.itemView.setOnClickListener(new View.OnClickListener() {
             @Override
             public void onClick(View v) {
+                final int position = getPosition(holder);
+                if (position < 0) return;
                 if (clickListener != null) clickListener.onItemClicked(holder, position, get(position));
             }
         });
 
         holder.itemView.setOnLongClickListener(new View.OnLongClickListener() {
             @Override
             public boolean onLongClick(View v) {
+                final int position = getPosition(holder);
+                if (position < 0) return false;
                 if (longPressedListener != null) longPressedListener.onItemLongPressed(holder, position, get(position));
                 return true;
             }