【问题标题】:FragmentPagerAdapter throwing IllegalArgumentException - Fragment Already AddedFragmentPagerAdapter 抛出 IllegalArgumentException - 已添加片段
【发布时间】:2019-10-24 21:30:38
【问题描述】:

我有一个 ViewPager 和一个 TabLayout 包含 3 个片段。有时,当我(在ViewPager 的帮助下)扫过 3 个片段时,会抛出一个 IllegalArgumentException 并显示 片段已添加:...(索引 0 处的片段到请明确点)。由于错误日志仅显示与 FragmentManager 类有关而与我的 Fragment 类无关,这可能是什么问题?以下是我正在使用的适配器


import androidx.fragment.app.Fragment
import androidx.fragment.app.FragmentManager
import androidx.fragment.app.FragmentPagerAdapter
import androidx.fragment.app.FragmentStatePagerAdapter

data class TitledFragment(val fragment: Fragment, val title: CharSequence?)

/**
 * Should be used when displaying more than 2 [Fragment]s in a view-pager otherwise use
 * [FragmentViewPagerAdapter]
 */
@Suppress("KDocUnresolvedReference")
class FragmentStateViewPagerAdapter(
    private val titledFragmentList: List<TitledFragment>,
    fragmentManager: FragmentManager
) : FragmentStatePagerAdapter(fragmentManager, BEHAVIOR_RESUME_ONLY_CURRENT_FRAGMENT) {
    private val tabCount = titledFragmentList.size

    /**
     * Selects the middle or first [Fragment] as the default selected [Fragment] from a list
     * ([titledFragmentList]) of odd-numbered [Fragment]s and even-numbered [Fragment]s respectively.
     * For example, a view-pager supposed to display 3 fragments ([titledFragmentList] = 3) will have
     * it's default selected fragment position equal to 1(2nd [Fragment]) from the list whereas a
     * view-pager supposed to display 2 fragments ([titledFragmentList] = 2) will have it's default
     * selected fragment position equal to 0(1st [Fragment])
     */
    val middleFragmentPosition: Int
        get() = if (tabCount <= 0 || tabCount == 1) {
            0
        } else {
            val fl = tabCount.toFloat() / 2
            when {
                tabCount % 2 == 0 -> fl.toInt() - 1
                else -> fl.toInt()
            }
        }

    override fun getPageTitle(position: Int): CharSequence? =
        titledFragmentList[position].title ?: super.getPageTitle(position)

    override fun getItem(position: Int): Fragment = titledFragmentList[position].fragment

    override fun getCount(): Int = tabCount
}

/**
 * Should be used when displaying less than 3 [Fragment]s in a view-pager otherwise use
 * [FragmentStateViewPagerAdapter]
 */
@Suppress("KDocUnresolvedReference")
class FragmentViewPagerAdapter(
    private val titledFragmentList: List<TitledFragment>,
    fragmentManager: FragmentManager
) : FragmentPagerAdapter(fragmentManager, BEHAVIOR_RESUME_ONLY_CURRENT_FRAGMENT) {
    private val tabCount = titledFragmentList.size

    /**
     * Selects the middle or first [Fragment] as the default selected [Fragment] from a list
     * ([titledFragmentList]) of odd-numbered [Fragment]s and even-numbered [Fragment]s respectively.
     * For example, a view-pager supposed to display 3 fragments ([titledFragmentList] = 3) will have
     * it's default selected fragment position equal to 1(2nd [Fragment]) from the list whereas a
     * view-pager supposed to display 2 fragments ([titledFragmentList] = 2) will have it's default
     * selected fragment position equal to 0(1st [Fragment])
     */
    val middleFragmentPosition: Int
        get() = if (tabCount <= 0 || tabCount == 1) {
            0
        } else {
            val fl = tabCount.toFloat() / 2
            when {
                tabCount % 2 == 0 -> fl.toInt() - 1
                else -> fl.toInt()
            }
        }

    override fun getPageTitle(position: Int): CharSequence? =
        titledFragmentList[position].title ?: super.getPageTitle(position)

    override fun getItem(position: Int): Fragment = titledFragmentList[position].fragment

    override fun getCount(): Int = tabCount
}

【问题讨论】:

  • middleFragmentPosition 很容易简化,不使用。

标签: android-fragments kotlin android-viewpager androidx fragmentmanager


【解决方案1】:

我认为,您不应该在适配器之外保存对片段的引用。它们应该在适配器内部创建。大概适配器可以这样写:

class FragmentStateViewPagerAdapter(
    private val tabTitles: List<String>,
    fragmentManager: FragmentManager
) : FragmentStatePagerAdapter(fragmentManager, BEHAVIOR_RESUME_ONLY_CURRENT_FRAGMENT) {

    private val tabs: List<Fragment> = (tabTitles.indices).map { YourFragment.newInstance(it) }

    override fun getCount(): Int = tabTitles.size

    override fun getItem(position: Int): Fragment = tabs[position]

    override fun getPageTitle(position: Int): CharSequence {
        // Generate a title depending on the position.
        return tabTitles[position]
    }
}

但你的例子也有效。也许需要更多代码。同样奇怪的是BEHAVIOR_RESUME_ONLY_CURRENT_FRAGMENT在我的情况下不是必需的(它显示错误,但不能)。

class MainActivity : AppCompatActivity() {

    private lateinit var adapter: FragmentViewPagerAdapter

    override fun onCreate(savedInstanceState: Bundle?) {
        super.onCreate(savedInstanceState)
        setContentView(R.layout.activity_main)

        val titledFragments = (1..3).map { TitledFragment(PlusOneFragment(), it.toString()) }
        adapter = FragmentViewPagerAdapter(titledFragments, supportFragmentManager)
        with(view_pager) {
            adapter = this@MainActivity.adapter
        }
    }
}

【讨论】:

    猜你喜欢
    • 1970-01-01
    • 2019-01-07
    • 2013-12-03
    • 1970-01-01
    • 1970-01-01
    • 2011-09-09
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    相关资源
    最近更新 更多