一个让我很无语的bug

被一个诡异的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的，或者干脆别让他们名称冲突就可以了。