Как получить доступ к ViewHolder у RecyclerView, чтобы вызвать метод его отрисовки?

Question

[Александр Синицын]

Есть адаптер private class MyAdapter extends RecyclerView.Adapter
и private class ItemHolder extends RecyclerView.ViewHolder

Отрисовка делается в updateHolder при вызове в ItemHolder bindItem

public void bindItem(ItemClass item) {

  mItem = item;
  updateHolder();
}

из

@Override
        public void onBindViewHolder(ItemHolder holder, int position) {

            holder.bindItem(mItems.get(position));
        }

В updateHolder() используется некоторое внешнее значение, в зависимости от которого выводится mItem.field1 или mItem.field2.

Как при изменении значения внешнего значения сделать перерисовку всех элементов списка лишь вызовом updateHolder()?

Comments

[Никита]

Вызвать notifyDataSetChanged() у адаптера.
Также есть notifyItemChanged

[Александр Синицын]

Никита, Перепробовал все notity и инвалидейты, но не работают, данные-то не изменились и бинд не вызывается.

[Никита]

Александр Синицын, А в чем проблема поместить эти "внешние" данные во внутрь

Answer 1

[Денис Загаевский]

В updateHolder() используется некоторое внешнее значение, в зависимости от которого выводится mItem.field1 или mItem.field2.

Это не очень хорошая практика. Лучше, чтобы все данные лежали в элементе данных. При изменении данных должен изменяться список с данными, вызываться DiffUtil и изменение данных передаваться в адаптер.

Comments

[Александр Синицын]

А если к холдеру обсервер привязать?
---
Привязал, вроде работает )
Мне думается, что это более корректно, чем перегружать весь список... или нет?

[Денис Загаевский]

Александр Синицын, это, как бы, слегка противоречит концепциям ресайклера.

[Александр Синицын]

Денис Загаевский, А разве не "чем короче тем лучше" ))

public ItemHolder(View itemView) {
            super(itemView);

            //... Код

            SkAppStorage.getInstance(getActivity()).addObserver(this);
        }

        @Override
        protected void finalize() throws Throwable {

            SkAppStorage.getInstance(getActivity()).deleteObserver(this);
            super.finalize();
        }

        @Override
        public void update(Observable o, Object arg) {
            updateHolder();
        }

[Денис Загаевский]

Александр Синицын, о_О'
Какое ещё "чем короче тем лучше", вы о чём вообще? Конечно нет, этот код вообще жестяная жесть. DI - нет, не слышал. Использование finalize - это вообще очень и очень плохо, его вызов не гарантирован, когда он вызовется тоже неизвестно. Это огромный риск утечки памяти. Обновление никак не завязано на жизненный цикл холдера(bind-attach-detach-unbind).

[Александр Синицын]

Денис Загаевский, событий attach-detach не нашёл.
А так было всё красиво, если не вдаваться в тонкости ))
Теперь придётся сильно думать.

[Денис Загаевский]

Александр Синицын, onViewAttachedTo/Detached FromWindow

[Александр Синицын]

Денис Загаевский, Перенёс обсервер на фрагмент и в update сделал

mAdapter.setItems(...);
mAdapter.notifyDataSetChanged();

[Денис Загаевский]

Александр Синицын, DiffUtil

[Александр Синицын]

Денис Загаевский, Я так понял, что DiffUtils - это когда меняется только часть данных, чтобы не обновлять все элементы. А у меня в каждом одно поле меняется на другое. Или не так понял?

[Денис Загаевский]

Александр Синицын, он вызывает правильные методы. Если данные изменилиль вот так, то можно не пересоздавать все холдеры, а просто перебиндить их

[Александр Синицын]

Денис Загаевский, Попробую. Спасибо.