如何正确处理ViewPager和嵌套片段的屏幕旋转？

Question

我有一个活动（Activity），其中包含一个碎片（Fragment）。这个碎片布局由一个带有几个碎片（实际上只有两个）的视图翻页器（ViewPager）组成。

当视图翻页器被创建时，它的适配器（Adapter）也会被创建，getItem 方法被调用，我的子碎片（Sub Fragments）就会被创建。这很棒。

现在，当我旋转屏幕时，框架会处理碎片的重新创建，在主碎片中的 onCreate 中会再次创建适配器，但是 getItem 永远不会被调用，因此我的适配器存储了错误的引用（实际上是 null），而不是这两个碎片。

我发现的是，碎片管理器（即子碎片管理器）包含一个名为 mActive 的碎片数组，当然这个数组无法从代码中访问。但是有这个 getFragment 方法：

@Override
public Fragment getFragment(Bundle bundle, String key) {
    int index = bundle.getInt(key, -1);
    if (index == -1) {
        return null;
    }
    if (index >= mActive.size()) {
        throwException(new IllegalStateException("Fragement no longer exists for key "
                + key + ": index " + index));
    }
    Fragment f = mActive.get(index);
    if (f == null) {
        throwException(new IllegalStateException("Fragement no longer exists for key "
                + key + ": index " + index));
    }
    return f;
}

我不会评论这个错别字 :)
这是我在适配器构造函数中实现更新片段引用的hack:

// fm holds a reference to a FragmentManager
Bundle hack = new Bundle();
try {
    for (int i = 0; i < mFragments.length; i++) {
        hack.putInt("hack", i);
        mFragments[i] = fm.getFragment(hack, "hack");
    }
} catch (Exception e) {
    // No need to fail here, likely because it's the first creation and mActive is empty
}

我不感到骄傲。这个方法可行，但很丑陋。在屏幕旋转后，有什么实际的方法可以获得一个有效的适配器？

PS：这里是完整代码

- Benoit Duffez

---

这里有同样的问题：https://dev59.com/J3HYa4cB1Zd3GeqPM5J7 你找到解决方法了吗？ - Poutrathor

不，我还在使用我的hack。我已经检查了你提到的问题，但这并没有解决我的问题。 - Benoit Duffez

你说得对，我在这里忘记修复一个错误了（但是我已经在我的代码中修复了）。我会编辑我的答案并发布所有的代码。 - Benoit Duffez

请看这个链接：http://chat.stackoverflow.com/rooms/40748/discussion-between-poutrathor-and-bicou - Benoit Duffez

也许这对你有用：https://dev59.com/kuo6XIcBkEYKwwoYSCc1 - Hoang Nguyen Huu

Answer 1

我曾遇到相同的问题 - 我假设你正在为你的页面适配器（因为 getItem() 是特定于 FragmentPagerAdapter）创建自己的子类。

我的解决方案是改为子类化 PagerAdapter 并自己处理片段的创建/删除（重新实现一些 FragmentPagerAdapter 代码）：

public class ListPagerAdapter extends PagerAdapter {
    FragmentManager fragmentManager;
    Fragment[] fragments;

    public ListPagerAdapter(FragmentManager fm){
        fragmentManager = fm;
        fragments = new Fragment[5];
    }

    @Override
    public void destroyItem(ViewGroup container, int position, Object object) {
        assert(0 <= position && position < fragments.length);
        FragmentTransaction trans = fragmentManager.beginTransaction();
        trans.remove(fragments[position]);
        trans.commit();
        fragments[position] = null;
}

    @Override
    public Fragment instantiateItem(ViewGroup container, int position){
        Fragment fragment = getItem(position);
        FragmentTransaction trans = fragmentManager.beginTransaction();
        trans.add(container.getId(),fragment,"fragment:"+position);
        trans.commit();
        return fragment;
    }

    @Override
    public int getCount() {
        return fragments.length;
    }

    @Override
    public boolean isViewFromObject(View view, Object fragment) {
        return ((Fragment) fragment).getView() == view;
    }

    public Fragment getItem(int position){
        assert(0 <= position && position < fragments.length);
        if(fragments[position] == null){
            fragments[position] = ; //make your fragment here
        }
        return fragments[position];
    }
}

希望这有所帮助。

- Josh Hunt

---

太棒了！谢谢！ - Denis Nek

是的，我花了3年时间才接受你的解决方案。谢谢！:D - Benoit Duffez

这是一个解决方案，但默认行为是意外的，或者我可能漏掉了什么？你有什么想法是背后的原因吗？ - Rany Albeg Wein

不错的答案，但是为了一个稍微简单一点的解决方案，请查看https://dev59.com/kuo6XIcBkEYKwwoYSCc1#29288093。 - piccy

Answer 2

为什么上面的解决方案如此复杂？看起来有些过度设计。我只需要在扩展自FragmentPagerAdapter的类中将旧引用替换为新引用即可解决它。

 @Override
public Object instantiateItem(ViewGroup container, int position) {
    frags[position] = (Fragment) super.instantiateItem(container, position);
    return frags[position];
}

所有适配器的代码都长成这样

public class RelationsFragmentsAdapter extends FragmentPagerAdapter {

private final String titles[] = new String[3];
private final Fragment frags[] = new Fragment[titles.length];

public RelationsFragmentsAdapter(FragmentManager fm) {
    super(fm);
    frags[0] = new FriendsFragment();
    frags[1] = new FriendsRequestFragment();
    frags[2] = new FriendsDeclinedFragment();

    Resources resources = AppController.getAppContext().getResources();

    titles[0] = resources.getString(R.string.my_friends);
    titles[1] = resources.getString(R.string.my_new_friends);
    titles[2] = resources.getString(R.string.followers);
}

@Override
public CharSequence getPageTitle(int position) {
    return titles[position];
}

@Override
public Fragment getItem(int position) {
    return frags[position];
}

@Override
public int getCount() {
    return frags.length;
}

@Override
public Object instantiateItem(ViewGroup container, int position) {
    frags[position] = (Fragment) super.instantiateItem(container, position);
    return frags[position];
}

}

- F0RIS

---

谢谢，这是一个很好的答案。轻量级、快速、易于实现，没有静态（泄漏）引用。而且很容易理解它为什么有效。 - ExactaBox

但是每次你反复旋转屏幕时，这将创建新的片段实例。 - Josef Vancura

Answer 3

我给出的答案与Joshua Hunt的有些相似，但是通过在finishUpdate方法中提交事务可以获得更好的性能。每次更新只需要一个事务而不是两个。

以下是代码：

private class SuchPagerAdapter extends PagerAdapter{

    private final FragmentManager mFragmentManager;
    private SparseArray<Fragment> mFragments;
    private FragmentTransaction mCurTransaction;

    private SuchPagerAdapter(FragmentManager fragmentManager) {
        mFragmentManager = fragmentManager;
        mFragments = new SparseArray<>();
    }

    @Override
    public Object instantiateItem(ViewGroup container, int position) {
        Fragment fragment = getItem(position);
        if (mCurTransaction == null) {
            mCurTransaction = mFragmentManager.beginTransaction();
        }
        mCurTransaction.add(container.getId(),fragment,"fragment:"+position);
        return fragment;
    }

    @Override
    public void destroyItem(ViewGroup container, int position, Object object) {
        if (mCurTransaction == null) {
            mCurTransaction = mFragmentManager.beginTransaction();
        }
        mCurTransaction.detach(mFragments.get(position));
        mFragments.remove(position);
    }

    @Override
    public boolean isViewFromObject(View view, Object fragment) {
        return ((Fragment) fragment).getView() == view;
    }

    public Fragment getItem(int position) {         
        return YoursVeryFragment.instantiate();
    }

    @Override
    public void finishUpdate(ViewGroup container) {
        if (mCurTransaction != null) {
            mCurTransaction.commitAllowingStateLoss();
            mCurTransaction = null;
            mFragmentManager.executePendingTransactions();
        }
    }


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

}

- simekadam

---

