最近我一直在研究 java/android 中的内存泄漏，几乎所有地方都说我应该使用具有弱引用的静态内部类而不是匿名类。
所以，在我的 android 应用程序中，我开始这样做，但很快就厌倦了，因为它有很多样板代码……我认为有一个我更愿意使用的替代解决方案，但我不确定它是否是在防止内存泄漏方面是静态内部类的有效替代方案。正如我之前所说，我还没有在其他任何地方看到建议的这个解决方案(都说使用静态内部类)，所以这就是为什么我不确定我的替代方案是否有效。

我将使用我的应用程序中的一个简单示例:
我有一个名为 WebClient 的类，它处理异步 Web 请求，它接受一个名为 iCallback 的接口(interface)，该接口(interface)将服务器的响应返回给调用者，在我的 Activity 中，一旦我得到这个回调，我需要关闭一个对话框，并可能执行一些 Activity 相关的东西(比如触发器 onBackPressed() 和 setResult())。
所以这是我创建的静态内部类:

private static class CallBack implements WebClient.ICallback
{
    private WeakReference<ProgressDialog> mProgDiag;
    private WeakReference<BaseActivity> mActivity;

    public CallBack(BaseActivity activity, ProgressDialog progDiag)
    {
        this.mProgDiag = new WeakReference<>(progDiag);
        this.mActivity = new WeakReference<>(activity);
    }

    @Override
    public void onCallback(String data)
    {
        String responseAsString = Utils.extractStringFromResponse(...);

        final BaseActivity parentActivity = mActivity.get();
        ProgressDialog dialog = mProgDiag.get();

        if(dialog != null)
        {
            dialog.dismiss();
        }

        if (responseAsString == null)
        {
            if(parentActivity != null)
            {
                Utils.makeServerErrorDialog(parentActivity,
                                            new iDialogButtonClickedListener()
                                            {
                                                @Override
                                                public void onDialogButtonClicked()
                                                {
                                                    parentActivity.onBackPressed();
                                                }
                                            });
            }

            return;
        }

        //everything is ok
        if (responseAsString.equals("1"))
        {
            if(parentActivity != null)
            {
                Intent result = new Intent();
                result.putExtra(...);

                parentActivity.setResult(Activity.RESULT_OK, result);
            }
        }

        else
        {
            Utils.reportErrorToServer(...);

            if(parentActivity != null)
            {
                parentActivity.setResult(Activity.RESULT_CANCELED);
            }
        }

        if(parentActivity != null)
        {
            parentActivity.onBackPressed();
        }
    }
}

所以对于我在这个静态内部类中需要的每个变量，我必须创建一个新的弱引用，然后检索对象本身，然后每次我想访问它时我都需要检查它是否为空...这似乎对我来说就像很多代码。

这是我建议的替代方案:

public abstract class BaseActivity extends AppCompatActivity
        implements WebClient.ICallback
{
    private static final String TAG = "BaseActivity";

    WebClient.ICallback mCallBack;
    ProgressDialog mProgDiag;

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

        setContentView(...);

        mCallBack = this;

        //some code to invoke a server request on button click
        //and passing mCallBack to the request
    }

    @Override
    public void onCallback(String data)
    {
        String responseAsString = Utils.extractStringFromResponse(...);

        mProgDiag.dismiss();

        if (responseAsString == null)
        {
            Utils.makeServerErrorDialog(this,
                                        new iDialogButtonClickedListener()
                                        {
                                            @Override
                                            public void onDialogButtonClicked()
                                            {
                                                onBackPressed();
                                            }
                                        });

            return;
        }

        //everything is ok
        if (responseAsString.equals("1"))
        {
            Intent result = new Intent();
            result.putExtra(...);

            setResult(Activity.RESULT_OK, result);
        }

        else
        {
            Utils.reportErrorToServer(...);

            setResult(Activity.RESULT_CANCELED);
        }

        onBackPressed();
    }

    @Override
    protected void onPause()
    {
        mCallBack = null;

        super.onPause();
    }

    @Override
    protected void onResume()
    {
        super.onResume();

        mCallBack = this;
    }
}

对我来说，这似乎更清晰:无需为我需要访问的每个变量创建和检索弱引用的实例，我可以直接调用 Activity 方法(例如 onBackPressed())，并且无需在任何地方检查 null。
在调用 callBack 方法之前，我现在唯一需要检查 null 的地方是在 WebClient 类中。

所以我的问题是，这种方法在防止内存泄漏方面是否达到了同样的效果？它是静态内部类的“有值(value)的”替代品吗？

最佳答案

不幸的是，您的方法行不通。通过在您的 Activity 中实现 WebClient.ICallback 而不是内部类，您无法消除泄漏。泄漏的发生不是因为对 Activity 和对话框的引用在匿名类、lambda 或非静态内部类实例中是隐式的；当 WebClient 在 Activity 消失时保留此引用时会发生这种情况(它不会被销毁，因为存在对它的强引用)。

在 Activity 暂停时设置为 null 的特殊 mCallBack 没有任何好处。同样，您可以简单地将您的 Activity 实例传递给 WebClient。现在有一个对您的 Activity 的强烈引用，它由不受您控制的某人(WebClient 的异步处理程序)管理。如果你不走运，异步处理程序会卡在某个地方，永远不会释放这个引用。

请仔细阅读本篇explanation .

请注意 WebView 本身可以 cause a memory leak ，如果不采取特殊措施!

关于java - 可能替代静态内部类以防止 android/java 中的内存泄漏？，我们在Stack Overflow上找到一个类似的问题： https://stackoverflow.com/questions/43779175/