it-roy-ru.com

изменить порядок страниц в FragmentStatePagerAdapter с помощью getItemPosition (Object object)

Я считаю, что FragmentStatePagerAdapter не работает правильно при переопределении getItemPosition(Object object) с целью изменения порядка страниц.

Ниже приведен простой пример. В исходном состоянии порядок страниц равен {A, B, C}. При вызове toggleState() порядок страниц меняется на {A, C, B}. Перекрывая getItemPosition(Object object), мы гарантируем, что текущая просматриваемая страница (A, B или C) не изменится.

public static class TestPagerAdapter extends FragmentStatePagerAdapter {
    private boolean mState = true;

    public TestPagerAdapter(FragmentManager fragmentManager) {
        super(fragmentManager);
    }

    @Override
    public int getCount() {
        return 3;
    }

    private void toggleState() {
        mState = !mState;
        notifyDataSetChanged();
    }

    private String getLabel(int position) {
        switch (position) {
            case 0:
                return "A";
            case 1:
                return mState ? "B" : "C";
            default:
                return mState ? "C" : "B";
        }
    }

    @Override
    public int getItemPosition(Object object) {
        String label = ((TestFragment) object).getLabel();
        if (label.equals("A")) {
            return 0;
        } else if (label.equals("B")) {
            return mState ? 1 : 2;
        } else {
            return mState ? 2 : 1;
        }
    }

    @Override
    public CharSequence getPageTitle(int position) {
        return getLabel(position);
    }

    @Override
    public Fragment getItem(int position) {
        return TestFragment.newInstance(getLabel(position));
    }
}

Я столкнулся с двумя отдельными поведениями, которые кажутся неправильными.

  1. Если я немедленно позвоню toggleState() (при просмотре страницы A, прежде чем перейти к любой другой странице), приложение вылетает.

    Java.lang.IndexOutOfBoundsException: Invalid index 2, size is 2
      at Java.util.ArrayList.throwIndexOutOfBoundsException(ArrayList.Java:251)
      at Java.util.ArrayList.set(ArrayList.Java:477)
      at Android.support.v4.app.FragmentStatePagerAdapter.destroyItem(FragmentStatePagerAdapter.Java:136)
      at Android.support.v4.view.ViewPager.populate(ViewPager.Java:867)
      at Android.support.v4.view.ViewPager.setCurrentItemInternal(ViewPager.Java:469)
      at Android.support.v4.view.ViewPager.setCurrentItemInternal(ViewPager.Java:441)
      at Android.support.v4.view.ViewPager.dataSetChanged(ViewPager.Java:766)
      at Android.support.v4.view.ViewPager$PagerObserver.onChanged(ViewPager.Java:2519)
      at Android.database.DataSetObservable.notifyChanged(DataSetObservable.Java:37)
      at Android.support.v4.view.PagerAdapter.notifyDataSetChanged(PagerAdapter.Java:276)
      at com.ugglynoodle.test.testfragmentstatepageradapter.MainActivity$TestPagerAdapter.toggleState(MainActivity.Java:55)
      ...
    

    Если посмотреть на источник FragmentStatePagerAdapter, это можно исправить, сначала проверив размер mFragments (как в строках 113-115) перед вызовом set() в строке 136.

  2. Если я сначала проведу пальцем по странице B, то вызовется getItem(2), будет создана страница C, а размер mFragments теперь равен 3 (это предотвратит мгновенное падение, описанное выше). Затем я проведу пальцем по странице A, и страница C будет уничтожена, как и должно быть (поскольку до нее осталось 2 страницы, и я использую ограничение по умолчанию для страницы за пределами экрана, равное 1). Теперь я звоню toggleState(). Страница B теперь уничтожена. Однако страница C НЕ воссоздана! Это означает, что когда я теперь проведу вправо, я получу пустую страницу.

Во-первых, было бы приятно узнать, прав ли я, и это на самом деле ошибки, или я делаю что-то не так. Если они являются ошибками, может ли кто-нибудь предложить обходной путь (кроме отладки и перестройки библиотеки поддержки самостоятельно)? Конечно, кто-то должен был успешно переопределить функцию getItemPosition(Object object) (кроме установки для POSITION_NONE)?

Я использую текущую версию (10) библиотеки поддержки.

23
UgglyNoodle

