这种对匿名 Runnable 的特殊使用会导致内存泄漏吗？

Question

private class ProductFragment extends Fragment implements OnGetProductFromDBListener {

    Activity a;

    void onCreate {
         a = getActivity();
        new GetProductFromDB();
    }


    void onGetProductSuccess(Product product) { // This is not called from the main thread because of the implementation of GetProductFromDB
        a.runOnUiThread(new Runnable() {

            void run() {
                // do some UI and Views related stuff
            }

        });
    } 

}

现在，作为匿名类的 Runnable 应该包含对 Fragment 的显式引用，对吧？这会阻止碎片被垃圾收集吗？

我应该担心发生内存泄漏吗？我应该如何防止它？

如果我实例化一个扩展 Runnable 并使其成为静态的类，会有帮助吗？

private static class RunnableTask extends Runnable {


  ProductFragment fragment;

   public RunnableTask (ProductFragment fragment) {
       this.fragment = fragment;
    }

    void run() {

       if(fragment.someBoolean) {
           fragment.doSomething();
       }

    }
}

RunnableTask runnable = new RunnableTask();
a.runOnUiThread(runnable)

Answer 1

严格来说，它不一定会导致内存泄漏。但是 - 在线程运行期间，它将持有对外部类的引用（在您的情况下为 ProductFragment）。因此，是否是内存泄漏将取决于线程将运行多长时间，以及它是否最终成为 ProductFragment 类的唯一引用。

您将其分解为静态内部类的建议是一个很好的建议，因为这将删除对外部类的引用，从而消除任何潜在内存泄漏的机会。

但是 - 在您的情况下，鉴于您提议的静态类仍然保留外部类实例的引用，这样做没有任何好处。