Утечка памяти обработчика?

Question

У меня есть обработчик, который объявлен в главном потоке:

mainHandler = new Handler() { 

      @Override 
      public void handleMessage(Message msg) { 

       switch (msg.what) { 

        case 1: 

         Bundle bundle = msg.getData(); 
         mTextView.setText(bundle.getString("message")); 
         break; 

        . . . 

        default: 
         super.handleMessage(msg); 
         break; 
       } 
      } 
     }; 

Где mTextView представляет собой определенный TextView OnCreate().

У меня есть задача, которая используется в отдельной теме. В Runnable магазины mainHandler от основного потока и говорит ему отправлять сообщения:

public class SomeRunnable implements Runnable { 

    private Handler mHandler; 

    public SomeRunnable(Handler handler) throws IOException { 
     . . .  
     mHandler = handler; 
    } 

    @Override 
    public void run() { 

     while (!Thread.currentThread().isInterrupted()) { 
      try { 
        . . . 

        Message msg = mHandler.obtainMessage(); 
        . . . 
        mHandler.sendMessage(msg); 
       } 

      } catch (IOException e) { 
       Log.e("Error", e.hashCode() + ": " + e.getLocalizedMessage()); 
      } 
     } 
    } 

Я видел, что вы потенциально можете создать утечку памяти с помощью таких методов, как Handler#postDelayed() если класс Handler не является статичным. Но я использую Handler#sendMessage(), который мгновенно помещает сообщение в очередь сообщений.

Я все еще в опасности иметь утечку памяти? Даже с:

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

     mThread.interrupt(); 
     mainHandler.removeCallbacksAndMessages(null); 
    } 

Спасибо!

Answer 1

Для решения всех случайностей, вы могли бы сделать mHandler в WeakReference, как показано ниже:

public class SomeRunnable implements Runnable { 

    private WeakReference<Handler> mHandlerRef; 

    public SomeRunnable(Handler handler) throws IOException { 
     . . .  
     mHandlerRef = new WeakReference<Handler>(handler); 
    } 

    @Override 
    public void run() { 

     while (!Thread.currentThread().isInterrupted()) { 
      try { 
        . . . 
        Handler mHandler = mHandlerRef.get(); 
        if(mHandler != null) { 
         Message msg = mHandler.obtainMessage(); 
         . . . 
         mHandler.sendMessage(msg); 
        } 
       } 

      } catch (IOException e) { 
       Log.e("Error", e.hashCode() + ": " + e.getLocalizedMessage()); 
      } 
     } 
    } 
}