22

I've updated my targetSdkVersion from 28 to 30 and I've noticed that getAdapterPosition() is deprecated (I'm not sure when this happened).

In the documentation, they say the following:

This method is deprecated.
This method is confusing when adapters nest other adapters. If you are calling this in the context of an Adapter, you probably want to call getBindingAdapterPosition() or if you want the position as RecyclerView sees it, you should call getAbsoluteAdapterPosition().

The documentation also says the following:

Note that if you are querying the position as RecyclerView sees, you should use getAbsoluteAdapterPosition() (e.g. you want to use it to save scroll state). If you are querying the position to access the RecyclerView.Adapter contents, you should use getBindingAdapterPosition().

How I understand it is:

  • getBindingAdapterPosition should be used when you want to get the adapter position (if it still exists in the adapter). If it no longer exists in the adapter, it will return -1(NO_POSITION).
  • getAbsoluteAdapterPosition should be used to get the position as the RecyclerView sees it. For example, if an item has been deleted, but not yet removed from theViewHolder.

In other words, if I have 4 items in my Adapter, I delete position 0 and query getAbsoluteAdapterPosition and getBindingAdapterPosition before the item has been removed from the ViewHolder, then getAbsoluteAdapterPosition will return 0(because view is still in the ViewHolder) and getBindingAdapterPosition return -1(Because it no longer exists in the adapter).


I have tested the difference by logging the following:

Log.e("difference", "getAdapterPosition = "+myHolder.getAdapterPosition()+" getBindingAdapterPosition = "+myHolder.getBindingAdapterPosition()+" getAbsoluteAdapterPosition = "+myHolder.getAbsoluteAdapterPosition());

They return exactly the same values. I could not see any difference.

I also see no difference before or after calling notifyItemChanged, notifyDataSetChanged or notifyItemRangeChanged. But when I delete position 0 and call notifyItemRemoved it returns -1 afterward (for all of them).

My questions

Do I understand this correctly, and when should we be using which? Also, when will there be a difference?

halfer
  • 18,701
  • 13
  • 79
  • 158
ClassA
  • 1,810
  • 14
  • 42
  • There is a difference if you use the new **`ConcatAdapter`** introduced in the `1.2.0`. In this case the `getBindingAdapterPosition()` returns the position in the specific adapter and not the whole `ConcatAdapter`. Instead the `getAbsoluteAdapterPosition()` returns the position in the `ConcatAdapter`. – Gabriele Mariotti Jul 24 '20 at 07:37
  • @GabrieleMariotti Ah that makes sense. So does that mean that, if I have a single adapter, then I can use either `getBindingAdapterPosition()` or `getAbsoluteAdapterPosition()`, or will there be a difference? – ClassA Jul 24 '20 at 07:44
  • @GabrieleMariotti Ok thanks for your reply. – ClassA Jul 24 '20 at 07:48
  • 1
    Also if you check the source code, `getAdapterPosition()` returns now `getBindingAdapterPosition();`. – Gabriele Mariotti Jul 24 '20 at 07:58
  • @GabrieleMariotti If you have time, please provide an answer for me to except. I had a look at the source code and as you mentioned, `getAdapterPosition();` get replaced with `getBindingAdapterPosition();`. I think it's safe to say that we should be using `getBindingAdapterPosition();` when we are using a single adapter. – ClassA Jul 25 '20 at 09:51
  • Personally I think this deprecation was uncalled for. In 98% of cases `ConcatAdapter` won't even be used yet one of the most commonly used method of `ViewHolder` was renamed to account for its existence. – Pawel Aug 14 '20 at 23:35

1 Answers1

35

The recyclerview:1.2.0-alpha02 introduced the MergeAdapter that has been renamed to ConcatAdapter with recyclerview:1.2.0-alpha04.
This RecyclerView Adapter can combine multiple adapters linearly.

Something like:

FirstAdapter adapter1 = ...;
SecondAdapter adapter2 = ...;
ConcatAdapter merged = new ConcatAdapter(adapter1, adapter2);
recyclerView.setAdapter(mergedAdapter);

As part of this change the method getAdapterPosition() was deprecated because the name is confusing when adapters nest other adapters. Now you should use the method getBindingAdapterPosition() which returns the position in the specific adapter (bindingAdapter).

Also checking the source code of RecyclerView.ViewHolder:

    @Deprecated
    public final int getAdapterPosition() {
        return getBindingAdapterPosition();
    }

Instead the method getAbsoluteAdapterPosition() returns the Adapter position of the item with respect to the entire RecyclerView. If you are using a ConcatAdapter it is the position in the ConcatAdapter and not the position in the single bindingAdapter.

Just an example with 2 adapters:

enter image description here

More details are available in the official blog post.

Gabriele Mariotti
  • 192,671
  • 57
  • 469
  • 489