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