鉴于此代码:
public class MainActivity extends FragmentActivity implements ActionBar.TabListener {
public static final int MESSAGE_NOT_CONNECTED = 1;
@Override
public void onCreate(Bundle savedInstanceState) {
super.onCreate(savedInstanceState);
setContentView(R.layout.main_layout);
}
// -------------------------------------------------
public final void setStatus(int Rid) {
final ActionBar actionBar = getActionBar();
actionBar.setSubtitle(Rid);
}
// -------------------------------------------------
static Handler mHandler = new Handler() {
@Override
public void handleMessage(Message msg) {
switch (msg.what) {
case MESSAGE_NOT_CONNECTED:
setStatus(R.string.title_not_connected);
break;
}
}
}
}
我收到编译错误:无法对非静态方法setStatus(int)进行静态引用...
这是有道理的,因为setStatus()中的getActionBar()是一个非静态方法。
由于警告,我使Handler类保持静态:此Handler类应该是静态的,否则可能会发生泄漏。
问题:我如何从静态处理程序中正确访问setStatus()方法?
编辑:新的处理程序代码就是答案。
static class hHandler extends Handler {
private final WeakReference<MainActivity> mTarget;
hHandler(MainActivity target) {
mTarget = new WeakReference<MainActivity>(target);
}
@Override
public void handleMessage(Message msg) {
MainActivity target = mTarget.get();
If(target == null) {
return;
}
switch (msg.what) {
case MESSAGE_NOT_CONNECTED:
target.setStatus(R.string.title_not_connected);
break;
}
}
}
答案 0 :(得分:9)
尝试使用WeakReference
,如article中所述。
答案 1 :(得分:1)
由于您现在使用WeakReference
,mTarget.get()
可能会返回null
。在您编辑的代码中,在执行target
之前,您不会检查null
是否为target.setStatus(R.string.title_not_connected)
。因此,如果弱引用对象已经过GC,这可能会抛出NullPointerException
。
答案 2 :(得分:0)
在我的活动的onDestroy方法中,我打电话:
this.myHandler.removeCallbacksAndMessages(null);
这并没有摆脱“此Handler类应该是静态的或可能发生泄漏”的警告,但我相信它会破坏消息,从而阻止泄漏。我的处理程序类是我的活动的内部非静态类。我的活动有一个MyHandler myHandler的实例。
当我这样做时,不调用处理程序的handleMessage方法,我假设这意味着包含处理程序的消息,其中包含对该活动的引用被销毁。由于我没有使用任何泄漏测试工具对其进行测试,因此我可以发表评论。这是我复制这个想法的地方:http://www.androiddesignpatterns.com/2013/01/inner-class-handler-memory-leak.html回答者:Cyril 2013年1月15日上午7:50