ListAdapter не оновлює елемент у RecyclerView

Я використовую нову бібліотеку підтримки ListAdapter. Ось мій код адаптера

class ArtistsAdapter : ListAdapter<Artist, ArtistsAdapter.ViewHolder>(ArtistsDiff()) {
    override fun onCreateViewHolder(parent: ViewGroup, viewType: Int): ViewHolder {
        return ViewHolder(parent.inflate(R.layout.item_artist))
    }

    override fun onBindViewHolder(holder: ViewHolder, position: Int) {
        holder.bind(getItem(position))
    }

    class ViewHolder(view: View) : RecyclerView.ViewHolder(view) {
        fun bind(artist: Artist) {
            itemView.artistDetails.text = artist.artistAlbums
                    .plus(" Albums")
                    .plus(" \u2022 ")
                    .plus(artist.artistTracks)
                    .plus(" Tracks")
            itemView.artistName.text = artist.artistCover
            itemView.artistCoverImage.loadURL(artist.artistCover)
        }
    }
}

Я оновлюю адаптер з

musicViewModel.getAllArtists().observe(this, Observer {
            it?.let {
                artistAdapter.submitList(it)
            }
        })

Мій диф клас

class ArtistsDiff : DiffUtil.ItemCallback<Artist>() {
    override fun areItemsTheSame(oldItem: Artist?, newItem: Artist?): Boolean {
        return oldItem?.artistId == newItem?.artistId
    }

    override fun areContentsTheSame(oldItem: Artist?, newItem: Artist?): Boolean {
        return oldItem == newItem
    }
}

Що відбувається, коли submitList викликається вперше, коли адаптер відтворює всі елементи, але коли submitList викликається знову з оновленими властивостями об’єкта, він не рендерить представлення, яке змінилося.

Він рендерить вигляд, коли я прокручую список, який, у свою чергу, викликає bindView()

Крім того, я помітив, що виклик adapter.notifyDatasSetChanged()після подання списку робить подання з оновленими значеннями, але я не хочу телефонувати, notifyDataSetChanged()оскільки в адаптер списку вбудовано різну використання

Хтось може мені тут допомогти?

Редагувати: Я розумію, чому так трапляється, що не було моєю метою. Моя думка полягає в тому, що принаймні потрібно дати попередження або викликати notifyDataSetChanged()функцію. Тому що, мабуть, я викликаю submitList(...)функцію з причини. Я майже впевнений, що люди намагаються з'ясувати, що пішло не так годинами, поки не зрозуміють, що submitList () мовчки ігнорує виклик.

Це через Googleдивну логіку. Отже, якщо ви передаєте той самий список адаптеру, він навіть не викликає DiffUtil.

public void submitList(final List<T> newList) {
    if (newList == mList) {
        // nothing to do
        return;
    }
....
}

Я справді не розумію цілого сенсу цього, ListAdapterякщо він не може обробляти зміни в тому ж списку. Якщо ви хочете змінити елементи списку, які ви переходите до, ListAdapterі побачити зміни, то вам або потрібно створити глибоку копію списку, або вам потрібно використовувати звичайний RecyclerViewдля власного DiffUtillкласу.

Бібліотека припускає, що ви використовуєте Room або будь-який інший ORM, який пропонує новий список асинхронізації кожного разу, коли він оновлюється, тому просто виклик submitList в ньому буде працювати, а для неакуратних розробників це заважає робити обчислення двічі, якщо викликається один і той же список.

Прийнята відповідь правильна, вона пропонує пояснення, але не рішення.

Що ви можете зробити, якщо ви не використовуєте жодної з таких бібліотек:

submitList(null);
submitList(myList);

Іншим рішенням буде перевизначення submitList (що не спричиняє такого швидкого мигання) як такого:

@Override
public void submitList(final List<Author> list) {
    super.submitList(list != null ? new ArrayList<>(list) : null);
}

Або з кодом Котліна:

override fun submitList(list: List<CatItem>?) {
    super.submitList(list?.let { ArrayList(it) })
}

Сумнівна логіка, але працює чудово. Моїм переважним методом є другий, оскільки він не призводить до кожного рядка, щоб отримати виклик onBind.

з Kotlin вам просто потрібно перетворити свій список на новий MutableList, такий як той чи інший тип списку, відповідно до вашого використання

.observe(this, Observer {
            adapter.submitList(it?.toMutableList())
        })

У мене була подібна проблема, але неправильний візуалізація була викликана поєднанням setHasFixedSize(true)і android:layout_height="wrap_content". Вперше адаптер отримав порожній список, тому висота ніколи не оновлювалась і була 0. У всякому разі, це вирішило мою проблему. Хтось інший може мати таку ж проблему і подумає, що це проблема в адаптері.

Якщо у вас виникають деякі проблеми під час використання

recycler_view.setHasFixedSize(true)

вам слід обов'язково перевірити цей коментар: https://github.com/thoughtbot/expandable-recycler-view/issues/53#issuecomment-362991531