Глядя на источник FragmentStatePagerAdapter, я выяснил, что именно идет не так. FragmentStatePagerAdapter кэширует фрагменты и сохраненные состояния в ArrayLists: mFragments и mSavedState. Но когда фрагменты переупорядочиваются, отсутствует механизм для переупорядочения элементов mFragments и mSavedState. Поэтому адаптер предоставит неверные фрагменты на пейджер.

Я подал вопрос для этого и приложил к проблеме фиксированную реализацию (NewFragmentStatePagerAdapter.Java). В этом исправлении я добавил функцию getItemId() во FragmentStatePagerAdapter. (Это отражает реализацию переупорядочения в FragmentPagerAdapter.) Массив itemIds по позиции адаптера сохраняется всегда. Затем в notifyDataSetChanged() адаптер проверяет, изменился ли массив itemIds. Если это так, то mFragments и mSavedState переупорядочиваются соответственно. Дальнейшие изменения можно найти в destroyItem(), saveState() и restoreState().

Чтобы использовать этот класс, getItemPosition() и getItemId() должны быть реализованы в соответствии с getItem().

35
UgglyNoodle

У меня сработал один из ответов вопрос . Ответы № 20 № 21. Ссылка на решение https://Gist.github.com/ypresto/8c13cb88a0973d071a64 . Лучшее решение, работает для обновления страниц, а также для изменения порядка. Только в этом решении Adapter не генерировал IndexOutOfBoundsExeption при уничтожении элемента (в методе destroyItem), что является известной ошибкой для других решений.

1
hovo888s

Я переопределил существующее решение в Kotlin так, что оно позволяет вам возвращать String вместо long для идентификатора элемента. Вы можете найти его здесь или ниже:

import Android.annotation.SuppressLint
import Android.os.Bundle
import Android.os.Parcelable
import Android.support.v4.app.Fragment
import Android.support.v4.app.FragmentManager
import Android.support.v4.app.FragmentTransaction
import Android.view.View
import Android.view.ViewGroup
import Java.util.HashSet
import Java.util.LinkedHashMap

/**
 * A PagerAdapter that can withstand item reordering. See
 * https://issuetracker.google.com/issues/36956111.
 *
 * @see Android.support.v4.app.FragmentStatePagerAdapter
 */
