片段上的更改列表正在其他片段中更改答案

【问题标题】：Changing list on fragment is changing in other fragments片段上的更改列表正在其他片段中更改
【发布时间】：2017-12-28 13:21:36
【问题描述】：

我的活动中有一个 TabLayout/ViewPager，其中包含两个片段。选项卡已创建，然后我请求一些信息。当我得到这些信息时，我会更新每个选项卡的 RecyclerView。

主要活动

private fun setViews() {
    val adapter = TabsAdapter(supportFragmentManager)
    adapter.addFragment(NewsFragment(), getString(R.string.dossier_activity_news))
    adapter.addFragment(PhotosFragment(), resources.getString(R.string.dossier_activity_photos))

    view_pager.adapter = adapter
    tab_layout.setupWithViewPager(view_pager)

    // request info
}

fun setPages(pages: List<Page>) {
    // got info

    ((view_pager.adapter as TabsAdapter).getItem(0) as NewsFragment).setPages(pages)
    ((view_pager.adapter as TabsAdapter).getItem(1) as PhotosFragment).setPages(pages)
}

问题是：在第二个标签（PhotosFragment）中，我想删除所有不包含照片/缩略图的页面。

新闻片段

fun setPages(pages: List<Page>) {
    if (pages.isNotEmpty()) recycler_view.adapter = PagesAdapter(this, pages)
}

照片片段

fun setPages(pages: List<Page>) {
    val iterator = (pages as ArrayList<Page>).iterator()
    while (iterator.hasNext()) if (iterator.next().thumbnail == null) iterator.remove()
    if (pages.isNotEmpty()) recycler_view.adapter = PagesAdapter(this, pages)
}

当我在第二个选项卡上创建此迭代器时，第一个片段上的所有页面也会更新为没有照片/缩略图页面。每个片段都有自己的生命周期，通过更改一个片段的内容，它不应该更新另一个片段吗？

这可能是因为我的 PagesAdapter 扩展了 RecyclerView 吗？

PagesAdapter

class PagesAdapter(private var fragment: Fragment, private var pages: List<Page>): RecyclerView.Adapter<RecyclerView.ViewHolder>() {

    override fun onCreateViewHolder(parent: ViewGroup, type: Int): RecyclerView.ViewHolder {
        return when (type) {
            0 -> HeaderViewHolder(LayoutInflater.from(parent.context).inflate(R.layout.article_header_item, parent, false))
            else -> ArticleViewHolder(LayoutInflater.from(parent.context).inflate(R.layout.article_item, parent, false))
        }
    }

    override fun onBindViewHolder(holder: RecyclerView.ViewHolder, position: Int) {
        val type = getItemViewType(position)
        when (type) {
            0 -> setHeader(holder as PagesAdapter.HeaderViewHolder, position)
            else -> setArticle(holder as PagesAdapter.ArticleViewHolder, position)
        }
    }

    inner class HeaderViewHolder internal constructor(view: View) : RecyclerView.ViewHolder(view) {
        val layout: RelativeLayout = view.article_header_layout
        val image: ImageView = view.article_header_image
        val title: AOTextView = view.title_header_text
        val date: AOTextView = view.date_header_text
    }

    inner class ArticleViewHolder internal constructor(view: View) : RecyclerView.ViewHolder(view) {
        val layout: RelativeLayout = view.article_layout
        val image: ImageView = view.article_image
        val strapline: AOTextView = view.strapline_text
        val title: AOTextView = view.title_text
        val date: AOTextView = view.date_text
    }
}

两个片段具有相同的页面列表，但我希望第二个片段过滤此列表以仅显示带有照片/缩略图的页面。通过过滤第二个片段中的列表，第一个显示相同的过滤列表。我应该以不同的方式解决这个问题吗？还是我的代码中有错误？

【问题讨论】：

标签： android android-fragments android-recyclerview kotlin android-lifecycle

【解决方案1】：

您在两个片段之间共享一个列表，这不是一个好主意。 Kotlin 支持不变性，它应该是默认的做事方式，除非你有充分的理由。

两个片段共享列表引用，这就是为什么当其中一个片段更改它时，这些更改会反映在另一个片段中。

你应该做的是代替

fun setPages(pages: List<Page>) {
    val iterator = (pages as ArrayList<Page>).iterator()
    while (iterator.hasNext()) if (iterator.next().thumbnail == null) iterator.remove()
    if (pages.isNotEmpty()) recycler_view.adapter = PagesAdapter(this, pages)
}

你应该写：

fun setPages(pages: List<Page>) {
    val pagesWithThumbnails = pages.filter { it.thumbnail != null }
    if (pagesWithThumbnails.isNotEmpty()) recycler_view.adapter = PagesAdapter(this, pagesWithThumbnails)
}

您无需修改原始列表，同时拥有更具可读性和防错性的代码。

【讨论】：