Переупорядочить страницы в FragmentStatePagerAdapter, используя getItemPosition (объект 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));
}
}
Я столкнулся с двумя отдельными типами поведения, которые кажутся неправильными.
-
Если я сразу вызову 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.
-
Если я сначала провёл страницу B, вызывается getItem(2)
, создается страница C, а mFragments
теперь имеет размер 3 (это предотвратит мгновенное появление сбоя). Затем я вернусь на страницу A, и страница C будет уничтожена, как и должно быть (поскольку она находится на расстоянии 2 страницы, и я использую ограничение по умолчанию для экрана на 1). Теперь я называю toggleState()
. Страница B теперь уничтожена. Однако страница C НЕ воссоздана! Это означает, что когда я теперь прокручиваю вправо, я получаю пустую страницу.
Во-первых, было бы неплохо узнать, правильно ли я, и на самом деле это ошибки, или я делаю что-то неправильно. Если они являются ошибками, может ли кто-нибудь предложить обходное решение (кроме отладки и перестройки библиотеки поддержки)? Разумеется, кто-то должен был успешно переопределить getItemPosition(Object object)
(кроме того, что все установлено на POSITION_NONE
)?
Я использую текущую версию (10) библиотеки поддержки.
Ответы
Ответ 1
Посмотрев на источник FragmentStatePagerAdapter, я понял, что происходит не так. FragmentStatePagerAdapter кэширует фрагменты и сохраненные состояния в ArrayLists: mFragments
и mSavedState
. Но когда фрагменты переупорядочены, нет механизма для переупорядочения элементов mFragments
и mSavedState
. Следовательно, адаптер предоставит неправильные фрагменты пейджеру.
Я зарегистрировал проблему для этого и приложил исправленную реализацию (NewFragmentStatePagerAdapter.java) к проблеме. В исправлении я добавил функцию getItemId()
в FragmentStatePagerAdapter. (Это отражает реализацию переупорядочения в FragmentPagerAdapter.) Массив элемента itemIds по позиции адаптера сохраняется в любое время. Затем в notifyDataSetChanged()
адаптер проверяет, изменился ли массив itemsIds. Если он имеет, то mFragments
и mSavedState
соответственно переупорядочиваются. Дальнейшие изменения можно найти в destroyItem()
, saveState()
и restoreState()
.
Чтобы использовать этот класс, getItemPosition()
и getItemId()
должны быть последовательно реализованы с помощью getItem()
.
Ответ 2
Для меня работал один из ответов вопрос. Ответы # 20 # 21. Ссылка на решение https://gist.github.com/ypresto/8c13cb88a0973d071a64.
Лучшее решение, работает для обновления страниц, а также для переупорядочения. Только в этом решении Адаптер не бросает IndexOutOfBoundsExeption при уничтожении элемента (в методе destroyItem), который известен как ошибка для других решений.
Ответ 3
Я пересмотрел существующее решение в 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 even in the dang source code which is hilariously
// ridiculous:
// `mAdapter.setPrimaryItem(this, mCurItem, curItem != null ? curItem.object : null);`
}
}
Ответ 4
Хорошо, я нашел решение. Это исправляет проблему изменения порядка фрагментов окна просмотра, если вы динамически создаете/изменяете новые вкладки.
Используйте этот класс вместо FragmentStatePagerAdapter.java
package android.support.v4.app;
import android.os.Bundle;
import android.os.Parcelable;
import android.support.v4.view.PagerAdapter;
import android.util.Log;
import android.view.View;
import android.view.ViewGroup;
import java.util.ArrayList;
public abstract class NewFragmentStatePagerAdapter extends PagerAdapter {
private static final String TAG = "FragmentStatePagerAdapt";
private static final boolean DEBUG = false;
private final FragmentManager mFragmentManager;
private FragmentTransaction mCurTransaction = null;
private ArrayList<Fragment.SavedState> mSavedState = new ArrayList<Fragment.SavedState>();
private ArrayList<Fragment> mFragments = new ArrayList<Fragment>();
private Fragment mCurrentPrimaryItem = null;
public NewFragmentStatePagerAdapter(FragmentManager fm) {
mFragmentManager = fm;
}
/**
* Return the Fragment associated with a specified position.
*/
public abstract Fragment getItem(int position);
@Override
public void startUpdate(ViewGroup container) {
if (container.getId() == View.NO_ID) {
throw new IllegalStateException("ViewPager with adapter " + this
+ " requires a view id");
}
}
public void destroyItemState(int position) {
mFragments.remove(position);
mSavedState.remove(position);
}
@Override
public Object instantiateItem(ViewGroup container, int position) {
// If we already have this item instantiated, there is nothing
// to do. This can happen when we are restoring the entire pager
// from its saved state, where the fragment manager has already
// taken care of restoring the fragments we previously had instantiated.
if (mFragments.size() > position) {
Fragment f = mFragments.get(position);
if (f != null) {
return f;
}
}
if (mCurTransaction == null) {
mCurTransaction = mFragmentManager.beginTransaction();
}
Fragment fragment = getItem(position);
if (DEBUG) Log.v(TAG, "Adding item #" + position + ": f=" + fragment);
if (mSavedState.size() > position) {
Fragment.SavedState fss = mSavedState.get(position);
if (fss != null) {
fragment.setInitialSavedState(fss);
}
}
while (mFragments.size() <= position) {
mFragments.add(null);
}
fragment.setMenuVisibility(false);
fragment.setUserVisibleHint(false);
mFragments.set(position, fragment);
mCurTransaction.add(container.getId(), fragment);
return fragment;
}
@Override
public void destroyItem(ViewGroup container, int position, Object object) {
Fragment fragment = (Fragment) object;
if (mCurTransaction == null) {
mCurTransaction = mFragmentManager.beginTransaction();
}
if (DEBUG) Log.v(TAG, "Removing item #" + position + ": f=" + object
+ " v=" + ((Fragment)object).getView());
while (mSavedState.size() <= position) {
mSavedState.add(null);
}
mSavedState.set(position, fragment.isAdded()
? mFragmentManager.saveFragmentInstanceState(fragment) : null);
mFragments.set(position, null);
mCurTransaction.remove(fragment);
}
@Override
@SuppressWarnings("ReferenceEquality")
public void setPrimaryItem(ViewGroup container, int position, Object object) {
Fragment fragment = (Fragment)object;
if (fragment != mCurrentPrimaryItem) {
if (mCurrentPrimaryItem != null) {
mCurrentPrimaryItem.setMenuVisibility(false);
mCurrentPrimaryItem.setUserVisibleHint(false);
}
if (fragment != null) {
fragment.setMenuVisibility(true);
fragment.setUserVisibleHint(true);
}
mCurrentPrimaryItem = fragment;
}
}
@Override
public void finishUpdate(ViewGroup container) {
if (mCurTransaction != null) {
mCurTransaction.commitNowAllowingStateLoss();
mCurTransaction = null;
}
}
@Override
public boolean isViewFromObject(View view, Object object) {
return ((Fragment)object).getView() == view;
}
@Override
public Parcelable saveState() {
Bundle state = null;
if (mSavedState.size() > 0) {
state = new Bundle();
Fragment.SavedState[] fss = new Fragment.SavedState[mSavedState.size()];
mSavedState.toArray(fss);
state.putParcelableArray("states", fss);
}
for (int i=0; i<mFragments.size(); i++) {
Fragment f = mFragments.get(i);
if (f != null && f.isAdded()) {
if (state == null) {
state = new Bundle();
}
String key = "f" + i;
mFragmentManager.putFragment(state, key, f);
}
}
return state;
}
@Override
public void restoreState(Parcelable state, ClassLoader loader) {
if (state != null) {
Bundle bundle = (Bundle)state;
bundle.setClassLoader(loader);
Parcelable[] fss = bundle.getParcelableArray("states");
mSavedState.clear();
mFragments.clear();
if (fss != null) {
for (int i=0; i<fss.length; i++) {
mSavedState.add((Fragment.SavedState)fss[i]);
}
}
Iterable<String> keys = bundle.keySet();
for (String key: keys) {
if (key.startsWith("f")) {
int index = Integer.parseInt(key.substring(1));
Fragment f = mFragmentManager.getFragment(bundle, key);
if (f != null) {
while (mFragments.size() <= index) {
mFragments.add(null);
}
f.setMenuVisibility(false);
mFragments.set(index, f);
} else {
Log.w(TAG, "Bad fragment at key " + key);
}
}
}
}
}
}