Це вирішило проблему на моєму боці.

(Ось скріншот коментаря за запитом)

Сьогодні я теж натрапив на цю "проблему". За допомогою відповіді insa_c в і рішення RJFares ігрова я зробив собі функцію розширення Котлин:

/**
 * Update the [RecyclerView]'s [ListAdapter] with the provided list of items.
 *
 * Originally, [ListAdapter] will not update the view if the provided list is the same as
 * currently loaded one. This is by design as otherwise the provided DiffUtil.ItemCallback<T>
 * could never work - the [ListAdapter] must have the previous list if items to compare new
 * ones to using provided diff callback.
 * However, it's very convenient to call [ListAdapter.submitList] with the same list and expect
 * the view to be updated. This extension function handles this case by making a copy of the
 * list if the provided list is the same instance as currently loaded one.
 *
 * For more info see 'RJFares' and 'insa_c' answers on
 * /programming/49726385/listadapter-not-updating-item-in-reyclerview
 */
fun <T, VH : RecyclerView.ViewHolder> ListAdapter<T, VH>.updateList(list: List<T>?) {
    // ListAdapter<>.submitList() contains (stripped):
    //  if (newList == mList) {
    //      // nothing to do
    //      return;
    //  }
    this.submitList(if (list == this.currentList) list.toList() else list)
}

які потім можна використовувати де завгодно, наприклад:

viewModel.foundDevices.observe(this, Observer {
    binding.recyclerViewDevices.adapter.updateList(it)
})

і він копіює список (і завжди), якщо він такий самий, як і завантажений.

Згідно з офіційними документами :

Кожного разу, коли ви викликаєте submitList, він подає новий список для розширення та відображення.

Ось чому, коли ви викликаєте submitList у попередньому (вже поданому списку), він не обчислює Diff і не повідомляє адаптер про зміну набору даних.

Для мене ця проблема виникла, якщо я використовував RecyclerViewусередині ScrollViewз nestedScrollingEnabled="false"і висоту RV встановлено на wrap_content.
Адаптер оновився належним чином і було викликано функцію прив'язки, але елементи не відображались - RecyclerViewоригінальний розмір застряг.

Зміна ScrollViewна NestedScrollViewвиправлену проблему.

У моєму випадку я забув встановити LayoutManagerдля RecyclerView. Ефект від цього такий же, як описано вище.

Для тих, хто має такий самий сценарій, як я, я залишаю своє рішення, яке я не знаю, чому воно працює, тут.

Рішення, яке спрацювало для мене, було від @Mina Samir, яка подає список як змінний список.

Сценарій мого випуску:

-Завантаження списку друзів у фрагмент.

1. ActivityMain додає FragmentFriendList (Спостерігає за даними live db елементів друга) і одночасно запитує на сервер запит http, щоб отримати весь мій список друзів.

2. Оновіть або вставте елементи з сервера http.

3. Кожна зміна запалює зворотний виклик onChanged даних live. Але коли я вперше запускаю програму, а це означає, що на моєму столі нічого не було, submitList вдається без будь-якої помилки, але на екрані нічого не відображається.

4. Однак, коли я вдруге запускаю програму, дані завантажуються на екран.

Рішення полягає в тому, як вже згадувалося вище, подання списку як змінного списку.

У мене була подібна проблема. Проблема полягала у Diffфункціях, які не адекватно порівнювали елементи. Будь-хто, хто має цю проблему, переконайтесь, що ваші Diffфункції (і, як розширення, класи об’єктів даних) містять правильні визначення порівняння - тобто, порівнюючи всі поля, які можуть бути оновлені в новому елементі. Наприклад в оригінальній публікації

    override fun areContentsTheSame(oldItem: Artist?, newItem: Artist?): Boolean {
    return oldItem == newItem
}

Ця функція (потенційно) не виконує те, що написано на етикетці: вона не порівнює вміст двох елементів - якщо ви не перевизначили equals()функцію в Artistкласі. У моєму випадку я цього не зробив, і визначення areContentsTheSameлише перевірило одне з необхідних полів, завдяки моєму нагляду при його реалізації. Це структурна рівність проти референтної рівності, ви можете дізнатись більше про це тут

Мені потрібно було змінити мої DiffUtils

override fun areContentsTheSame(oldItem: Vehicle, newItem: Vehicle): Boolean {

Щоб фактично повернути, чи є вміст новим, а не просто порівняти ідентифікатор моделі.

Використання @RJFares first answer успішно оновлює список, але не підтримує стан прокрутки. Весь RecyclerViewпочинається з 0-ї позиції. Як обхідний шлях я зробив це:

   fun updateDataList(newList:List<String>){ //new list from DB or Network

     val tempList = dataList.toMutableList() // dataList is the old list
     tempList.addAll(newList)
     listAdapter.submitList(tempList) // Recyclerview Adapter Instance
     dataList = tempList

   }

Таким чином, я можу підтримувати стан прокрутки RecyclerViewразом із модифікованими даними.