旋转后,片段事务会产生IllegalStateException

时间:2013-06-20 07:20:03

标签: android android-fragments android-viewpager fragmentstatepageradapter

我有Fragments和BroadcastManager的问题。 在我的应用程序中,我切换到一个主要活动并使用新的NavigationDrawer。所有内容都包含在片段中。

One Fragment(搜索用户)包含两个选项卡(按名称搜索,按标准搜索),使用Android设计网站上的导航模式“横向导航”:它有一个带有FragmentStatePagerAdapter的ViewPager。

|  Main Activity  |    |  Main Activity  |
-------------------    -------------------
| Search Fragment |    | Search Fragment |
| >Tab 1< | Tab 2 |    | Tab 1 | >Tab 2< |
|    View Pager   |    |    View Pager   |
|   Fragment 1    |    |   Fragment 2    |

我希望两个标签都使用相同的操作栏(选项)菜单:搜索操作。 但只有活跃的片段才能对它做出反应。

我尝试过不同的方法,最烦人的是我无法直接从视图分页器中获取当前片段(不依赖于非API实现细节)。

方法

我现在使用LocalBroadcast来通知片段已经点击了搜索。每个片段在onResume中注册一个小的Wrapper-Receiver(并在onPause中删除它),它将onReceive转发给片段本身(下面显示的方法)。 我重写setMenuVisibility这是FragmentStatePagerAdpater在Fragments上调用的回调,以了解哪个是活动片段。 只有菜单集可见的片段才会对广播作出反应。

ActionBar Tab -> ViewPager -> ViewPagerAdapter -> Fragment.setMenuVisibility
ActionBar Menu Action -> Broadcast ->  ... -> BroadcastReceiver -> Fragments

两个片段都会触发片段事务以显示搜索结果并将其添加到后栈。

问题 片段事务通常起作用,但是当我旋转设备然后按搜索时,我得到一个IllegalStateException(在onSaveInstanceState之后无法提交)。

@Override
    public void onReceive(Context context, Intent intent) {
        if (m_menuVisible) {
            final String s = ((TextView) getView().findViewById(R.id.search_text)).getText().toString();

            SearchResultsFragment f = new UserSearchResultsFragment();
            Bundle b = new Bundle();
            b.putString(SearchResultsFragment.EXTRA_SEARCHNAME, s);
            f.setArguments(b);

            FragmentTransaction transaction = getSherlockActivity().getSupportFragmentManager().beginTransaction();
            transaction.replace(R.id.fragment_container_frame, f).addToBackStack(null).commit(); // <<< CRASH
        }
    }

我尝试将提交更改为commitAllowingStateLoss,但随后我收到IllegalStateException,其中“Activity已被销毁”。

你知道这里出了什么问题吗?我不知道该做什么......


附加代码:

MainActivities onCreate(基于NavigationDrawer示例)和selectItem

@Override
protected void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    requestWindowFeature(Window.FEATURE_INDETERMINATE_PROGRESS);
    setContentView(R.layout.fragment_container_layout);

    setupDrawer(); // Sets up the drawer layout, adapter, etc.

    if (savedInstanceState == null) {
        selectItem(0); // Selects and adds the fragment(s) for the position
    }

    // Other setup stuff
}

private void selectItem(int position) {
        // update the main content by replacing fragments

        Fragment f = null;
        switch (position) {
            case 0:
                f = ...
                break;
            case 1:
                ...
            default:
                throw new IllegalArgumentException("Could not find right fragment");
        }
        f.setRetainInstance(true);
        m_drawerList.setItemChecked(position, true);
        setTitle(mDrawerTitles.get(position).titleRes);

        // Hide any progress bar that might be visible in the actionbar
        setProgressBarIndeterminateVisibility(false);

        // When we select something from the navigation drawer, the back stack is discarded
        final FragmentManager fm = getSupportFragmentManager();
        fm.popBackStack(null, FragmentManager.POP_BACK_STACK_INCLUSIVE);

        fm.beginTransaction().replace(R.id.fragment_container_frame, f).commit();

        // update selected item and title, then close the drawer
        m_drawerLayout.closeDrawer(GravityCompat.START);
    }

故障标签片段中的FragmentStatePagerAdapter:

protected class TabPagerAdapter extends FragmentStatePagerAdapter {

        private List<String> m_fragmentTags = new ArrayList<String>();

        public TabPagerAdapter(FragmentManager fm) {
            super(fm);
        }

        public void addFragments(List<String> list) {
            ViewPager pager = (ViewPager) getView().findViewById(R.id.viewpager);

            startUpdate(pager);
            for (String tag : list) {
                m_fragmentTags.add(tag);
            }
            finishUpdate(pager);
            notifyDataSetChanged();
        }

        public void addFragment(String tag) {
            addFragments(Collections.singletonList(tag));
        }

        @Override
        public Fragment getItem(int pos) {
            // CreateFragmentForTag: Retrieves the classes, instantiates the fragment
            // Does not do retainInstance or any transaction there.
            return createFragmentForTag(m_fragmentTags.get(pos));
        }

        @Override
        public int getCount() {
            return m_fragmentTags.size();
        }

}

项目的精简版,仅包含必需品,发布在https://docs.google.com/file/d/0ByjUMh5UybW7cnB4a0NQeUlYM0E/edit?usp=sharing

重现如下:启动,您会看到两个标签。选择其中一个并单击ActionBar中的搜索 - &gt;片段交易有效。使用返回键,旋转设备并重复 - &gt;崩溃! [在赏金结束或发现错误后,可能无法访问该文件。]

修改: (1)澄清onReceive生活在片段的上下文中 (2)增加了主要活动的代码

2 个答案:

答案 0 :(得分:3)

使用commit()时最初获得错误的原因是,在为onSaveInstanceState()调用包含活动之前必须提交事务,这就是使用{错误修正的原因{1}},允许在提交后可能丢失状态。

在此方法更改后,您将在方向更改期间获得commitAllowingStateLoss ()异常,因为片段提交保留对在方向更改期间被破坏的原始活动的引用。因此,您可以定义片段容器并将片段添加到父活动的Activity has been destroyed中的容器,而不是在布局xml中定义片段,以确保在重新创建活动时,您还创建了一个新片段并提交

您可以在此处找到有关setRetainInstance的详细信息。

答案 1 :(得分:2)

将我的评论转移到这里,然后扩展它。

第一个线索是你只在轮换后崩溃,而你有setRetainInstance(true)。首先,尝试将其设置为false。即使这可以防止崩溃,你仍然需要谨慎。它可能指向你现在因未保留片段实例而隐藏的其他一些错误。

首先有setRetainInstance(true)的特殊原因吗?

至于为什么你开始看到崩溃,我仍然不能指出这个问题。我怀疑旋转后调用了错误的onReceive - 但这只是在黑暗中拍摄的。

如果您可以创建应用程序的骨架并在某处发布代码,我可以查看它。我知道你已经总结了如何“转发”onReceive() s,但如果做得不对,那么就有可能在那里引入bug。

如果无法做到这一点,我建议这样做 - 将记录语句放在ActivityFragmentonReceive()的主要生命周期方法中。这会告诉你发生的确切顺序。