Question

我使用带有保留片段的简单活动来保存活动使用的一些数据。保留的片段使用加载器从内容提供者获取数据。在配置更改（屏幕旋转）时，将重新创建活动，并且LeakCanary库（保留片段 - ＆gt;加载器管理器 - >旧活动）报告旧实例泄漏。这与support-v4 23.0.0库（以及之前的版本）一起复制。带有保留片段的活动样本，其中包含泄漏（此处没有有用的代码，仅用于演示泄漏）：

package com.leaksample;

import android.database.Cursor;
import android.os.Bundle;
import android.provider.MediaStore;
import android.support.v4.app.Fragment;
import android.support.v4.app.FragmentManager;
import android.support.v4.app.LoaderManager;
import android.support.v4.content.CursorLoader;
import android.support.v4.content.Loader;
import android.support.v7.app.AppCompatActivity;

public class MainActivity extends AppCompatActivity {
    private ModelFragment mModelFragment;

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

        FragmentManager fm = getSupportFragmentManager();
        mModelFragment = (ModelFragment) fm.findFragmentByTag(ModelFragment.TAG);
        if (mModelFragment == null) {
            mModelFragment = new ModelFragment();
            fm.beginTransaction()
                    .add(mModelFragment, ModelFragment.TAG)
                    .commit();
            fm.executePendingTransactions();
        }
    }

    public static class ModelFragment extends Fragment implements LoaderManager.LoaderCallbacks<Cursor> {
        private static final String TAG = ModelFragment.class.getSimpleName();

        @Override
        public void onCreate(Bundle savedInstanceState) {
            super.onCreate(savedInstanceState);
            setRetainInstance(true);
            getLoaderManager().initLoader(0, null, this);
        }

        @Override
        public Loader<Cursor> onCreateLoader(int id, Bundle args) {
            return new CursorLoader(getActivity(), MediaStore.Images.Media.EXTERNAL_CONTENT_URI,
                    new String[]{MediaStore.Images.Media.DATA}, null, null, null);
        }

        @Override
        public void onLoadFinished(Loader<Cursor> loader, Cursor data) {
        }

        @Override
        public void onLoaderReset(Loader<Cursor> loader) {
        }
    }
}

来自LeakCanary的堆栈：

08-29 19:52:34.129  15271-16632/com.leaksample D/LeakCanary﹕ In com.leaksample:1.0:1.
08-29 19:52:34.129  15271-16632/com.leaksample D/LeakCanary﹕ * com.leaksample.MainActivity has leaked:
08-29 19:52:34.129  15271-16632/com.leaksample D/LeakCanary﹕ * GC ROOT thread java.lang.Thread.<Java Local> (named 'Binder_1')
08-29 19:52:34.129  15271-16632/com.leaksample D/LeakCanary﹕ * references android.view.ViewRootImpl.mContext
08-29 19:52:34.129  15271-16632/com.leaksample D/LeakCanary﹕ * references com.leaksample.MainActivity.mModelFragment
08-29 19:52:34.129  15271-16632/com.leaksample D/LeakCanary﹕ * references com.leaksample.MainActivity$ModelFragment.mLoaderManager
08-29 19:52:34.129  15271-16632/com.leaksample D/LeakCanary﹕ * references android.support.v4.app.LoaderManagerImpl.mHost
08-29 19:52:34.129  15271-16632/com.leaksample D/LeakCanary﹕ * references android.support.v4.app.FragmentActivity$HostCallbacks.this$0
08-29 19:52:34.129  15271-16632/com.leaksample D/LeakCanary﹕ * leaks com.leaksample.MainActivity instance

也许我做错了什么并且忘了拨打某些close或release方法？我认为将保留的片段与加载器一起使用是一种常见的模式，不应该是内存泄漏。

Answer 1

您对ModelFragment中的getLoaderManager（）的调用最终会创建一个新的LoaderManagerImpl实例，该实例包含对FragmentActivity的引用。见下文：

// FragmentActivity.java

LoaderManagerImpl getLoaderManager(String who, boolean started, boolean create) {
        if (mAllLoaderManagers == null) {
            mAllLoaderManagers = new SimpleArrayMap<String, LoaderManagerImpl>();
        }
        LoaderManagerImpl lm = mAllLoaderManagers.get(who);
        if (lm == null) {
            if (create) {
                lm = new LoaderManagerImpl(who, this, started);
                mAllLoaderManagers.put(who, lm);
            }
        } else {
            lm.updateActivity(this);
        }
        return lm;
    }

从进一步检查FragmentActivity时，看起来当Fragment保留其实例时，不会删除加载器，并且仍然保留对被破坏活动的引用。

@Override
    public final Object onRetainNonConfigurationInstance() {
        ...
            for (int i=0; i<N; i++) {
                LoaderManagerImpl lm = loaders[i];
                if (lm.mRetaining) {
                    retainLoaders = true;
                } else {
                    lm.doDestroy();
                    mAllLoaderManagers.remove(lm.mWho);
                }
            }
    ...
    }

解决此问题的一种方法是更改ModelFragment以使retainInstance为false。

Answer 2

除了使用下一个解决方法之外，我还没有找到最佳解决方案。我已将getLoaderManager().initLoader(0, null, this);调用getActivity().getSupportLoaderManager().initLoader(0, null, this);替换为使用活动的加载程序管理器而不是片段的加载程序管理器。我不知道这可能产生哪些副作用，但似乎有效。如果在活动和片段中都使用加载器，则应确保没有潜在的加载器ID冲突。此外，我还将initLoader方法调用从onCreate移至onActivityCreated（从onCreate调用时，加载程序停止以在配置更改后接收内容更新。）

保留片段泄漏

2 个答案: