被一个诡异的bug折腾了半天,当找到原因时哭笑不得。先看如下代码,大家分析一下打出来的会是什么:
public class TestDebug {
private static HashMap<String, BleDeviceProp> mPropCache = new HashMap<String, BleDeviceProp>();
public static void main(String[] args) {
// TODO Auto-generated method stub
BleDeviceProp prop = new BleDeviceProp("one");
prop.name = "apple";
mPropCache.put(prop.mac, prop);
setPropName(prop.mac, "orange");
System.out.println(prop.name);
}
private static class BleDeviceProp {
private String mac;
private String name;
BleDeviceProp(String mac) {
this.mac = mac;
}
}
private static void setPropName(String mac, String name) {
setProp(mac, new PropSetter("name") {
@Override
public boolean setProp(BleDeviceProp prop) {
// TODO Auto-generated method stub
prop.name = name;
return false;
}
});
}
private static void setProp(String mac, PropSetter setter) {
BleDeviceProp prop = mPropCache.get(mac);
if (prop == null) {
prop = new BleDeviceProp("");
mPropCache.put(mac, prop);
}
setter.setProp(prop);
}
private interface IPropSetter {
boolean setProp(BleDeviceProp prop);
}
private static abstract class PropSetter implements IPropSetter {
String name;
PropSetter(String name) {
this.name = name;
}
}
}
按理说这里给name设置成orange,打出来的应该是orange才对,但事实上打出来的是name。很奇怪吧,原因在哪里?
原因在于这个PropSetter内部也有一个成员变量为name,没有加修饰符,所以默认为protected。在回调里,这个name的优先级高于setPropName的参数name,所以被抢占了。解决办法很简单,给PropSetter的name改成private的,或者干脆别让他们名称冲突就可以了。