25

ViewPager因此,在屏幕方向更改后,我在销毁(删除)一页时遇到了问题。我将尝试在以下几行中描述问题。

我正在使用 的FragmentStatePagerAdapter适配器ViewPager和一个描述无限视图寻呼机应该如何工作的小界面。它背后的想法是您可以向右滚动直到到达ViewPager. 如果您可以从 API 调用加载更多结果,则会显示一个进度页面,直到结果出现。

到目前为止一切都很好,现在问题来了。如果在这个加载过程中,我旋转屏幕(这不会影响 API 调用,它基本上是一个AsyncTask),当调用返回时,应用程序崩溃给我这个异常:

E/AndroidRuntime(13471): java.lang.IllegalStateException: Fragment ProgressFragment{42b08548} is not currently in the FragmentManager
E/AndroidRuntime(13471):    at android.support.v4.app.FragmentManagerImpl.saveFragmentInstanceState(FragmentManager.java:573)
E/AndroidRuntime(13471):    at android.support.v4.app.FragmentStatePagerAdapter.destroyItem(FragmentStatePagerAdapter.java:136)
E/AndroidRuntime(13471):    at mypackage.OutterFragment$PagedSingleDataAdapter.destroyItem(OutterFragment.java:609)

在对库的代码进行了一些挖掘之后,似乎mIndex片段的数据字段比0这种情况下要少,这会引发该异常。

这是寻呼机适配器的代码:

static class PagedSingleDataAdapter extends FragmentStatePagerAdapter implements
        IEndlessPagerAdapter {

    private WeakReference<OutterFragment> fragment;
    private List<DataItem> data;
    private SparseArray<WeakReference<Fragment>> currentFragments = new SparseArray<WeakReference<Fragment>>();

    private ProgressFragment progressElement;

    private boolean isLoadingData;

    public PagedSingleDataAdapter(SherlockFragment fragment, List<DataItem> data) {
        super(fragment.getChildFragmentManager());
        this.fragment = new WeakReference<OutterFragment>(
                (OutterFragment) fragment);
        this.data = data;
    }

    @Override
    public Object instantiateItem(ViewGroup container, int position) {
        Object item = super.instantiateItem(container, position);
        currentFragments.append(position, new WeakReference<Fragment>(
                (Fragment) item));
        return item;
    }

    @Override
    public void destroyItem(ViewGroup container, int position, Object object) {
        currentFragments.put(position, null);
        super.destroyItem(container, position, object);
    }

    @Override
    public Fragment getItem(int position) {
        if (isPositionOfProgressElement(position)) {
            return getProgessElement();
        }

        WeakReference<Fragment> fragmentRef = currentFragments.get(position);
        if (fragmentRef == null) {
            return PageFragment.newInstance(args); // here I'm putting some info
                                                // in the args, just deleted
                                                // them now, not important
        }

        return fragmentRef.get();
    }

    @Override
    public int getCount() {
        int size = data.size();
        return isLoadingData ? ++size : size;
    }

    @Override
    public int getItemPosition(Object item) {
        if (item.equals(progressElement) && !isLoadingData) {
            return PagerAdapter.POSITION_NONE;
        }
        return PagerAdapter.POSITION_UNCHANGED;
    }

    public void setData(List<DataItem> data) {
        this.data = data;
        notifyDataSetChanged();
    }

    @Override
    public boolean isPositionOfProgressElement(int position) {
        return isLoadingData && position == data.size();
    }

    @Override
    public void setLoadingData(boolean isLoadingData) {
        this.isLoadingData = isLoadingData;
    }

    @Override
    public boolean isLoadingData() {
        return isLoadingData;
    }

    @Override
    public Fragment getProgessElement() {
        if (progressElement == null) {
            progressElement = new ProgressFragment();
        }
        return progressElement;
    }

    public static class ProgressFragment extends SherlockFragment {

        public ProgressFragment() {
        }

        @Override
        public View onCreateView(LayoutInflater inflater, ViewGroup container,
                Bundle savedInstanceState) {

            TextView progressView = new TextView(container.getContext());
            progressView.setGravity(Gravity.CENTER_HORIZONTAL
                    | Gravity.CENTER_VERTICAL);
            progressView.setText(R.string.loading_more_data);
            LayoutParams params = new LayoutParams(LayoutParams.FILL_PARENT,
                    LayoutParams.FILL_PARENT);
            progressView.setLayoutParams(params);

            return progressView;
        }
    }
}

下面的onPageSelected()回调,如果需要,它基本上会启动 api 调用:

 @Override
    public void onPageSelected(int currentPosition) {
        updatePagerIndicator(currentPosition);
        activity.invalidateOptionsMenu();
        if (requestNextApiPage(currentPosition)) {
            pagerAdapter.setLoadingData(true);
            requestNextPageController.requestNextPageOfData(this);
        }

现在,还值得一提的是,API 调用在传递结果后做了什么。这是回调:

@Override
public boolean onTaskSuccess(Context arg0, List<DataItem> result) {
    data = result;
    pagerAdapter.setLoadingData(false);
    pagerAdapter.setData(result);
    activity.invalidateOptionsMenu();

    return true;
}

好的,现在因为该setData()方法调用notifiyDataSetChanged(),这将为getItemPosition()当前在currentFragments数组中的片段调用 。当然,对于它返回的进度元素,POSITION_NONE因为我想删除这个页面,所以这基本上是destroyItem()PagedSingleDataAdapter. 如果我不旋转屏幕,一切正常,但正如我所说,如果我在显示进度元素并且 API 调用尚未完成时旋转它,destroyItem()则将在活动重新启动后调用回调。

ViewPager也许我还应该说我OutterFragmentViewPager. 我pagerAdapteronActivityCreated()回调中实例化OutterFragment并使用,setRetainInstance(true)这样当屏幕旋转时pagerAdapter保持不变(什么都不应该改变,对吧?),代码在这里:

if (pagerAdapter == null) {
    pagerAdapter = new PagedSingleDataAdapter(this, data);
}
pager.setAdapter(pagerAdapter);

if (savedInstanceState == null) {
    pager.setOnPageChangeListener(this);
    pager.setCurrentItem(currentPosition);
}

现在总结一下,问题是:

如果我尝试从ViewPager实例化后的进度元素中删除它并且活动被销毁并重新创建(屏幕方向更改)我得到上述异常(pagerAdapter保持不变,因此其中的所有内容也保持不变,引用等......因为未销毁的OutterFragment主机pagerAdapter仅与活动分离,然后重新连接)。可能它发生在片段管理器上,但我真的不知道是什么。

我已经尝试过的:

  1. 尝试使用另一种技术删除我的进度片段,即在onTaskSuccess()我试图从片段管理器中删除片段的回调中,没有奏效。

  2. 我还尝试隐藏进度元素,而不是从片段管理器中完全删除它。这工作了 50%,因为视图不再存在,但我有一个空页面,所以这不是我真正想要的。

  3. 我还尝试progressFragment在屏幕方向更改后(重新)将其附加到片段管理器,这也不起作用。

  4. 在重新创建活动后,我还尝试将进度片段删除然后再次添加到片段管理器,但没有奏效。

  5. 试图destroyItem()从回调中手动onTaskSuccess()调用(这真的非常难看)但没有奏效。

对不起,这么长的帖子,但我试图尽可能地解释这个问题,以便你们能够理解它。

任何解决方案,建议都非常感谢。

谢谢!

更新:解决方案 确定,所以这需要一段时间。问题是在进度片段上调用了 destroyItem() 回调两次,一次是在屏幕方向改变时,一次是在 api 调用完成后。这就是为什么例外。我找到的解决方案如下:继续跟踪 api 调用是否完成,并在这种情况下销毁进度片段,代码如下。

@Override
        public void destroyItem(ViewGroup container, int position, Object object) {
            if (object.equals(progressElement) && apiCallFinished == true) {
                apiCallFinished = false;
                currentFragments.put(position, currentFragments.get(position + 1));
                super.destroyItem(container, position, object);
            } else if (!(object.equals(progressElement))) {
                currentFragments.put(position, null);
                super.destroyItem(container, position, object);
            }
        }

然后这个 apiCallFinished 在适配器的构造函数中设置为 false,在onTaskSuccess()回调中设置为 true。它真的有效!

4

1 回答 1

4

更新:解决方案确定,所以这需要一段时间。问题是在进度片段上调用了 destroyItem() 回调两次,一次是在屏幕方向改变时,一次是在 api 调用完成后。这就是为什么例外。我找到的解决方案如下:继续跟踪 api 调用是否完成,并在这种情况下销毁进度片段,代码如下。

@Override
        public void destroyItem(ViewGroup container, int position, Object object) {
            if (object.equals(progressElement) && apiCallFinished == true) {
                apiCallFinished = false;
                currentFragments.put(position, currentFragments.get(position + 1));
                super.destroyItem(container, position, object);
            } else if (!(object.equals(progressElement))) {
                currentFragments.put(position, null);
                super.destroyItem(container, position, object);
            }
        }

然后这个 apiCallFinished 在适配器的构造函数中设置为 false,在 onTaskSuccess() 回调中设置为 true。它真的有效!

于 2013-03-27T19:23:52.540 回答