Android. Оправдано ли использование фрагментов в RecyclerView?

Question

[Михаил]

У меня есть карточка которая помимо прочей информации может содержать в себе изображение, слайдер изображений (сторонняя библиотека) или видео (YouTubeAPI). Раньше я в layout карточки просто указывал все три медиа элемента (изображение, слайдер, видео) и, в зависимости от нужд, изменял их видимость, скрывая ненужные элементы и оставляя видимым только нужный.

Потом я решил попробовать использовать фрагменты. Сделал три фрагмента, под каждый элемент (замечу что класс фрагментов содержит в себе всю логику элемента, ему стоит лишь передать параметры для инициализации, а все остальное он сделает сам), и попытался использовать в RecyclerView. Если раньше у меня в карточке было три элемента, то теперь я хотел просто помещать нужный мне фрагмент в специальный контейнер в карточке. Но не тут то было. В onBindViewHolder моего адаптера RecyclerView вот такой код заменяет только первый элемент:

ImageSliderFragment fragment = ImageSliderFragment.newInstance(currentItem.images, mContext);
fragmentManager.beginTransaction().replace(R.id.card_media_container, fragment).commit();

Также я прочитал что использование фрагментов в RecyclerView крайне нежелательно. Вопрос - стоит ли в данном случае использовать фрагменты, ради упрощения (эти фрагменты также используются в другом активити, к тому же раньше вся логика медиа элементов раньше была в адаптере RecyclerView, а теперь вынесена в ихние классы фрагментов). Или же стоит использовать старую схему - карточка сразу содержит в себе все медиа элементы, которые делаются видимыми/невидимыми и заполняются по мере необходимости? Но тогда возникает дублирование кода т.к. логика элементов опять будет в адаптере RecyclerView.

И если использование фрагментов оправдано - как правильно использовать их в RecyclerView? У меня почему-то заполняется только первый элемент.

Answer 1

[Artem Gapchenko]

Не нужно объединять три View в один фрагмент, так как у RecyclerView уже есть следующие методы для создания и привязки ViewHolder'a:

public abstract void onBindViewHolder(VH holder, int position);

и

public abstract VH onCreateViewHolder(ViewGroup parent, int viewType);

Параметр viewType - это то, что вам нужно. Переопределяете метод

public int getItemViewType(int position);

так, чтобы для каждого элемента адаптера он возвращал нужный тип (видео/слайдер/изображение), в onCreateViewHolder() инфлейтите нужную вьюху в зависимости от типа вьюхи, и создаёте ViewHolder, а в onBindViewHolder() привязываете нужный элемент к вьюхолдеру, в зависимости от того, какой тип у элемента.

Comments

[Михаил]

Вы немного не так поняли. RecyclerView создает у меня только один тип - CardView. А вот уже CardView может видоизменятся в зависимости от контента, и содержать только изображение, слайдер, или видео. Поэтому я создал в CardView контейнер в который и добавляю нужный мне на данный момент фрагмент. Раньше у меня в карточке сразу были объявлены все три элемента, и я просто скрывал ненужные, а сейчас хочу добавлять их как фрагменты. Разные типы карточек не делаю потому что там изменяется только медиа блок, остальная часть карточки не изменяется.

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

Михаил: ну вот это вы не поняли. Не надо так делать, кардвью не должен "сам" изменяться.

[Михаил]

Денис Загаевский: медиа элементы в карточке - всего лишь часть карточки, сама карточка содержит также другие вьюшки с информацией. Мне показалось что логично вынести медиа отдельно (т.к. именно они могут изменятся). А если для каждого типа медиа я буду делать отдельный тип карточек - то возникнет дублирование вьюшек карточек, ведь кроме медиа в них ничего не меняется.

[Artem Gapchenko]

@Михаил не возникнет никакого дублирования, если вы выделите AbstractMediaCardView, в который уйдёт весь общий для карточек функционал. Этот класс будет расширяться тремя конкретными типами - VideoCardView, ImageCardView и ImageSliderCardView.
О фрагментах я всё так же думаю, что это overkill. Во-первых они тяжелее обычного View, а во-вторых вангую появление всяких загадочных крэшей, уходящих в FragmentManager, и связанных с таким использованием фрагментов.

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

Михаил: дело ваше, жуйте кактус.

[Евгений Василенко]

Михаил: сам сейчас задумался над похожей задачей. Вот у меня наводящий вопрос по теме автора. Что, если сделать так, как Вы советуете (с разными viewType), и создать, например, отельный holder для видео. Усложняем задачу. Видео должно останавливаться при onPause(). Как посоветуете такое обрабатывать? Во фрагменте или активити, которое содержит этот recylerView, передавать это событие адаптеру? Мне кажется, что это уже неправильная архитектура.
P.S. а попал я в эту тему из-за следующей проблемы. Мне нужно в RecylerView в одном из item-ов отобразить Spinner. Выпадающий список для этого спиннера должен изменяться в зависимости от внешних факторов. Планировал это сделать с помощью лоадера. Но лоадер нужно привязать к Активити или Фрагменту. Вот так и появился вопрос. А можно ли Фрагменты использовать в RecyclerView.

Answer 2

[Михаил]

Решение по поводу заполнения только первого элемента - нужно указывать не обычный id контейнера, а присваивать контейнеру каждого элемента уникальный id и уже в него вставлять фрагмент.

Вопрос по поводу целесообразности использования фрагмента в конкретно данном случае остается открытым.

Comments

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

Не нужны там фрагменты.