如何修复这个涉及线程的 Android 内存泄漏？

Question

所以我发现，随着MAT，我不断创建多个Threads，每个surfaceCreate

我认为我需要这些线程，但是当用户浏览我的应用程序时，这种方法会导致多个 ViewThread 实例，这是内存泄漏。

如何重新组织创建和处理线程的方式，以免发生这种情况，或者如何阻止泄漏的发生？

@Override
public void surfaceCreated(SurfaceHolder holder) {
    loading=false;
    if (!mThread.isAlive()){
        mThread = new ViewThread(this);
        mThread.setMenuRunning(true);
        mThread.start();
    }
}

@Override
public void surfaceDestroyed(SurfaceHolder holder) {

    if (mThread.isAlive()){ 
        mThread.setMenuRunning(false);
    }
}

我五次打开并离开游戏的 Career Activity，这就是 MAT 上显示的内容

编辑：我发现依赖surfaceDestroyed 来销毁我的线程是不可靠的。我现在从不同的方法调用适当的线程破坏调用，触发onPause。

Answer 1

您应该使用 Wea​​kReference 来引用您线程中的职业。这样，当没有更多对职业的硬引用时，引用将被清除。

您可以通过右键单击职业并选择 Path To GC Roots 来跟踪 MAT 中的所有引用，然后选择所有引用。这将向您显示内存中保留的对象的路径。确保在完成活动后清除这些引用，或者使用 Wea​​kReferences 让 GC 自动清除它们。

Answer 2

在surfaceDestroyed 中，您应该等待以确保线程在您返回之前停止。

您可以参考this question了解更多详情

@Override
public void surfaceDestroyed(SurfaceHolder holder) {
    boolean retry = true;
    mThread.setRunning(false);
    while (retry) {
        try {
            mThread.join();
            retry = false;
        } catch (InterruptedException e) {
        }
    }
}

Answer 3

所以我通过注释一行来修复它：

@Override
public void surfaceCreated(SurfaceHolder holder) {
    loading=false;
    if (!mThread.isAlive()){
        //mThread = new ViewThread(this);
        mThread.setMenuRunning(true);
        mThread.start();
    }
}

这也与WeakReference 和SurfaceDestroyed 答案结合使用。 我稍后会测试它并确定它是否只是删除了那一行，或者它与弱引用的组合，或者其他东西，然后奖励答案