Будет ли это конкретное использование анонимного 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, являющийся анонимным классом, должен содержать явную ссылку на фрагмент, не так ли? И это помешало бы Фрагменту быть собранным Мусором?

Должен ли я беспокоиться о возникшей утечке памяти и как ее предотвратить?

Помогите, если я создам экземпляр класса, который расширяет 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.

Ваше предложение взломать его в статическом внутреннем классе является хорошим, так как это удалит эту ссылку на внешний класс и, следовательно, избавит вас от потенциальной утечки памяти.

ОДНАКО - В вашем случае, поскольку ваш предложенный статический класс в любом случае сохраняет привязку к экземпляру внешнего класса, от этого нет никакой пользы.