This Handler class should be static or leaks might occur Android

dmw_zgl 2015-03-18

展开全文

首先解释下这句话This Handler class should be static or leaks might occur，大致意思就是说：Handler类应该定义成静态类，否则可能导致内存泄露。

具体如何解决，在国外有人提出，如下：

Issue: Ensures that Handler classes do not hold on to a reference to an outer class

In Android, Handler classes should be static or leaks might occur. Messages enqueued on the application thread's MessageQueue also retain their target Handler. If the Handler is an inner class, its outer class will be retained as well. To avoid leaking the outer class, declare the Handler as a static nested class with a WeakReference to its outer class.

大体翻译如下:

Handler 类应该应该为static类型，否则有可能造成泄露。在程序消息队列中排队的消息保持了对目标Handler类的应用。如果Handler是个内部类，那么它也会保持它所在的外部类的引用。为了避免泄露这个外部类，应该将Handler声明为static嵌套类，并且使用对外部类的弱应用。

使用范例：

static class MyHandler extends Handler {
WeakReference<PopupActivity> mActivity;
MyHandler(PopupActivity activity) {
mActivity = new WeakReference<PopupActivity>(activity);
}
@Override
public void handleMessage(Message msg) {
PopupActivity theActivity = mActivity.get();
switch (msg.what) {
case 0:
theActivity.popPlay.setChecked(true);
break;
}
}
};
MyHandler ttsHandler = new MyHandler(this);
private Cursor mCursor;
private void test() {
ttsHandler.sendEmptyMessage(0);
}

原文：http://www.cnblogs.com/savagemorgan/archive/2013/01/23/2872371.html

疑问：是否有其它解决方法？

这个提示就是由于Handler的直接引用会导致相关的Activity、Service等无法被GC。如果这么弱应用的话，会出现空指针，有其它解决方法？

抽时间研究下。

==================================================================================================================================

原始代码：

public class MainActivity extends Activity {
private static int urlIndex = 0;
private final static String TAG = MainActivity.class.getSimpleName();
private static final String[] url = {
"http://vdn.apps.cntv.cn/api/getLiveUrlCommonRedirectApi.do?channel=pa://cctv_p2p_hdcctv1&type=ipad",
"http://74.82.62.53:1935/liverepeater/13.stream/playlist.m3u8", "http://rtmp.cntv./live/cctv3/playlist.m3u8", };
private static final int MSG_PLAY = 100;
private static final int MSG_RUN_ADB = 101;
Handler playHandler = new Handler() {
@Override
public void handleMessage(Message msg) {
switch (msg.what) {
case MSG_PLAY:
urlIndex = urlIndex > url.length - 1 ? 0 : urlIndex;
videoView.setVideoPath(url[urlIndex]);
++urlIndex;
break;
case MSG_RUN_ADB:
killMediaServer();
break;
}
}
};
@Override
protected void onCreate(Bundle savedInstanceState) {
super.onCreate(savedInstanceState);
requestWindowFeature(Window.FEATURE_NO_TITLE);
HHVideoView.create();
setContentView(R.layout.activity_main);
videoView = ((HHVideoView) findViewById(R.id.videoView));
videoView.setOnPreparedListener(mPreparedListener);
videoView.setOnCompletionListener(mCompletionListener);
videoView.setOnErrorListener(mOnErrorListener);
playHandler.sendEmptyMessage(MSG_PLAY);
}
private HHVideoView videoView = null;
private MediaPlayer.OnPreparedListener mPreparedListener = new MediaPlayer.OnPreparedListener() {
public void onPrepared(MediaPlayer paramMediaPlayer) {
// playerHandler.sendEmptyMessage(uiAction.MEDIAPLAYER_ONPREPAREED);
videoView.start();
}
};
private MediaPlayer.OnCompletionListener mCompletionListener = new MediaPlayer.OnCompletionListener() {
public void onCompletion(MediaPlayer paramMediaPlayer) {
}
};
private MediaPlayer.OnErrorListener mOnErrorListener = new MediaPlayer.OnErrorListener() {
public boolean onError(MediaPlayer paramMediaPlayer, int paramInt1, int paramInt2) {
return false;
}
};
@Override
public boolean onCreateOptionsMenu(Menu menu) {
// Inflate the menu; this adds items to the action bar if it is present.
return true;
}
public boolean onKeyDown(int keyCode, KeyEvent event) {
if (event.getAction() == KeyEvent.ACTION_DOWN) {
switch (keyCode) {
case KeyEvent.KEYCODE_0:
playHandler.sendEmptyMessage(MSG_RUN_ADB);
break;
case KeyEvent.KEYCODE_DPAD_DOWN:
case KeyEvent.KEYCODE_DPAD_UP:
playHandler.sendEmptyMessage(MSG_PLAY);
break;
}
}
return super.onKeyDown(keyCode, event);
}
}

修改后的代码：

//activity code
public class MainActivity extends Activity {
private static int urlIndex = 0;
private final static String TAG = MainActivity.class.getSimpleName();
private static final String[] url = {
"http://vdn.apps.cntv.cn/api/getLiveUrlCommonRedirectApi.do?channel=pa://cctv_p2p_hdcctv1&type=ipad",
"http://74.82.62.53:1935/liverepeater/13.stream/playlist.m3u8", "http://rtmp.cntv./live/cctv3/playlist.m3u8", };
PlayHandler playHandler ;
@Override
protected void onCreate(Bundle savedInstanceState) {
super.onCreate(savedInstanceState);
requestWindowFeature(Window.FEATURE_NO_TITLE);
HHVideoView.create();
setContentView(R.layout.activity_main);
videoView = ((HHVideoView) findViewById(R.id.videoView));
videoView.setOnPreparedListener(mPreparedListener);
videoView.setOnCompletionListener(mCompletionListener);
videoView.setOnErrorListener(mOnErrorListener);
playHandler.sendEmptyMessage(PlayHandler.MSG_PLAY);
}
private HHVideoView videoView = null;
private MediaPlayer.OnPreparedListener mPreparedListener = new MediaPlayer.OnPreparedListener() {
public void onPrepared(MediaPlayer paramMediaPlayer) {
// playerHandler.sendEmptyMessage(uiAction.MEDIAPLAYER_ONPREPAREED);
videoView.start();
}
};
private MediaPlayer.OnCompletionListener mCompletionListener = new MediaPlayer.OnCompletionListener() {
public void onCompletion(MediaPlayer paramMediaPlayer) {
}
};
private MediaPlayer.OnErrorListener mOnErrorListener = new MediaPlayer.OnErrorListener() {
public boolean onError(MediaPlayer paramMediaPlayer, int paramInt1, int paramInt2) {
return false;
}
};
@Override
public boolean onCreateOptionsMenu(Menu menu) {
// Inflate the menu; this adds items to the action bar if it is present.
return true;
}
public boolean onKeyDown(int keyCode, KeyEvent event) {
if (event.getAction() == KeyEvent.ACTION_DOWN) {
switch (keyCode) {
case KeyEvent.KEYCODE_0:
playHandler.sendEmptyMessage(PlayHandler.MSG_RUN_ADB);
break;
case KeyEvent.KEYCODE_DPAD_DOWN:
case KeyEvent.KEYCODE_DPAD_UP:
playHandler.sendEmptyMessage(PlayHandler.MSG_PLAY);
break;
}
}
return super.onKeyDown(keyCode, event);
}
public void setVideoPath() {
urlIndex = urlIndex > url.length - 1 ? 0 : urlIndex;
videoView.setVideoPath(url[urlIndex]);
++urlIndex;
}
}