abstract class MovableFragmentStatePagerAdapter(
        private val manager: FragmentManager
) : NullablePagerAdapter() {
    private var currentTransaction: FragmentTransaction? = null
    private var currentPrimaryItem: Fragment? = null

    private val savedStates = LinkedHashMap<String, Fragment.SavedState>()
    private val fragmentsToItemIds = LinkedHashMap<Fragment, String>()
    private val itemIdsToFragments = LinkedHashMap<String, Fragment>()
    private val unusedRestoredFragments = HashSet<Fragment>()

    /** @see Android.support.v4.app.FragmentStatePagerAdapter.getItem */
    abstract fun getItem(position: Int): Fragment

    /**
     * @return a unique identifier for the item at the given position.
     */
    abstract fun getItemId(position: Int): String

    /** @see Android.support.v4.app.FragmentStatePagerAdapter.startUpdate */
    override fun startUpdate(container: ViewGroup) {
        check(container.id != View.NO_ID) {
            "ViewPager with adapter $this requires a view id."
        }
    }

    /** @see Android.support.v4.app.FragmentStatePagerAdapter.instantiateItem */
    override fun instantiateItem(container: ViewGroup, position: Int): Any {
        val itemId = getItemId(position)

        val f = itemIdsToFragments[itemId]
        if (f != null) {
            unusedRestoredFragments.remove(f)
            return f
        }

        if (currentTransaction == null) {
            // We commit the transaction later
            @SuppressLint("CommitTransaction")
            currentTransaction = manager.beginTransaction()
        }

        val fragment = getItem(position)
        fragmentsToItemIds.put(fragment, itemId)
        itemIdsToFragments.put(itemId, fragment)

        val fss = savedStates[itemId]
        if (fss != null) {
            fragment.setInitialSavedState(fss)
        }
        fragment.setMenuVisibility(false)
        fragment.userVisibleHint = false

        currentTransaction!!.add(container.id, fragment)

        return fragment
    }

    /** @see Android.support.v4.app.FragmentStatePagerAdapter.destroyItem */
    override fun destroyItem(container: ViewGroup, position: Int, fragment: Any) {
        (fragment as Fragment).destroy()
    }

    /** @see Android.support.v4.app.FragmentStatePagerAdapter.setPrimaryItem */
    override fun setPrimaryItem(container: ViewGroup, position: Int, fragment: Any?) {
        fragment as Fragment?
        if (fragment !== currentPrimaryItem) {
            currentPrimaryItem?.let {
                it.setMenuVisibility(false)
                it.userVisibleHint = false
            }

            fragment?.setMenuVisibility(true)
            fragment?.userVisibleHint = true
            currentPrimaryItem = fragment
        }
    }

    /** @see Android.support.v4.app.FragmentStatePagerAdapter.finishUpdate */
    override fun finishUpdate(container: ViewGroup) {
        if (!unusedRestoredFragments.isEmpty()) {
            for (fragment in unusedRestoredFragments) fragment.destroy()
            unusedRestoredFragments.clear()
        }

        currentTransaction?.let {
            it.commitAllowingStateLoss()
            currentTransaction = null
            manager.executePendingTransactions()
        }
    }

    /** @see Android.support.v4.app.FragmentStatePagerAdapter.isViewFromObject */
    override fun isViewFromObject(view: View, fragment: Any): Boolean =
            (fragment as Fragment).view === view

    /** @see Android.support.v4.app.FragmentStatePagerAdapter.saveState */
    override fun saveState(): Parcelable? = Bundle().apply {
        putStringArrayList(KEY_FRAGMENT_IDS, ArrayList<String>(savedStates.keys))
        putParcelableArrayList(
                KEY_FRAGMENT_STATES,
                ArrayList<Fragment.SavedState>(savedStates.values)
        )

        for ((f, id) in fragmentsToItemIds.entries) {
            if (f.isAdded) {
                manager.putFragment(this, "$KEY_FRAGMENT_STATE$id", f)
            }
        }
    }

    /** @see Android.support.v4.app.FragmentStatePagerAdapter.restoreState */
    override fun restoreState(state: Parcelable?, loader: ClassLoader?) {
        if ((state as Bundle?)?.apply { classLoader = loader }?.isEmpty == false) {
            state!!

            fragmentsToItemIds.clear()
            itemIdsToFragments.clear()
            unusedRestoredFragments.clear()
            savedStates.clear()

            val fragmentIds: List<String> = state.getStringArrayList(KEY_FRAGMENT_IDS)
            val fragmentStates: List<Fragment.SavedState> =
                    state.getParcelableArrayList(KEY_FRAGMENT_STATES)

            for ((index, id) in fragmentIds.withIndex()) {
                savedStates.put(id, fragmentStates[index])
            }

            for (key: String in state.keySet()) {
                if (key.startsWith(KEY_FRAGMENT_STATE)) {
                    val itemId = key.substring(KEY_FRAGMENT_STATE.length)

                    manager.getFragment(state, key)?.let {
                        it.setMenuVisibility(false)
                        fragmentsToItemIds.put(it, itemId)
                        itemIdsToFragments.put(itemId, it)
                    }
                }
            }

            unusedRestoredFragments.addAll(fragmentsToItemIds.keys)
        }
    }

    private fun Fragment.destroy() {
        if (currentTransaction == null) {
            // We commit the transaction later
            @SuppressLint("CommitTransaction")
            currentTransaction = manager.beginTransaction()
        }

        val itemId = fragmentsToItemIds.remove(this)
        itemIdsToFragments.remove(itemId)
        if (itemId != null) {
            savedStates.put(itemId, manager.saveFragmentInstanceState(this))
        }

        currentTransaction!!.remove(this)
    }

    private companion object {
        const val KEY_FRAGMENT_IDS = "fragment_keys_"
        const val KEY_FRAGMENT_STATES = "fragment_states_"
        const val KEY_FRAGMENT_STATE = "fragment_state_"
    }
}

И кусок Java:

import Android.support.annotation.NonNull;
import Android.support.annotation.Nullable;
import Android.support.v4.view.PagerAdapter;
import Android.view.ViewGroup;

/**
 * A PagerAdapter whose {@link #setPrimaryItem} is overridden with proper nullability annotations.
 */
public abstract class NullablePagerAdapter extends PagerAdapter {
    @Override
    public void setPrimaryItem(@NonNull ViewGroup container,
                               int position,
                               @Nullable Object object) {
        // `object` is actually nullable. It's even in the dang source code which is hilariously
        // ridiculous:
        // `mAdapter.setPrimaryItem(this, mCurItem, curItem != null ? curItem.object : null);`
    }
}
0
SUPERCILEX