来自mitenko的评论：在instantiateItem中没有填充mFragments，需要在里面加上mFragments.put(position, fragment);，否则你会遇到这个错误：Trying to remove fragment from view gives me NullPointerException on mNextAnim.。 - Artjom B.

Answer 4

问题在于FragmentPageAdapter中的getItem()名称错误。它应该被命名为createItem()。 因为它的工作方式是用于创建片段，而不安全地调用它来查询/查找片段。

我的建议是复制当前的FragmentPagerAdapter并进行更改：

添加：

    public abstract Fragment createFragment(int position);

并将getItem更改为：

public Fragment getItem(int position) {
    if(containerId!=null) {
        final long itemId = getItemId(position);
        String name = makeFragmentName(containerId, itemId);
        return mFragmentManager.findFragmentByTag(name);
    } else {
        return null;
    }
}

最后将其添加到instantiateItem中：

    if(containerId==null)
        containerId = container.getId();
    else if(containerId!=container.getId())
        throw new RuntimeException("Container id not expected to change");

完整代码请参见此处的gist

我认为这个实现更安全、更易于使用，并且与Google工程师原始适配器具有相同的性能。

- lujop

Answer 5

我的代码：

    public class SampleAdapter extends FragmentStatePagerAdapter {

    private Fragment mFragmentAtPos2;
    private FragmentManager mFragmentManager;
    private Fragment[] mFragments = new Fragment[3];


    public SampleAdapter(FragmentManager mgr) {
        super(mgr);
        mFragmentManager = mgr;
        Bundle hack = new Bundle();
        try {
            for (int i = 0; i < mFragments.length; i++) {
                hack.putInt("hack", i);
                mFragments[i] = mFragmentManager.getFragment(hack, "hack");
            }
        } catch (Exception e) {
            // No need to fail here, likely because it's the first creation and mActive is empty
        }

    }

    public void switchFrag(Fragment frag) {

        if (frag == null) {
            Dbg.e(TAG, "- switch(frag) frag is NULL");
            return;
        } else Dbg.v(TAG, "- switch(frag) - frag is " + frag.getClass());
        // We have to check for mFragmentAtPos2 null in case of first time (only Mytrips fragment being instatiante).
        if (mFragmentAtPos2!= null) 
            mFragmentManager.beginTransaction()  
                .remove(mFragmentAtPos2)
                .commit();
        mFragmentAtPos2 = frag;
        notifyDataSetChanged();
    }

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

    @Override
    public int getItemPosition(Object object) {
        Dbg.v(TAG,"getItemPosition : "+object.getClass());
        if (object instanceof MyTripsFragment
                || object instanceof FindingDriverFragment
                || object instanceof BookingAcceptedFragment
                || object instanceof RideStartedFragment
                || object instanceof RideEndedFragment
                || object instanceof ContactUsFragment
                )
            return POSITION_NONE;
        else return POSITION_UNCHANGED;

    }

    @Override
    public Fragment getItem(int position) {
        Dbg.v("SampleAdapter", "getItem called on: "+position);
        switch (position) {
        case 0: 
            if (snapbookFrag==null) {
                snapbookFrag = new SnapBookingFragment();
                Dbg.e(TAG, "snapbookFrag created");
            }
            return snapbookFrag;
        case 1: 
            if(bookingFormFrag==null) {
                bookingFormFrag = new BookingFormFragment();
                Dbg.e(TAG, "bookingFormFrag created");
            }
            return bookingFormFrag;
        case 2: 
            if (mFragmentAtPos2 == null) {
                myTripsFrag = new MyTripsFragment();
                mFragmentAtPos2 = myTripsFrag;
                return mFragmentAtPos2;
            }
            return mFragmentAtPos2;
        default:
            return(new SnapBookingFragment());
        }
    }
}

- Poutrathor

Answer 6

关于simekadam的解决方案，mFragments在instantiateItem中没有被填充，需要在里面加上mFragments.put(position, fragment);，否则你会遇到这个错误：尝试从视图中删除片段时，在mNextAnim上给我NullPointerException。

- mitenko

---

我已将您的评论添加到答案中。只要确实有意义且不改变含义，您也可以建议对帖子进行编辑。 - Artjom B.