Best way to update data with a RecyclerView adapter [duplicate]

Question

When I have to use a classic adapter with a ListView, I update my data in the ListView like this:

myAdapter.swapArray(data);

public swapArray(List<Data> data) {
  clear();
  addAll(data);
  notifyDataSetChanged();
}

I would like to know what is the best practice for a RecyclerView. Because in a RecyclerView adapter you can't do a clear and addAll as in ListView.

So I tried just with a notifyDataSetChanged, but it didn't work. Then I tried with a swapAdapter on my view:

List<Data> data = newData;

MyRecyclerAdapter adapter = new MyRecyclerAdapter(data);

// swapAdapter on my recyclerView (instead of a .setAdapter like with a classic listView).
recyclerViewList.swapAdapter(adapter, false);

But with this last solution, I still have to create a new instance of my adapter and I feel like it's not the best solution. I should be able just to change my data without a new MyRecyclerAdapter.

Pretzel

It is better (for animations and performance) if you dispatch detailed notify events (e.g. notfiyItemInserted/Removed/Moved/Changed). Also take a look at SortedList class, which can calculate these values for you if you have a sorting criteria in your list. – Gluconeogenesis

I'm working on every change in my adapter notifyDataSetChanged(); – Darius

notifyDataSetChanged() is the worst method for performance. Use it as the last chance. See #31368099 for notifyItemRangeInserted/Removed/Moved/Changed as @Gluconeogenesis advised. – Brace

Answer 1

RecyclerView's Adapter doesn't come with many methods otherwise available in ListView's adapter. But your swap can be implemented quite simply as:

class MyRecyclerAdapter extends RecyclerView.Adapter<RecyclerView.ViewHolder> {
   List<Data> data;
   ...

    public void swap(ArrayList<Data> datas)
    {
        data.clear();
        data.addAll(datas);
        notifyDataSetChanged();     
    }
}

Also there is a difference between

list.clear();
list.add(data);

and

list = newList;

The first is reusing the same list object. The other is dereferencing and referencing the list. The old list object which can no longer be reached will be garbage collected but not without first piling up heap memory. This would be the same as initializing new adapter everytime you want to swap data.

Purloin

Though the above one works for simple list.size(), of data's mine was bit complex where i have multiple list inside the recyclerview. The recycler view list views have managed by multiple view type. Hence it was hard to remove subset of list items that is not visible or not bind by bindViewholder. The only way which is worked for me is RecyclerView.swapAdapter(); – Ribeiro

The above mention swap(ArrayList<Data> datas) doesn't work and neither the Log.d comment method RecyclerView.swapAdapter(); ,anyway found my own way to clear the datasource(ArrayList at the creation place) first ,next clear the adapter datasource using method in adapter and again create new datasource(ArrayList) with new data and than adding that data to the adapter arraylist and finally notifydatasetchanged() method. – Gazebo

@Purloin possible edit in your answer above: should "notifyDataSetChanged();" be "data.notifyDataSetChanged();" ? – Tepefy

I don't see why you wouldn't allow a data set that's empty to be loaded. What if i want to clear out my RecyclerView? That datas.size==0 shouldn't really be there. – Histrionics

My answer has been edited many times by many different people. You are right, the only check needed is just for null. – Purloin

but null check is not supposed to be the scope of the adapter. Which should be null, which should be empty is the responsibility of the data provider. If there's no consistency here expect every single element throughout the app to have null check, which is messed up. – Purloin

i need a kotlin answer – Putup

Answer 2

@inmyth's answer is correct, just modify the code a bit, to handle empty list.

public class NewsAdapter extends RecyclerView.Adapter<...> {    
    ...
    private static List mFeedsList;
    ...    
    public void swap(List list){
            if (mFeedsList != null) {
                mFeedsList.clear();
                mFeedsList.addAll(list);
            }
            else {
                mFeedsList = list;
            }
            notifyDataSetChanged();
    }

I am using Retrofit to fetch the list, on Retrofit's onResponse() use,

adapter.swap(feedList);

Flare

Hi i'm struggling to use this code in my project where use retrofit to get data from an api, would you mind helping me? – Putup

clear and addAll is red for me with the Kotlin translation. – Putup

Answer 3

DiffUtil can the best choice for updating the data in the RecyclerView Adapter which you can find in the android framework. DiffUtil is a utility class that can calculate the difference between two lists and output a list of update operations that converts the first list into the second one.

Most of the time our list changes completely and we set new list to RecyclerView Adapter. And we call notifyDataSetChanged to update adapter. NotifyDataSetChanged is costly. DiffUtil class solves that problem now. It does its job perfectly!

Acquit

Answer 4

Found following solution working for my similar problem:

private ExtendedHashMap mData = new ExtendedHashMap();
private  String[] mKeys;

public void setNewData(ExtendedHashMap data) {
    mData.putAll(data);
    mKeys = data.keySet().toArray(new String[data.size()]);
    notifyDataSetChanged();
}

Using the clear-command

mData.clear()

is not nessescary

Wanonah