ViewPager在方向更改时崩溃 - 片段堆叠并导致OutOfMemoryException

时间:2016-09-10 14:13:09

标签: android android-fragments android-asynctask android-viewpager android-bitmap

我的项目结构如下:

MainActivity-> PagerFragment-> PagerViewAdapter,它包含多个PagerImageFragments

我在几次方向更改后得到OutOfMemoryException,但是寻呼机显示位图"有效地" (我使用了AsyncTask / LruCache)。然后我注意到app使用的内存在每次方向更改后都会自动增加,从日志中我可以看到每次调用PagerFragment中的onCreate()两次。我认为因为我的片段被重新创建和堆叠,导致问题。如何阻止这种情况发生?

我尝试过设置

@Override
protected void onSaveInstanceState(final Bundle outState) {
// super.onSaveInstanceState(outState);
}

哪个对我毫无用处。然后我试了一下:

setRetainInstance(true);

这使我的PagerFragment只重新创建一次,但那仍然不是我想要的。

编辑:我稍微更改了MainActivity,因此如果没有保存的实例,它只会执行初始化操作,并在启动新的实例时删除所有片段。我删除了setRetainInstance(true)。我现在可以多次更改方向(它仍将以OOM异常结束,但触发它需要更长的时间)。但现在又出现了另一个问题 - 如果我将PagerFragment留给另一个片段并返回它会崩溃。

MainActivity:

public class MainActivity extends AppCompatActivity {
private final String TAG = "LOG";

@Override
protected void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    setContentView(R.layout.activity_main);

    if (savedInstanceState==null) {
    ...
        FragmentTransaction fragmentTransaction = getSupportFragmentManager().beginTransaction();
        PagerFragment fragment = new PagerFragment();
        fragmentTransaction.add(R.id.fragment_container, fragment);
        fragmentTransaction.commit();
    ...
        switchContent(new PagerFragment());
    }
}

    public void switchContent(Fragment fragment) {
    FragmentManager fm = getSupportFragmentManager();
    List<Fragment> fragmentList = fm.getFragments();
    if (fragmentList != null) {
        for (Fragment frag : fragmentList ) {
            Log.d(TAG, "switchContent: destroying "+frag);
            fm.beginTransaction()
              .remove(frag).commit();
        }
    }

    fm.beginTransaction()
      .replace(R.id.fragment_container, fragment)
      .commit();
}

Inside PagerFragment:

PagerAdapter adapter = new PagerViewAdapter(getFragmentManager(), pdfFactory.getPageCount());
    ViewPager pager = (ViewPager) getActivity().findViewById(R.id.pager);
    pager.setAdapter(adapter);
    pager.setPageMargin((int) getResources().getDimension(R.dimen.activity_horizontal_margin));
    pager.setCurrentItem(position);
    pager.setOffscreenPageLimit(2);

PagerViewAdapter:

public class PagerViewAdapter extends FragmentStatePagerAdapter {
    private final int size;

    public PagerViewAdapter(FragmentManager fm, int size) {
        super(fm);
        this.size = size;
    }

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

    @Override
    public Fragment getItem(int position) {
        return PagerImageFragment.newInstance(position);
    }

}

PagerImageFragment:

public class PagerImageFragment extends Fragment {
private PhotoView photoView;
int position;
private MyCache<Integer, Bitmap> myCache;

public static PagerImageFragment newInstance(Integer pos) {
    final PagerImageFragment f = new PagerImageFragment();
    final Bundle args = new Bundle();
    args.putInt("position", pos);
    f.setArguments(args);

    Log.d("DISPLAY", "Bundling...");

    return f;
}

public PagerImageFragment() {}

@Override
public void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);

    //Initialize the cache
    myCache = new MyCache<>( 5 * 1024 * 1024 );

}

@Override
public View onCreateView(LayoutInflater inflater, ViewGroup container,
                         Bundle savedInstanceState) {
    // Inflate and locate the main ImageView
    final View v = inflater.inflate(R.layout.pager_element, container, false);
    photoView = (PhotoView) v.findViewById(R.id.pager_image);
    Bitmap image = myCache.get(position);
    if (image != null) {
        photoView.setImageBitmap(image);
    }
    else {
        new SetImageTask(photoView, myCache).execute(position);
    }
    return v;
}

例外(编辑后):

java.lang.NullPointerException: Attempt to write to field 'int android.support.v4.app.Fragment.mNextAnim' on a null object reference
                                                                             at android.support.v4.app.BackStackRecord.run(BackStackRecord.java:770)
                                                                             at android.support.v4.app.FragmentManagerImpl.execPendingActions(FragmentManager.java:1677)
                                                                             at android.support.v4.app.FragmentManagerImpl$1.run(FragmentManager.java:536)
                                                                             at android.os.Handler.handleCallback(Handler.java:739)
                                                                             at android.os.Handler.dispatchMessage(Handler.java:95)
                                                                             at android.os.Looper.loop(Looper.java:155)
                                                                             at android.app.ActivityThread.main(ActivityThread.java:5696)
                                                                             at java.lang.reflect.Method.invoke(Native Method)
                                                                             at java.lang.reflect.Method.invoke(Method.java:372)
                                                                             at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:1028)
                                                                             at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:823)

3 个答案:

答案 0 :(得分:1)

在清单

中的活动中添加此内容

<强>机器人:configChanges =&#34;取向| keyboardHidden |屏幕尺寸&#34;

<activity android:name="MainActivity"
      android:configChanges="orientation|keyboardHidden|screenSize">
</activity>
方向更改时将调用

onConfigurationChanged ,但不会重新创建您的活动,而是会调用 onConfigurationChanged ,您必须管理方向更改如果有是任何。因此,它可以防止 OutOfMemoryException 问题,因为在方向更改时重新创建了所有内容。

@Override
public void onConfigurationChanged(Configuration newConfig) {
    super.onConfigurationChanged(newConfig);
    // Handle orientation changes here
}

答案 1 :(得分:0)

首先,您不应该使用getFragmentManager作为寻呼机适配器。而不是它,使用getChildFragmentManager

PagerAdapter adapter = new PagerViewAdapter(getChildFragmentManager(), pdfFactory.getPageCount());

如果它没有帮助,请添加Leak Canary库以构建.gradle依赖项以检测内存泄漏https://github.com/square/leakcanary

答案 2 :(得分:0)

原来它是FragmentStateViewPager中的一个错误 - 它不会删除视图。修复是使用它,虽然不完美,但最好的事情我可以找到https://github.com/adamsp/FragmentStatePagerIssueExample/blob/master/app/src/main/java/com/example/fragmentstatepagerissueexample/app/FixedFragmentStatePagerAdapter.java