Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

项目中使用AdapterViewPager类会出现跳转bug #219

Closed
yogkin opened this issue Jul 27, 2018 · 6 comments
Closed

项目中使用AdapterViewPager类会出现跳转bug #219

yogkin opened this issue Jul 27, 2018 · 6 comments

Comments

@yogkin
Copy link

yogkin commented Jul 27, 2018

Bug Description:

Related Code:

  @Override
    public Object instantiateItem(ViewGroup container, int position) {
        Fragment f = (Fragment) super.instantiateItem(container, position);
        View view = f.getView();
        if (view != null)
            container.addView(view);
        return f;
    }

    @Override
    public void destroyItem(ViewGroup container, int position, Object object) {
        View view = mList.get(position).getView();
        if (view != null)
            container.removeView(view);
    }

分析原因

当切换viewpager会执行上面的2个方法,如果执行了destroyItem() 则在页面跳转时候(比如跳转到其他Activity)会发生crash
原因应该是 viewpager会在跳转前执行removeViews()方法,导致NullPointerException 不知道我分析得对不对
最后感谢作者提供这么好的框架给我们学习

Bug Log:

image

Others:

@JessYanCoding
Copy link
Owner

JessYanCoding commented Jul 28, 2018

你截的报错图片,倒是像你在 Activity 的 onPause 或和 onPause 相关的地方做了什么操作,导致了奔溃

你说的空指针,是否能截图具体报空指针的位置

你是否能肯定错误一定和 AdapterViewPager 有关,你是使用的 Activity.startActivity() 跳转到其他 Activity,和 Fragment 以及 AdapterViewPager 都没什么关系,你为什么会将错误定位到 AdapterViewPager 这个类上,如果 AdapterViewPager 有报错就请截图

如果确定报错在 AdapterViewPager 中,那就请不使用 AdapterViewPager,使用他的父类 FragmentStatePagerAdapter 自行实现,看是否还是有同样的报错,如果还有报错说明不是 AdapterViewPager 的问题,需要自行排除是否是自己业务引起的问题

@yogkin
Copy link
Author

yogkin commented Jul 28, 2018

嗯,在AdapterViewPager移除这两个方法之后正常了

@JessYanCoding
Copy link
Owner

AdapterViewPager 的具体报错位置?

@yogkin
Copy link
Author

yogkin commented Jul 28, 2018

image

@yogkin
Copy link
Author

yogkin commented Jul 28, 2018

#134

@JessYanCoding
Copy link
Owner

cf239c4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants